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

Unified Diff: content/browser/permissions/permission_service_context.cc

Issue 1726323002: Have Permission{Manager,Service} use Origin. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clarify and test Origin.empty_. Created 4 years, 10 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: content/browser/permissions/permission_service_context.cc
diff --git a/content/browser/permissions/permission_service_context.cc b/content/browser/permissions/permission_service_context.cc
index 91bd4d04ac187b0a0ccda348726b0de06bba0081..5af21b7673c299bbecce0f04e5efae91202d6de1 100644
--- a/content/browser/permissions/permission_service_context.cc
+++ b/content/browser/permissions/permission_service_context.cc
@@ -82,9 +82,9 @@ BrowserContext* PermissionServiceContext::GetBrowserContext() const {
return web_contents()->GetBrowserContext();
}
-GURL PermissionServiceContext::GetEmbeddingOrigin() const {
- return web_contents() ? web_contents()->GetLastCommittedURL().GetOrigin()
- : GURL();
+url::Origin PermissionServiceContext::GetEmbeddingOrigin() const {
+ return web_contents() ? url::Origin(web_contents()->GetLastCommittedURL())
+ : url::Origin();
}
RenderFrameHost* PermissionServiceContext::render_frame_host() const {

Powered by Google App Engine
This is Rietveld 408576698