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

Unified Diff: content/common/frame_param_macros.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_param_macros.h
diff --git a/content/common/frame_param_macros.h b/content/common/frame_param_macros.h
new file mode 100644
index 0000000000000000000000000000000000000000..d40b8c28dc8942447e8df5ffbd70fd828a9fa994
--- /dev/null
+++ b/content/common/frame_param_macros.h
@@ -0,0 +1,53 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
nasko 2013/12/20 17:21:05 nit: 2013
awong 2013/12/20 23:53:38 Done.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// Allows for sharing of IPC param structures between BrowserPlugin code and
+// RenderFrame code. All these should be folded directly back into the IPCs in
+// frame_messages.h once BrowserPlugin has been fully converted over to use
+// the RenderFrame infrastructure.
nasko 2013/12/20 17:21:05 nit: Maybe a TODO for easy searching later on?
awong 2013/12/20 23:53:38 Good idea. Also created a bug. Done.
+
+#ifndef CONTENT_COMMON_FRAME_PARAM_MACROS_H_
+#define CONTENT_COMMON_FRAME_PARAM_MACROS_H_
+
+#include "cc/output/compositor_frame_ack.h"
+#include "cc/output/compositor_frame.h"
+#include "content/public/common/common_param_traits.h"
+#include "ipc/ipc_message_macros.h"
+
+#undef IPC_MESSAGE_EXPORT
+#define IPC_MESSAGE_EXPORT CONTENT_EXPORT
+
+IPC_STRUCT_BEGIN(FrameMsg_BuffersSwapped_Params)
+ IPC_STRUCT_MEMBER(int, gpu_host_id)
+ IPC_STRUCT_MEMBER(int, gpu_route_id)
+ IPC_STRUCT_MEMBER(std::string, mailbox_name)
+ IPC_STRUCT_MEMBER(gfx::Size, size)
+IPC_STRUCT_END()
+
+IPC_STRUCT_BEGIN(FrameMsg_CompositorFrameSwapped_Params)
+ // Specifies which RenderWidget produced the CompositorFrame.
+ IPC_STRUCT_MEMBER(int, producing_host_id)
+ IPC_STRUCT_MEMBER(int, producing_route_id)
+
+ IPC_STRUCT_MEMBER(cc::CompositorFrame, frame)
+ IPC_STRUCT_MEMBER(uint32, output_surface_id)
+IPC_STRUCT_END()
+
+IPC_STRUCT_BEGIN(FrameHostMsg_BuffersSwappedACK_Params)
+ IPC_STRUCT_MEMBER(int, gpu_host_id)
+ IPC_STRUCT_MEMBER(int, gpu_route_id)
+ IPC_STRUCT_MEMBER(std::string, mailbox_name)
+ IPC_STRUCT_MEMBER(uint32, sync_point)
+IPC_STRUCT_END()
+
+IPC_STRUCT_BEGIN(FrameHostMsg_CompositorFrameSwappedACK_Params)
+ // Specifies which RenderWidget produced the CompositorFrame.
+ IPC_STRUCT_MEMBER(int, producing_host_id)
+ IPC_STRUCT_MEMBER(int, producing_route_id)
+
+ IPC_STRUCT_MEMBER(uint32, output_surface_id)
+ IPC_STRUCT_MEMBER(cc::CompositorFrameAck, ack)
+IPC_STRUCT_END()
+
+#endif // CONTENT_COMMON_FRAME_PARAM_MACROS_H_

Powered by Google App Engine
This is Rietveld 408576698