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

Unified Diff: chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.cc

Issue 562263002: Check media permissions through RenderFrameHostDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@render_frame_get_sources
Patch Set: Rebase Created 6 years, 3 months 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: chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.cc
diff --git a/chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.cc b/chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.cc
index 3279b883352a0d4b662adb35c548fe158a5d6385..07d924c52cf28c7368b18b2f4a9568ea0af018c2 100644
--- a/chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.cc
+++ b/chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.cc
@@ -138,7 +138,18 @@ void ChromeWebViewPermissionHelperDelegate::RequestMediaAccessPermission(
false /* allowed_by_default */);
}
- void ChromeWebViewPermissionHelperDelegate::OnMediaPermissionResponse(
+bool ChromeWebViewPermissionHelperDelegate::CheckMediaAccessPermission(
+ content::WebContents* source,
+ const GURL& security_origin,
+ content::MediaStreamType type) {
+ return web_view_guest()
+ ->embedder_web_contents()
+ ->GetDelegate()
+ ->CheckMediaAccessPermission(
+ web_view_guest()->embedder_web_contents(), security_origin, type);
+}
+
+void ChromeWebViewPermissionHelperDelegate::OnMediaPermissionResponse(
const content::MediaStreamRequest& request,
const content::MediaResponseCallback& callback,
bool allow,

Powered by Google App Engine
This is Rietveld 408576698