Enable ESLint `function-paren-newline` `multiline`

Per discussion.

.eslintrc.js

@@ -21,7 +21,7 @@ module.exports = {
     }],
 
     // putting params on their own line helps stay within line length limit
-    'function-paren-newline': ['error', 'consistent'],
+    'function-paren-newline': ['error', 'multiline'],
 
     // 90 characters allows three+ side-by-side screens on a standard-size monitor
     'max-len': ['error', {

js/background.js

@@ -497,18 +497,19 @@
 
   /* eslint-enable */
   /* jshint ignore:start */
-  function createMessageHandler(
+  function createMessageHandler({
-    { createMessage, getMessageDescriptor, handleProfileUpdate }
+    createMessage,
-  ) {
+    getMessageDescriptor,
+    handleProfileUpdate,
+  }) {
     return async (event) => {
       const { data, confirm } = event;
 
       const messageDescriptor = getMessageDescriptor(data);
 
-      const isProfileUpdate = Boolean(
+      const { PROFILE_KEY_UPDATE } = textsecure.protobuf.DataMessage.Flags;
-        // eslint-disable-next-line no-bitwise
+      // eslint-disable-next-line no-bitwise
-        data.message.flags & textsecure.protobuf.DataMessage.Flags.PROFILE_KEY_UPDATE
+      const isProfileUpdate = Boolean(data.message.flags & PROFILE_KEY_UPDATE);
-      );
       if (isProfileUpdate) {
         return handleProfileUpdate({ data, confirm, messageDescriptor });
       }
@@ -522,7 +523,8 @@
 
       const upgradedMessage = await Message.upgradeSchema(data.message);
       await ConversationController.getOrCreateAndWait(
-        messageDescriptor.id, messageDescriptor.type
+        messageDescriptor.id,
+        messageDescriptor.type
       );
       return message.handleDataMessage(
         upgradedMessage,
@@ -533,9 +535,11 @@
   }
 
   // Received:
-  async function handleMessageReceivedProfileUpdate(
+  async function handleMessageReceivedProfileUpdate({
-    { data, confirm, messageDescriptor }
+    data,
-  ) {
+    confirm,
+    messageDescriptor,
+  }) {
     const profileKey = data.message.profileKey.toArrayBuffer();
     const sender = await ConversationController.getOrCreateAndWait(
       messageDescriptor.id,
@@ -552,11 +556,10 @@
   });
 
   // Sent:
-  async function handleMessageSentProfileUpdate(
+  async function handleMessageSentProfileUpdate({ confirm, messageDescriptor }) {
-    { confirm, messageDescriptor }
-  ) {
     const conversation = await ConversationController.getOrCreateAndWait(
-      messageDescriptor.id, messageDescriptor.type
+      messageDescriptor.id,
+      messageDescriptor.type
     );
     await conversation.save({ profileSharing: true });
     return confirm();

js/models/conversations.js

@@ -617,9 +617,8 @@
           now
         );
 
-        const upgradedAttachments = await Promise.all(
+        const upgradedAttachments =
-          attachments.map(Attachment.upgradeSchema)
+          await Promise.all(attachments.map(Attachment.upgradeSchema));
-        );
         const message = this.messageCollection.add({
           body,
           conversationId: this.id,
@@ -658,16 +657,14 @@
           profileKey = storage.get('profileKey');
         }
 
-        message.send(
+        message.send(sendFunc(
-          sendFunc(
+          this.get('id'),
-            this.get('id'),
+          body,
-            body,
+          upgradedAttachments,
-            upgradedAttachments,
+          now,
-            now,
+          this.get('expireTimer'),
-            this.get('expireTimer'),
+          profileKey
-            profileKey
+        ));
-          )
-        );
       });
     },
     /* jshint ignore:end */