Fix page zoom to be frame-centric for out-of-process frames.

content/common/page_messages.h

Index: content/common/page_messages.h
diff --git a/content/common/page_messages.h b/content/common/page_messages.h
index 284c134508b5d9227303ec5f96ec52235e628176..668b3f7ccd23d100f39a9d6053ad8f1f7f8dbbc0 100644
--- a/content/common/page_messages.h
+++ b/content/common/page_messages.h
@@ -2,8 +2,13 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#ifndef CONTENT_COMMON_PAGE_MESSAGES_H_
+#define CONTENT_COMMON_PAGE_MESSAGES_H_
+
 #include "ipc/ipc_message_macros.h"
 

[alexmos, 2016/04/07 23:48:08]

nit: why this blank line?

[wjmaclean, 2016/04/08 20:13:29]

Done.

+#include "ui/gfx/geometry/rect.h"
+
 // IPC messages for page-level actions.
 // Multiply-included message file, hence no include guard.
 
@@ -12,14 +17,28 @@
 
 #define IPC_MESSAGE_START PageMsgStart
 
+// Create a 'type' for our enum, since IPC doesn't support enum classes (yet).

[alexmos, 2016/04/07 23:48:08]

Hmm, we use things like WebSandboxFlags and WebTreeScopeType in IPCs just fine,
and those are enum class.  Also seems like you'll need IPC_ENUM_TRAITS_MAX_VALUE
for this (see examples in frame_messages.h)

[wjmaclean, 2016/04/08 20:13:29]

Ahh, ok. I was getting messages about the pickler not being able to generate the
read/write code for these, and a colleague indicated we don't support enum
classes for IPC. If we do, then that's fine too. :-)

Done.

+using PageMsg_SetZoomLevel_Command = int;
+enum {
+  ZOOM_SET_TEMPORARY,
+  ZOOM_CLEAR_TEMPORARY,
+  ZOOM_USE_CURRENT_TEMPORARY_MODE
+};
+
 // Messages sent from the browser to the renderer.
 
 IPC_MESSAGE_ROUTED1(PageMsg_UpdateWindowScreenRect,
                     gfx::Rect /* window_screen_rect */)
 
+IPC_MESSAGE_ROUTED2(PageMsg_SetZoomLevel,
+                    PageMsg_SetZoomLevel_Command /* command */,
+                    double /* zoom_level */)
+
 // -----------------------------------------------------------------------------
 // Messages sent from the renderer to the browser.
 
 // Adding a new message? Stick to the sort order above: first platform
 // independent PageMsg, then ifdefs for platform specific PageMsg, then platform
 // independent PageHostMsg, then ifdefs for platform specific PageHostMsg.
+
+#endif  // CONTENT_COMMON_PAGE_MESSAGES_H_