Remove unused IPC::Message priority.

ipc/ipc_message.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef IPC_IPC_MESSAGE_H_
 #define IPC_IPC_MESSAGE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 16 matching lines @@
 class FileDescriptorSet;
 
 namespace IPC {
 
 //------------------------------------------------------------------------------
 
 struct LogData;
 
 class IPC_EXPORT Message : public Pickle {
  public:
-  enum PriorityValue {
-    PRIORITY_LOW = 1,
-    PRIORITY_NORMAL,
-    PRIORITY_HIGH
-  };
-
   // Bit values used in the flags field.
   // Upper 24 bits of flags store a reference number, so this enum is limited to
   // 8 bits.
   enum {
-    PRIORITY_MASK     = 0x03,  // Low 2 bits of store the priority value.
     SYNC_BIT          = 0x04,

[jam, 2013/10/25 00:26:24]

while it's a positive that we get rid of unused priority bits, it's a bit odd
reading this and seeing unused bits.

hmm, perhaps just have

UNUSED = 0x03;
?

     REPLY_BIT         = 0x08,
     REPLY_ERROR_BIT   = 0x10,
     UNBLOCK_BIT       = 0x20,
     PUMPING_MSGS_BIT  = 0x40,
     HAS_SENT_TIME_BIT = 0x80,
   };
 
   virtual ~Message();
 
   Message();
 
-  // Initialize a message with a user-defined type, priority value, and
-  // destination WebView ID.
-  Message(int32 routing_id, uint32 type, PriorityValue priority);
+  // Initialize a message with routing ID and a user-defined type.
+  Message(int32 routing_id, uint32 type);
 
   // Initializes a message from a const block of data.  The data is not copied;
   // instead the data is merely referenced by this message.  Only const methods
   // should be used on the message when initialized this way.
-  Message(const char* data, int data_len);
+  Message(const char* data, size_t data_len);
 
   Message(const Message& other);
   Message& operator=(const Message& other);
 
-  PriorityValue priority() const {
-    return static_cast<PriorityValue>(header()->flags & PRIORITY_MASK);
-  }
-
   // True if this is a synchronous message.
   void set_sync() {
     header()->flags |= SYNC_BIT;
   }
   bool is_sync() const {
     return (header()->flags & SYNC_BIT) != 0;
   }
 
   // Set this on a reply to a synchronous message.
   void set_reply() {
@@ skipping 198 matching lines @@
   MSG_ROUTING_NONE = -2,
 
   // indicates a general message not sent to a particular tab.
   MSG_ROUTING_CONTROL = kint32max,
 };
 
 #define IPC_REPLY_ID 0xFFFFFFF0  // Special message id for replies
 #define IPC_LOGGING_ID 0xFFFFFFF1  // Special message id for logging
 
 #endif  // IPC_IPC_MESSAGE_H_