<webview>: First stab at implementing media permission request for guests.

content/common/browser_plugin_messages.h

 // Copyright (c) 2012 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.
 
 // Multiply-included message header, no traditional include guard.
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/process.h"
 #include "base/shared_memory.h"
+#include "base/values.h"
 #include "content/common/content_export.h"
 #include "content/common/content_param_traits.h"
 #include "content/public/common/common_param_traits.h"
 #include "ipc/ipc_channel_handle.h"
 #include "ipc/ipc_message_macros.h"
 #include "ipc/ipc_message_utils.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDragOperation.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDragStatus.h"
 #include "ui/gfx/point.h"
 #include "ui/gfx/rect.h"
@@ skipping 207 matching lines @@
 IPC_MESSAGE_ROUTED3(BrowserPluginHostMsg_PluginAtPositionResponse,
                     int /* instance_id */,
                     int /* request_id */,
                     gfx::Point /* position */)
 
 // Sets the name of the guest window to the provided |name|.
 IPC_MESSAGE_ROUTED2(BrowserPluginHostMsg_SetName,
                     int /* instance_id */,
                     std::string /* name */)
 
+// Tells the guest that its request for an API permission has been allowed or
+// denied.
+// Note that |allow| = true does not readily mean that the guest will be granted
+// permission, since a security check in the embedder might follow. For example
+// for media access permission, the guest will be granted permission only if its
+// embedder also has access.
+IPC_MESSAGE_ROUTED4(BrowserPluginHostMsg_AllowPermission,
+                    int /* instance_id */,
+                    std::string /* permission_type */,

[Cris Neckar, 2013/02/19 19:40:56]

It seems like this should be an enum rather than a string. Is there a particular
reason you are mapping these to string constants?

[lazyboy, 2013/02/19 21:38:40]

Changed to enum.

+                    int /* request_id */,
+                    bool /* allow */)
+
 // -----------------------------------------------------------------------------
 // These messages are from the guest renderer to the browser process
 
 // A embedder sends this message to the browser when it wants
 // to resize a guest plugin container so that the guest is relaid out
 // according to the new size.
 IPC_MESSAGE_ROUTED2(BrowserPluginHostMsg_ResizeGuest,
                     int /* instance_id*/,
                     BrowserPluginHostMsg_ResizeGuest_Params)
 
@@ skipping 107 matching lines @@
 // Guest renders into an FBO with textures provided by the embedder.
 // When HW accelerated buffers are swapped in the guest, the message
 // is forwarded to the embedder to notify it of a new texture
 // available for compositing.
 IPC_MESSAGE_ROUTED5(BrowserPluginMsg_BuffersSwapped,
                     int /* instance_id */,
                     gfx::Size /* size */,
                     std::string /* mailbox_name */,
                     int /* route_id */,
                     int /* gpu_host_id */)
+
+// When the guest requests permission, the browser process forwards this
+// request to the embeddder through this message.
+IPC_MESSAGE_ROUTED4(BrowserPluginMsg_RequestPermission,
+                    int /* instance_id */,
+                    std::string /* permission_type */,

[Cris Neckar, 2013/02/19 19:40:56]

ditto

[lazyboy, 2013/02/19 21:38:40]

Done.

+                    int /* request_id */,
+                    DictionaryValue /* request_info */)