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

Unified Diff: content/browser/frame_host/navigation_handle_impl.cc

Issue 2161073002: Adds RequestContextType information to the NavigationHandle. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 5 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/frame_host/navigation_handle_impl.cc
diff --git a/content/browser/frame_host/navigation_handle_impl.cc b/content/browser/frame_host/navigation_handle_impl.cc
index 2e67b73868d0485b2802b8330c563bb600a96a95..47dc9a23962c4e42ceaede2f50cd7418f4283fe0 100644
--- a/content/browser/frame_host/navigation_handle_impl.cc
+++ b/content/browser/frame_host/navigation_handle_impl.cc
@@ -72,7 +72,8 @@ NavigationHandleImpl::NavigationHandleImpl(
frame_tree_node_(frame_tree_node),
next_index_(0),
navigation_start_(navigation_start),
- pending_nav_entry_id_(pending_nav_entry_id) {
+ pending_nav_entry_id_(pending_nav_entry_id),
+ request_context_type_(REQUEST_CONTEXT_TYPE_UNSPECIFIED) {
DCHECK(!navigation_start.is_null());
GetDelegate()->DidStartNavigation(this);
@@ -102,6 +103,11 @@ NavigatorDelegate* NavigationHandleImpl::GetDelegate() const {
return frame_tree_node_->navigator()->GetDelegate();
}
+RequestContextType NavigationHandleImpl::GetRequestContextType() const {
+ DCHECK_GE(state_, WILL_SEND_REQUEST);
+ return request_context_type_;
+}
+
const GURL& NavigationHandleImpl::GetURL() {
return url_;
}
@@ -270,6 +276,7 @@ NavigationHandleImpl::CallWillStartRequestForTesting(
WillStartRequest(method, resource_request_body, sanitized_referrer,
has_user_gesture, transition, is_external_protocol,
+ REQUEST_CONTEXT_TYPE_LOCATION,
base::Bind(&UpdateThrottleCheckResult, &result));
// Reset the callback to ensure it will not be called later.
@@ -318,6 +325,7 @@ void NavigationHandleImpl::WillStartRequest(
bool has_user_gesture,
ui::PageTransition transition,
bool is_external_protocol,
+ RequestContextType request_context_type,
const ThrottleChecksFinishedCallback& callback) {
// |method != "POST"| should imply absence of |resource_request_body|.
if (method != "POST" && resource_request_body) {
@@ -333,7 +341,7 @@ void NavigationHandleImpl::WillStartRequest(
has_user_gesture_ = has_user_gesture;
transition_ = transition;
is_external_protocol_ = is_external_protocol;
-
+ request_context_type_ = request_context_type;
state_ = WILL_SEND_REQUEST;
complete_callback_ = callback;
« no previous file with comments | « content/browser/frame_host/navigation_handle_impl.h ('k') | content/browser/frame_host/navigation_handle_impl_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698