Make EstablishGpuChannel synchronous. Remove obsolete Synchronize msg.

content/common/gpu/gpu_messages.h

 // Copyright (c) 2011 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.
 
 #include <vector>
 #include <string>
 
 
 #include "base/shared_memory.h"
 #include "content/common/common_param_traits.h"
@@ skipping 80 matching lines @@
 // the GPU process reuses an existing channel to that process if it exists.
 // This ID is a unique opaque identifier generated by the browser process.
 IPC_MESSAGE_CONTROL1(GpuMsg_EstablishChannel,
                      int /* renderer_id */)
 
 // Tells the GPU process to close the channel identified by IPC channel
 // handle.  If no channel can be identified, do nothing.
 IPC_MESSAGE_CONTROL1(GpuMsg_CloseChannel,
                      IPC::ChannelHandle /* channel_handle */)
 
-// Provides a synchronization point to guarantee that the processing of
-// previous asynchronous messages (i.e., GpuMsg_EstablishChannel) has
-// completed. (This message can't be synchronous because the
-// GpuProcessHost uses an IPC::ChannelProxy, which sends all messages
-// asynchronously.) Results in a GpuHostMsg_SynchronizeReply.
-IPC_MESSAGE_CONTROL0(GpuMsg_Synchronize)
-
 IPC_MESSAGE_CONTROL3(GpuMsg_VisibilityChanged,
                      int32, /*render_view_id */
                      int32, /* renderer_id */
                      bool /* is_visible */)
 
 // Tells the GPU process to create a new command buffer that renders directly
 // to a native view. A corresponding GpuCommandBufferStub is created.
 IPC_MESSAGE_CONTROL4(GpuMsg_CreateViewCommandBuffer,
                      gfx::PluginWindowHandle, /* compositing_surface */
                      int32, /* render_view_id */
@@ skipping 31 matching lines @@
 
 // Tells the GPU process to remove all contexts.
 IPC_MESSAGE_CONTROL0(GpuMsg_Clean)
 
 // Tells the GPU process to crash.
 IPC_MESSAGE_CONTROL0(GpuMsg_Crash)
 
 // Tells the GPU process to hang.
 IPC_MESSAGE_CONTROL0(GpuMsg_Hang)
 
-// The browser sends this to a renderer process in response to a
-// GpuHostMsg_EstablishGpuChannel message.
-IPC_MESSAGE_CONTROL3(GpuMsg_GpuChannelEstablished,
-                     IPC::ChannelHandle /* handle to channel */,
-                     base::ProcessHandle /* renderer_process_for_gpu */,
-                     GPUInfo /* stats about GPU process*/)
-
 //------------------------------------------------------------------------------
 // GPU Host Messages
 // These are messages to the browser.
 
 // A renderer sends this when it wants to create a connection to the GPU
 // process. The browser will create the GPU process if necessary, and will
 // return a handle to the channel via a GpuChannelEstablished message.
-IPC_MESSAGE_CONTROL1(GpuHostMsg_EstablishGpuChannel,
-                     content::CauseForGpuLaunch)
-
-// A renderer sends this to the browser process to provide a synchronization
-// point for GPU operations, in particular to make sure the GPU channel has
-// been established.
-IPC_SYNC_MESSAGE_CONTROL0_0(GpuHostMsg_SynchronizeGpu)
+IPC_SYNC_MESSAGE_CONTROL1_3(GpuHostMsg_EstablishGpuChannel,
+                            content::CauseForGpuLaunch,
+                            IPC::ChannelHandle /* handle to channel */,
+                            base::ProcessHandle /* renderer_process_for_gpu */,
+                            GPUInfo /* stats about GPU process*/)
 
 // A renderer sends this to the browser process when it wants to
 // create a GL context associated with the given view_id.
 IPC_SYNC_MESSAGE_CONTROL3_1(GpuHostMsg_CreateViewCommandBuffer,
                             gfx::PluginWindowHandle, /* view */
                             int32, /* render_view_id */
                             GPUCreateCommandBufferConfig, /* init_params */
                             int32 /* route_id */)
 
 // Response from GPU to a GpuHostMsg_EstablishChannel message.
@@ skipping 14 matching lines @@
 // Response from GPU to a GpuMsg_CollectGraphicsInfo.
 IPC_MESSAGE_CONTROL1(GpuHostMsg_GraphicsInfoCollected,
                      GPUInfo /* GPU logging stats */)
 
 // Message from GPU to add a GPU log message to the about:gpu page.
 IPC_MESSAGE_CONTROL3(GpuHostMsg_OnLogMessage,
                      int /*severity*/,
                      std::string /* header */,
                      std::string /* message */)
 
-// Response from GPU to a GpuMsg_Synchronize message.
-IPC_MESSAGE_CONTROL0(GpuHostMsg_SynchronizeReply)
-
 #if defined(TOOLKIT_USES_GTK) && !defined(TOUCH_UI) || defined(OS_WIN)
 // Resize the window that is being drawn into. It's important that this
 // resize be synchronized with the swapping of the front and back buffers.
 IPC_MESSAGE_CONTROL4(GpuHostMsg_ResizeView,
                      int32 /* renderer_id */,
                      int32 /* render_view_id */,
                      int32 /* command_buffer_route_id */,
                      gfx::Size /* size */)
 #elif defined(OS_MACOSX)
 // This message, used on Mac OS X 10.6 and later (where IOSurface is
@@ skipping 281 matching lines @@
 
 // Confirm decoder has been aborted.
 IPC_MESSAGE_ROUTED0(AcceleratedVideoDecoderHostMsg_AbortDone)
 
 // Decoder has faced end of stream marker in the stream.
 IPC_MESSAGE_ROUTED0(AcceleratedVideoDecoderHostMsg_EndOfStream)
 
 // Video decoder has encountered an error.
 IPC_MESSAGE_ROUTED1(AcceleratedVideoDecoderHostMsg_ErrorNotification,
                     uint32) /* Error ID */