Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.

content/public/browser/navigation_handle.h

Index: content/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index eae9cd46bb8b6910c79c93eb67b4ac1215d7c657..d731dfa404a9a466d7ed09f5e89dbc92a4f16342 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -6,6 +6,7 @@
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 
 #include <memory>
+#include <string>
 
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_throttle.h"
@@ -20,6 +21,7 @@ class HttpResponseHeaders;
 }  // namespace net
 
 namespace content {
+struct GlobalRequestID;
 class NavigationData;
 class NavigationThrottle;
 class RenderFrameHost;
@@ -125,6 +127,9 @@ class CONTENT_EXPORT NavigationHandle {
   // handlers.
   virtual bool IsExternalProtocol() = 0;
 
+  // Whether the network response has started.
+  virtual bool HasNetworkResponseStarted() = 0;
+
   // Navigation control flow --------------------------------------------------
 
   // The net error code if an error happened prior to commit. Otherwise it will
@@ -176,6 +181,10 @@ class CONTENT_EXPORT NavigationHandle {
   virtual void CancelDeferredNavigation(
       NavigationThrottle::ThrottleCheckResult result) = 0;
 
+  // This is valid after the network response has started. Returns the ID of the
+  // MAIN_FRAME URLRequest associated with this navigation.

[clamy, 2016/11/04 14:44:17]

Actually, considering the case of transfer navigations, this only valid in
NavigationThrottle::WillProcessResponse and
WebContentsObserver::AboutToCommitNavigation, since we don't update the ID to
account for the new transferred request. This makes HasNetworkResponseStarted
unnecessary, since it can only be used in the 2 aforementioned methods.

[Not at Google. Contact bengr, 2016/11/07 22:17:18]

Done.

I'm guessing you mean ReadyToCommitNavigation()? Changed to use
ReadyToCommitNavigation() throughout.

+  virtual const GlobalRequestID& GetGlobalRequestID() = 0;
+
   // Testing methods ----------------------------------------------------------
   //
   // The following methods should be used exclusively for writing unit tests.