content/gpu: Remove separate queue for out-of-order messages.

content/common/gpu/gpu_channel.h

Index: content/common/gpu/gpu_channel.h
diff --git a/content/common/gpu/gpu_channel.h b/content/common/gpu/gpu_channel.h
index 89ae376968ed041205db45ed0282d68ce23d4f4c..989d8612494d797f5a025bad6183021461a606a5 100644
--- a/content/common/gpu/gpu_channel.h
+++ b/content/common/gpu/gpu_channel.h
@@ -180,6 +180,11 @@ class CONTENT_EXPORT GpuChannel
 
   void HandleMessage();
 
+  // Some messages such as WaitForGetOffsetInRange and WaitForTokenInRange are
+  // processed as soon as possible because the client is blocked until they
+  // are completed.
+  void HandleOutOfOrderMessage(const IPC::Message& msg);
+
  protected:
   // The message filter on the io thread.
   scoped_refptr<GpuChannelMessageFilter> filter_;
@@ -302,7 +307,8 @@ class CONTENT_EXPORT GpuChannel
 // - it generates mailbox names for clients of the GPU process on the IO thread.
 class GpuChannelMessageFilter : public IPC::MessageFilter {
  public:
-  GpuChannelMessageFilter(GpuChannelMessageQueue* message_queue,
+  GpuChannelMessageFilter(const base::WeakPtr<GpuChannel>& gpu_channel,
+                          GpuChannelMessageQueue* message_queue,
                           gpu::SyncPointManager* sync_point_manager,
                           base::SingleThreadTaskRunner* task_runner,
                           bool future_sync_points);
@@ -362,6 +368,7 @@ class GpuChannelMessageFilter : public IPC::MessageFilter {
   // It is reset when we transition to IDLE.
   base::TimeDelta max_preemption_time_;
 
+  base::WeakPtr<GpuChannel> gpu_channel_;
   // The message_queue_ is used to handle messages on the main thread.
   scoped_refptr<GpuChannelMessageQueue> message_queue_;
   IPC::Sender* sender_;
@@ -378,9 +385,6 @@ class GpuChannelMessageFilter : public IPC::MessageFilter {
 
   // True if this channel can create future sync points.
   bool future_sync_points_;
-
-  // This number is only ever incremented/read on the IO thread.
-  static uint32_t global_order_counter_;
 };
 
 struct GpuChannelMessage {
@@ -392,9 +396,9 @@ struct GpuChannelMessage {
   bool retire_sync_point;
   uint32 sync_point;
 
-  GpuChannelMessage(uint32_t order_num, const IPC::Message& msg)
-      : order_number(order_num),
-        time_received(base::TimeTicks::Now()),
+  GpuChannelMessage(const IPC::Message& msg)
+      : order_number(0),
+        time_received(base::TimeTicks()),
         message(msg),
         retire_sync_point(false),
         sync_point(0) {}
@@ -424,12 +428,11 @@ class GpuChannelMessageQueue
 
   // Should be called after a message returned by GetNextMessage is processed.
   // Returns true if there are more messages on the queue.
-  bool MessageProcessed(uint32_t order_number);
+  bool MessageProcessed();
 
-  void PushBackMessage(uint32_t order_number, const IPC::Message& message);
+  void PushBackMessage(const IPC::Message& message);
 
   bool GenerateSyncPointMessage(gpu::SyncPointManager* sync_point_manager,
-                                uint32_t order_number,
                                 const IPC::Message& message,
                                 bool retire_sync_point,
                                 uint32_t* sync_point_number);
@@ -447,7 +450,8 @@ class GpuChannelMessageQueue
 
   void PushMessageHelper(scoped_ptr<GpuChannelMessage> msg);
 
-  bool HasQueuedMessagesHelper() const;
+  // This number is only ever incremented/read on the IO thread.
+  static uint32_t global_order_counter_;
 
   bool enabled_;
 
@@ -455,9 +459,8 @@ class GpuChannelMessageQueue
   uint32_t unprocessed_order_num_;
   // Both deques own the messages.
   std::deque<GpuChannelMessage*> channel_messages_;
-  std::deque<GpuChannelMessage*> out_of_order_messages_;
 
-  // This lock protects enabled_, unprocessed_order_num_, and both deques.
+  // This lock protects enabled_, unprocessed_order_num_, and channel_messages_.
   mutable base::Lock channel_messages_lock_;
 
   // Last finished IPC order number. Not protected by a lock as it's only