Adds support for media permission request handling on <appview> tag.

extensions/browser/guest_view/app_view/app_view_guest.cc

Index: extensions/browser/guest_view/app_view/app_view_guest.cc
diff --git a/extensions/browser/guest_view/app_view/app_view_guest.cc b/extensions/browser/guest_view/app_view/app_view_guest.cc
index 2309380e6ff77e0f3ff1f0a9a7c0924929d26875..ef4908ea78b12c5e70750f967236700b63aa296a 100644
--- a/extensions/browser/guest_view/app_view/app_view_guest.cc
+++ b/extensions/browser/guest_view/app_view/app_view_guest.cc
@@ -9,6 +9,7 @@
 #include "content/public/common/renderer_preferences.h"
 #include "extensions/browser/api/app_runtime/app_runtime_api.h"
 #include "extensions/browser/api/extensions_api_client.h"
+#include "extensions/browser/app_window/app_delegate.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_host.h"
 #include "extensions/browser/extension_registry.h"
@@ -104,6 +105,8 @@ AppViewGuest::AppViewGuest(content::BrowserContext* browser_context,
       app_view_guest_delegate_(
           ExtensionsAPIClient::Get()->CreateAppViewGuestDelegate()),
       weak_ptr_factory_(this) {
+  if (app_view_guest_delegate_)
+    app_delegate_.reset(app_view_guest_delegate_->CreateAppDelegate());
 }
 
 AppViewGuest::~AppViewGuest() {
@@ -133,6 +136,42 @@ bool AppViewGuest::HandleContextMenu(const content::ContextMenuParams& params) {
   return false;
 }
 
+void AppViewGuest::RequestMediaAccessPermission(
+    content::WebContents* web_contents,
+    const content::MediaStreamRequest& request,
+    const content::MediaResponseCallback& callback) {
+  if (!app_delegate_) {
+    WebContentsDelegate::RequestMediaAccessPermission(web_contents, request,
+                                                      callback);

[Fady Samuel, 2014/12/19 11:21:51]

early exit after so you don't need else. Slight improvement to readability.

[lfg, 2014/12/19 16:29:08]

Done.

+  } else {
+    const ExtensionSet& enabled_extensions =
+        ExtensionRegistry::Get(browser_context())->enabled_extensions();
+    const Extension* guest_extension =
+        enabled_extensions.GetByID(guest_extension_id_);
+
+    app_delegate_->RequestMediaAccessPermission(web_contents, request, callback,
+                                                guest_extension);
+  }
+}
+
+bool AppViewGuest::CheckMediaAccessPermission(
+    content::WebContents* web_contents,
+    const GURL& security_origin,
+    content::MediaStreamType type) {
+  if (!app_delegate_) {
+    return WebContentsDelegate::CheckMediaAccessPermission(
+        web_contents, security_origin, type);

[Fady Samuel, 2014/12/19 11:21:51]

early exit then remove else.

[lfg, 2014/12/19 16:29:08]

Done.

+  } else {
+    const ExtensionSet& enabled_extensions =
+        ExtensionRegistry::Get(browser_context())->enabled_extensions();
+    const Extension* guest_extension =
+        enabled_extensions.GetByID(guest_extension_id_);
+
+    return app_delegate_->CheckMediaAccessPermission(
+        web_contents, security_origin, type, guest_extension);
+  }
+}
+
 const char* AppViewGuest::GetAPINamespace() const {
   return appview::kEmbedderAPINamespace;
 }
@@ -258,4 +297,8 @@ void AppViewGuest::LaunchAppAndFireEvent(
       browser_context(), embed_request.Pass(), extension_host->extension());
 }
 
+void AppViewGuest::SetAppDelegateForTest(AppDelegate* delegate) {
+  app_delegate_.reset(delegate);
+}
+
 }  // namespace extensions