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

Unified Diff: chrome/browser/background_sync/background_sync_permission_context_unittest.cc

Issue 1726323002: Have Permission{Manager,Service} use Origin. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 9 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/background_sync/background_sync_permission_context_unittest.cc
diff --git a/chrome/browser/background_sync/background_sync_permission_context_unittest.cc b/chrome/browser/background_sync/background_sync_permission_context_unittest.cc
index aa6e24cb5095ffb76d6039b735d6975d67046dd4..3a9b44c90f242760160461886573ef5cf6c3245d 100644
--- a/chrome/browser/background_sync/background_sync_permission_context_unittest.cc
+++ b/chrome/browser/background_sync/background_sync_permission_context_unittest.cc
@@ -21,6 +21,7 @@
#include "content/public/test/mock_render_process_host.h"
#include "content/public/test/web_contents_tester.h"
#include "testing/gtest/include/gtest/gtest.h"
+#include "url/origin.h"
class BackgroundSyncPermissionContextTest
: public ChromeRenderViewHostTestHarness {
@@ -40,7 +41,7 @@ class BackgroundSyncPermissionContextTest
web_contents()->GetRenderProcessHost()->GetID(),
web_contents()->GetMainFrame()->GetRoutingID(), -1 /* request_id */);
permission_context->RequestPermission(
- web_contents(), id, url,
+ web_contents(), id, url::Origin(url),
base::Bind(
&BackgroundSyncPermissionContextTest::TrackPermissionDecision,
base::Unretained(this), run_loop.QuitClosure()));

Powered by Google App Engine
This is Rietveld 408576698