Adds RequestContextType information to the NavigationHandle.

content/browser/frame_host/navigation_handle_impl.h

Index: content/browser/frame_host/navigation_handle_impl.h
diff --git a/content/browser/frame_host/navigation_handle_impl.h b/content/browser/frame_host/navigation_handle_impl.h
index ac7680ab0ba80d7ac6dc5ac172a9e09363bb604b..bdc6c056a97575d378db54efc0a9e60a7fd552d7 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -18,6 +18,7 @@
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_data.h"
 #include "content/public/browser/navigation_throttle.h"
+#include "content/public/common/request_context_type.h"
 #include "url/gurl.h"
 
 struct FrameHostMsg_DidCommitProvisionalLoad_Params;
@@ -26,7 +27,6 @@ namespace content {
 
 class NavigatorDelegate;
 class ResourceRequestBodyImpl;
-struct NavigationRequestInfo;
 
 // This class keeps track of a single navigation. It is created upon receipt of
 // a DidStartProvisionalLoad IPC in a RenderFrameHost. The RenderFrameHost owns
@@ -118,6 +118,8 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
 
   NavigatorDelegate* GetDelegate() const;
 
+  RequestContextType GetRequestContextType() const;

[nasko, 2016/07/22 23:36:25]

This is just an accessor, so it should be hacker_case.

[carlosk, 2016/07/25 13:06:10]

It used to be the case before but as I added the DCHECK clamy@ asked for I moved
the implementation to the .CC file and renamed it to the CamelCase.

It is still a simple function but can a hacker_case function implementation live
on the .CC file?

[nasko, 2016/07/25 13:57:58]

DCHECK isn't really adding complexity and in release build is noop. Example:
https://cs.chromium.org/chromium/src/content/renderer/render_widget.h?sq=pack....

content$ git grep DCHECK *.h | grep -v "//" | wc -l
     111

+
   // Returns the response headers for the request or nullptr if there are none.
   // This should only be accessed after a redirect was encountered or after the
   // navigation is ready to commit. The headers returned should not be modified,
@@ -176,6 +178,7 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
       bool has_user_gesture,
       ui::PageTransition transition,
       bool is_external_protocol,
+      RequestContextType request_context_type,
       const ThrottleChecksFinishedCallback& callback);
 
   // Called when the URLRequest will be redirected in the network stack.
@@ -309,6 +312,9 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   // The unique id of the corresponding NavigationEntry.
   int pending_nav_entry_id_;
 
+  // The fetch request context type.
+  RequestContextType request_context_type_;
+
   // This callback will be run when all throttle checks have been performed.
   ThrottleChecksFinishedCallback complete_callback_;