Adding RenderWidgetHostViewChildFrame for OOPIF view.

content/common/frame_messages.h

 // Copyright 2013 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.
 
 // IPC messages for interacting with frames.
 // Multiply-included message file, hence no include guard.
 
+#include "cc/output/begin_frame_args.h"
+#include "cc/output/compositor_frame.h"
+#include "cc/output/compositor_frame_ack.h"
 #include "content/common/content_export.h"
 #include "content/public/common/common_param_traits.h"
 #include "ipc/ipc_message_macros.h"
+#include "url/gurl.h"
 
 #undef IPC_MESSAGE_EXPORT
 #define IPC_MESSAGE_EXPORT CONTENT_EXPORT
 
 #define IPC_MESSAGE_START FrameMsgStart
 
+// -----------------------------------------------------------------------------
+// Messages sent from the browser to the renderer.
+
+// When HW accelerated buffers are swapped in an out of process child frame
+// renderer, the message is forwarded to the top level frame renderer to
+// notify it of a new texture available for compositing.
+IPC_MESSAGE_ROUTED4(FrameMsg_ChildFrameBuffersSwapped,
+                    gfx::Size /* size */,
+                    std::string /* mailbox_name */,
+                    int /* route_id */,

[piman, 2013/12/17 00:14:57]

route_id of what? How does that relate to the implicit routing id from this
ROUTED message?

[awong, 2013/12/17 02:45:27]

Ooh...good catch. It's the gpu_route_id.  Comes from
GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params.route_id (later in the
plumbing CL).

+                    int /* gpu_host_id */)

[piman, 2013/12/17 00:14:57]

2 things about these messages:
1- I don't see the renderer-side handlers for these.
2- These seem to duplicate the
BrowserPluginMsg_BuffersSwapped/BrowserPluginMsg_CompositorFrameSwapped
messages... is it possible to factor the logic between the 2? It's a non-trivial
amount of code with subtle things (though we're trying to make it better), and
I'd hate to duplicate it all (see the BrowserPluginCompositingHelper)

[awong, 2013/12/17 02:45:27]

This is part 1 of a set of CLs that are being upstreamed.  The big-ball
prototype is here:
  https://codereview.chromium.org/download/issue23889010_209001.diff

As for combining with BrowserPluginMsg_, yes, that is definitely on the roadmap.
Ken was working with Fady on this. I think the plan was to land these
first...unblock the Site Isolation work, and the migrate webview over.  Is that
an okay plan, or do you want me to find a way to combine in this CL?

+
+IPC_MESSAGE_ROUTED4(FrameMsg_ChildFrameCompositorFrameSwapped,
+                    cc::CompositorFrame /* frame */,
+                    int /* route_id */,
+                    uint32 /* output_surface_id */,
+                    int /* renderer_host_id */)
+
+// -----------------------------------------------------------------------------
+// Messages sent from the renderer to the browser.
+
 // Sent by the renderer when a child frame is created in the renderer. The
 // |parent_frame_id| and |frame_id| are NOT routing ids. They are
 // renderer-allocated identifiers used for tracking a frame's creation.
 //
 // Each of these messages will have a corresponding FrameHostMsg_Detach message
 // sent when the frame is detached from the DOM.
 //
 // TOOD(ajwong): replace parent_render_frame_id and frame_id with just the
 // routing ids.
 IPC_SYNC_MESSAGE_CONTROL4_1(FrameHostMsg_CreateChildFrame,
@@ skipping 51 matching lines @@
 // create a plugin.  The browser will create the plugin process if
 // necessary, and will return a handle to the channel on success.
 // On error an empty string is returned.
 IPC_SYNC_MESSAGE_CONTROL4_2(FrameHostMsg_OpenChannelToPlugin,
                             int /* render_frame_id */,
                             GURL /* url */,
                             GURL /* page_url */,
                             std::string /* mime_type */,
                             IPC::ChannelHandle /* channel_handle */,
                             content::WebPluginInfo /* info */)
+

[nasko, 2013/12/16 23:16:32]

nit: just one empty line

[awong, 2013/12/17 02:45:27]

Done.

+
+// Acknowledge that we presented a HW buffer and provide a sync point
+// to specify the location in the command stream when the compositor
+// is no longer using it.
+IPC_MESSAGE_ROUTED4(FrameHostMsg_BuffersSwappedACK,
+                    std::string /* mailbox_name */,
+                    int /* route_id */,
+                    int /* gpu_host_id */,
+                    uint32 /* sync_point */)