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

Unified Diff: content/browser/loader/navigation_url_loader_unittest.cc

Issue 2099243002: PlzNavigate: properly set the initiator of the navigation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 4 years 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/loader/navigation_url_loader_unittest.cc
diff --git a/content/browser/loader/navigation_url_loader_unittest.cc b/content/browser/loader/navigation_url_loader_unittest.cc
index 0db232393f6c308f1d68ace490d9a5395618b5e9..2b60f6501cf7d2c0c43432dc0528b4a7ed42fad4 100644
--- a/content/browser/loader/navigation_url_loader_unittest.cc
+++ b/content/browser/loader/navigation_url_loader_unittest.cc
@@ -107,14 +107,14 @@ class NavigationURLLoaderTest : public testing::Test {
const GURL& url,
NavigationURLLoaderDelegate* delegate) {
BeginNavigationParams begin_params(std::string(), net::LOAD_NORMAL, false,
- false, REQUEST_CONTEXT_TYPE_LOCATION);
+ false, REQUEST_CONTEXT_TYPE_LOCATION,
+ url::Origin(url));
CommonNavigationParams common_params;
common_params.url = url;
std::unique_ptr<NavigationRequestInfo> request_info(
- new NavigationRequestInfo(
- common_params, begin_params, url, url::Origin(url), true, false,
- false, -1, false, false, blink::WebPageVisibilityStateVisible));
-
+ new NavigationRequestInfo(common_params, begin_params, url, true, false,
+ false, -1, false, false,
+ blink::WebPageVisibilityStateVisible));
return NavigationURLLoader::Create(browser_context_.get(),
std::move(request_info), nullptr,
nullptr, nullptr, delegate);
« no previous file with comments | « content/browser/frame_host/navigation_request_info.cc ('k') | content/browser/loader/resource_dispatcher_host_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698