Expose flash playback status and volume control to content renderer

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index 8420a2e8617874403df00e908a8290490fd4320b..7e4f8c67e52abad51f35e68953cfeb9ccd7c1b1f 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -1156,11 +1156,13 @@ IPC_SYNC_MESSAGE_CONTROL3_1(FrameHostMsg_Are3DAPIsBlocked,
 #if defined(ENABLE_PLUGINS)
 // Notification sent from a renderer to the browser that a Pepper plugin
 // instance is created in the DOM.
-IPC_MESSAGE_ROUTED0(FrameHostMsg_PepperInstanceCreated)
+IPC_MESSAGE_ROUTED1(FrameHostMsg_PepperInstanceCreated,
+                    int32_t /* instance */)
 
 // Notification sent from a renderer to the browser that a Pepper plugin
 // instance is deleted from the DOM.
-IPC_MESSAGE_ROUTED0(FrameHostMsg_PepperInstanceDeleted)
+IPC_MESSAGE_ROUTED1(FrameHostMsg_PepperInstanceDeleted,
+                    int32_t /* instance */)
 
 // Sent to the browser when the renderer detects it is blocked on a pepper
 // plugin message for too long. This is also sent when it becomes unhung
@@ -1171,6 +1173,12 @@ IPC_MESSAGE_ROUTED3(FrameHostMsg_PepperPluginHung,
                     base::FilePath /* path */,
                     bool /* is_hung */)
 
+IPC_MESSAGE_ROUTED1(FrameHostMsg_PepperStartsPlayback,
+                    int32_t /* instance */)
+
+IPC_MESSAGE_ROUTED1(FrameHostMsg_PepperStopsPlayback,
+                    int32_t /* instance */)
+
 // Sent by the renderer process to indicate that a plugin instance has crashed.
 // Note: |plugin_pid| should not be trusted. The corresponding process has
 // probably died. Moreover, the ID may have been reused by a new process. Any
@@ -1534,6 +1542,12 @@ IPC_MESSAGE_ROUTED2(FrameHostMsg_GetNearestFindResult_Reply,
                     float /* distance */)
 #endif
 
+#if defined(ENABLE_PLUGINS)
+IPC_MESSAGE_ROUTED2(FrameMsg_SetPepperVolume,
+                    int32_t,

[dcheng, 2016/06/14 20:00:32]

Can you comment on what these fields mean? I assume int32_t is the plugin
instance, and double is the volume, but it's not so clear here.

[Zhiqiang Zhang (Slow), 2016/06/20 19:11:54]

Done. Also added comments for other messages, and put FrameMsg_ to the correct
place.

+                    double)
+#endif  // defined(ENABLE_PLUGINS)
+
 // Adding a new message? Stick to the sort order above: first platform
 // independent FrameMsg, then ifdefs for platform specific FrameMsg, then
 // platform independent FrameHostMsg, then ifdefs for platform specific