diff options
| author | lalitm <lalitm@google.com> | 2015-09-10 03:20:02 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-10 10:20:51 +0000 |
| commit | c886a256e642e88614ecb38cec4aee1b8f14ab67 (patch) | |
| tree | 6650044547629b86a4bcc3f794940e747dd380c8 /chrome/browser/geolocation | |
| parent | ecf113f67c8ff43e8eddeb2585c98bbc31ed79e2 (diff) | |
| download | chromium_src-c886a256e642e88614ecb38cec4aee1b8f14ab67.zip chromium_src-c886a256e642e88614ecb38cec4aee1b8f14ab67.tar.gz chromium_src-c886a256e642e88614ecb38cec4aee1b8f14ab67.tar.bz2 | |
permissions: Remove origin from permission_request_id
It is not needed and moreover, it makes life difficult for
introduction of multiple permissions. The comment about
uniqueness being based on origin is no longer relevant so
can be removed.
BUG=516626
Review URL: https://codereview.chromium.org/1325923003
Cr-Commit-Position: refs/heads/master@{#348131}
Diffstat (limited to 'chrome/browser/geolocation')
| -rw-r--r-- | chrome/browser/geolocation/geolocation_permission_context_unittest.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/chrome/browser/geolocation/geolocation_permission_context_unittest.cc b/chrome/browser/geolocation/geolocation_permission_context_unittest.cc index 95cf26d..790168a 100644 --- a/chrome/browser/geolocation/geolocation_permission_context_unittest.cc +++ b/chrome/browser/geolocation/geolocation_permission_context_unittest.cc @@ -167,8 +167,7 @@ PermissionRequestID GeolocationPermissionContextTests::RequestID( return PermissionRequestID( web_contents()->GetRenderProcessHost()->GetID(), web_contents()->GetMainFrame()->GetRoutingID(), - request_id, - GURL()); + request_id); } PermissionRequestID GeolocationPermissionContextTests::RequestIDForTab( @@ -177,8 +176,7 @@ PermissionRequestID GeolocationPermissionContextTests::RequestIDForTab( return PermissionRequestID( extra_tabs_[tab]->GetRenderProcessHost()->GetID(), extra_tabs_[tab]->GetMainFrame()->GetRoutingID(), - request_id, - GURL()); + request_id); } void GeolocationPermissionContextTests::RequestGeolocationPermission( |