summaryrefslogtreecommitdiffstats
path: root/android_webview
diff options
context:
space:
mode:
authorqinmin@chromium.org <qinmin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-05-29 03:09:02 +0000
committerqinmin@chromium.org <qinmin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-05-29 03:09:02 +0000
commit5e170a50333a4ed280c1a5e127e03043f7fb27be (patch)
tree98e2543f358713a8088091f6a028b8247cc62bab /android_webview
parent880a24b2d76d12cbe57757b61f33aff45a48339b (diff)
downloadchromium_src-5e170a50333a4ed280c1a5e127e03043f7fb27be.zip
chromium_src-5e170a50333a4ed280c1a5e127e03043f7fb27be.tar.gz
chromium_src-5e170a50333a4ed280c1a5e127e03043f7fb27be.tar.bz2
use origin instead of session_id in detecting duplicate EME permission request
The same origin can create a lot of sessions, using session id will generate a lot of duplicate permission requests from a single origin. This change fixes the behavior by using the origin url. It also removed the group_id in PermissionRequestId as it is no longer used. BUG=373641 Review URL: https://codereview.chromium.org/289023003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273457 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'android_webview')
-rw-r--r--android_webview/browser/aw_browser_context.cc8
-rw-r--r--android_webview/browser/aw_browser_context.h10
2 files changed, 9 insertions, 9 deletions
diff --git a/android_webview/browser/aw_browser_context.cc b/android_webview/browser/aw_browser_context.cc
index 4efdd6c..c6c020c 100644
--- a/android_webview/browser/aw_browser_context.cc
+++ b/android_webview/browser/aw_browser_context.cc
@@ -237,16 +237,16 @@ void AwBrowserContext::CancelMidiSysExPermissionRequest(
void AwBrowserContext::RequestProtectedMediaIdentifierPermission(
int render_process_id,
int render_view_id,
- int bridge_id,
- int group_id,
- const GURL& requesting_frame,
+ const GURL& origin,
const ProtectedMediaIdentifierPermissionCallback& callback) {
NOTIMPLEMENTED();
callback.Run(false);
}
void AwBrowserContext::CancelProtectedMediaIdentifierPermissionRequests(
- int group_id) {
+ int render_process_id,
+ int render_view_id,
+ const GURL& origin) {
NOTIMPLEMENTED();
}
diff --git a/android_webview/browser/aw_browser_context.h b/android_webview/browser/aw_browser_context.h
index de1c545..8b0c068 100644
--- a/android_webview/browser/aw_browser_context.h
+++ b/android_webview/browser/aw_browser_context.h
@@ -114,12 +114,12 @@ class AwBrowserContext : public content::BrowserContext,
virtual void RequestProtectedMediaIdentifierPermission(
int render_process_id,
int render_view_id,
- int bridge_id,
- int group_id,
- const GURL& requesting_frame,
+ const GURL& origin,
const ProtectedMediaIdentifierPermissionCallback& callback) OVERRIDE;
- virtual void CancelProtectedMediaIdentifierPermissionRequests(int group_id)
- OVERRIDE;
+ virtual void CancelProtectedMediaIdentifierPermissionRequests(
+ int render_process_id,
+ int render_view_id,
+ const GURL& origin) OVERRIDE;
virtual content::ResourceContext* GetResourceContext() OVERRIDE;
virtual content::DownloadManagerDelegate*
GetDownloadManagerDelegate() OVERRIDE;