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

Unified Diff: content/common/navigation_params.cc

Issue 2584513003: PlzNavigate: identify same-page browser-initiated navigation. (Closed)
Patch Set: Fix 2 WebContentsImplTest by assigning a document_sequence_number. Created 3 years, 11 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/common/navigation_params.cc
diff --git a/content/common/navigation_params.cc b/content/common/navigation_params.cc
index 51220c03b9bc75dbb97da4af9bb69047ec2d23d0..075717085fb3f633aa5ca44c1e4938f0b2fce3c7 100644
--- a/content/common/navigation_params.cc
+++ b/content/common/navigation_params.cc
@@ -21,8 +21,6 @@ bool ShouldMakeNetworkRequestForURL(const GURL& url) {
// Javascript URLs, about:blank, srcdoc should not send a request
// to the network stack.
- // TODO(clamy): same document navigations should not send requests to the
- // network stack. Neither should pushState/popState.
return url != url::kAboutBlankURL && !url.SchemeIs(url::kJavaScriptScheme) &&
!url.is_empty() && !url.SchemeIs(url::kContentIDScheme) &&
url != content::kAboutSrcDocURL;
@@ -129,6 +127,7 @@ RequestNavigationParams::RequestNavigationParams()
: is_overriding_user_agent(false),
can_load_local_resources(false),
nav_entry_id(0),
+ is_same_document_fragment_change(false),
is_same_document_history_load(false),
is_history_navigation_in_new_child(false),
has_committed_real_load(false),
@@ -150,6 +149,7 @@ RequestNavigationParams::RequestNavigationParams(
bool can_load_local_resources,
const PageState& page_state,
int nav_entry_id,
+ bool is_same_document_fragment_change,
bool is_same_document_history_load,
bool is_history_navigation_in_new_child,
std::map<std::string, bool> subframe_unique_names,
@@ -166,6 +166,7 @@ RequestNavigationParams::RequestNavigationParams(
can_load_local_resources(can_load_local_resources),
page_state(page_state),
nav_entry_id(nav_entry_id),
+ is_same_document_fragment_change(is_same_document_fragment_change),
is_same_document_history_load(is_same_document_history_load),
is_history_navigation_in_new_child(is_history_navigation_in_new_child),
subframe_unique_names(subframe_unique_names),

Powered by Google App Engine
This is Rietveld 408576698