Adds RequestContextType information to the NavigationHandle.

content/browser/frame_host/navigation_request.cc

Index: content/browser/frame_host/navigation_request.cc
diff --git a/content/browser/frame_host/navigation_request.cc b/content/browser/frame_host/navigation_request.cc
index b994fa9dabd47d5e105aaf015cc354a711d86b5c..40021fd2685c8f80e3ee4c8732f9aa8f973d0ed9 100644
--- a/content/browser/frame_host/navigation_request.cc
+++ b/content/browser/frame_host/navigation_request.cc
@@ -24,6 +24,7 @@
 #include "content/public/browser/storage_partition.h"
 #include "content/public/browser/stream_handle.h"
 #include "content/public/common/content_client.h"
+#include "content/public/common/request_context_type.h"
 #include "content/public/common/resource_response.h"
 #include "net/base/load_flags.h"
 #include "net/http/http_request_headers.h"
@@ -210,11 +211,15 @@ void NavigationRequest::BeginNavigation() {
     // TODO(clamy): pass the real value for |is_external_protocol| if needed.
     // TODO(clamy): pass the method to the NavigationHandle instead of a
     // boolean.
+    // TODO(clamy): set the RequestContextType in a more correct way, if
+    // actually needed. See: determineRequestContextFromNavigationType in
+    // FrameLoader.cpp.
     navigation_handle_->WillStartRequest(
         common_params_.method, common_params_.post_data,
         Referrer::SanitizeForRequest(common_params_.url,
                                      common_params_.referrer),
         begin_params_.has_user_gesture, common_params_.transition, false,
+        REQUEST_CONTEXT_TYPE_LOCATION,

[clamy, 2016/07/21 11:30:21]

The RequestContextType is passed to the browser in BeginNavigationParams. Please
use it and remove the TODO.

[carlosk, 2016/07/22 12:51:26]

Done. I have no idea how I missed that...

         base::Bind(&NavigationRequest::OnStartChecksComplete,
                    base::Unretained(this)));
     return;