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..5fb1eb2d6ed8ed5b4b3c566cee7fc8df6e2beeb6 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(carlosk): set the RequestContextType in a more correct way, if
+    // actually needed. See: determineRequestContextFromNavigationType in
+    // FrameLoader.cpp.

[Mike West, 2016/07/20 09:52:46]

Why not do this now? If we can set this field correctly, that seems better than
hard-coding it into something arbitrary. CSP does look at differences in
navigation type when enforcing `form-action`, for instance; let's give it
correct values when doing so.

[carlosk, 2016/07/20 10:12:02]

To save time (as I disappear in less than a week) and because this code is
PlzNavigate only and so not active IRL yet. The other TODOs are there for the
same reasons.

[Mike West, 2016/07/20 11:56:36]

Perhaps `TODO(clamy)`, then? I worry that things like this will get dropped.

[carlosk, 2016/07/20 16:26:27]

Done. But it's unlikely that will be forgotten as there are other cases of
TODO(carlosk) that will have to be addressed for PlzNavigate to be ready.

     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,
         base::Bind(&NavigationRequest::OnStartChecksComplete,
                    base::Unretained(this)));
     return;