Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1245)

Unified Diff: content/common/frame_messages.h

Issue 100473010: Adding RenderWidgetHostViewChildFrame for OOPIF view. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix comment. also squished change locally. Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index 87f769b5fd0455b106a38bd6d9a7a4c54869ce94..ab7c9d5ed74302f462c72cee192fcb20c1b30340 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -6,14 +6,31 @@
// Multiply-included message file, hence no include guard.
#include "content/common/content_export.h"
+#include "content/common/frame_param.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
nasko 2013/12/20 17:21:05 Is it really the top level or the parent? I though
awong 2013/12/20 23:53:38 Erp...I think this is a naming collosion. "frame r
+// notify it of a new texture available for compositing.
+IPC_MESSAGE_ROUTED1(FrameMsg_BuffersSwapped,
+ FrameMsg_BuffersSwapped_Params /* params */)
+
+IPC_MESSAGE_ROUTED1(FrameMsg_CompositorFrameSwapped,
nasko 2013/12/20 17:21:05 nit: It will be nice to have a comment on this IPC
awong 2013/12/20 23:53:38 Done.
+ FrameMsg_CompositorFrameSwapped_Params /* params */)
+
+// -----------------------------------------------------------------------------
+// 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.
@@ -85,3 +102,14 @@ IPC_SYNC_MESSAGE_CONTROL4_2(FrameHostMsg_OpenChannelToPlugin,
std::string /* mime_type */,
IPC::ChannelHandle /* channel_handle */,
content::WebPluginInfo /* info */)
+
+// 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_ROUTED1(FrameHostMsg_BuffersSwappedACK,
+ FrameHostMsg_BuffersSwappedACK_Params /* params */)
+
+// Acknowledge that we presented an ubercomp frame.
+IPC_MESSAGE_ROUTED1(FrameHostMsg_CompositorFrameSwappedACK,
+ FrameHostMsg_CompositorFrameSwappedACK_Params /* params */)
+

Powered by Google App Engine
This is Rietveld 408576698