Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.

content/public/browser/navigation_handle.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_
 
 #include <memory>
+#include <string>
 
 #include "content/common/content_export.h"
+#include "content/public/browser/global_request_id.h"

[RyanSturm, 2016/10/27 22:41:21]

nit: can you forward declare this instead?

[Not at Google. Contact bengr, 2016/10/28 18:43:33]

Done.

 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/common/referrer.h"
 #include "net/base/net_errors.h"
 #include "ui/base/page_transition_types.h"
 
 class GURL;
 
 namespace net {
 class HttpResponseHeaders;
 }  // namespace net
@@ skipping 153 matching lines @@
 
   // Resumes a navigation that was previously deferred by a NavigationThrottle.
   virtual void Resume() = 0;
 
   // Cancels a navigation that was previously deferred by a NavigationThrottle.
   // |result| should be equal to NavigationThrottle::CANCEL or
   // NavigationThrottle::CANCEL_AND_IGNORE.
   virtual void CancelDeferredNavigation(
       NavigationThrottle::ThrottleCheckResult result) = 0;
 
+  virtual bool HasNetworkResponseStarted() const = 0;
+
+  // This is valid after the network response has started. Returns the ID of the
+  // MAIN_FRAME URLRequest associated with this navigation.
+  virtual const GlobalRequestID& GetGlobalRequestID() const = 0;

[RyanSturm, 2016/10/27 22:41:21]

const isn't allowed in content/public for the most part:
https://www.chromium.org/developers/content-module/content-api

You should be able to leave the const ref, but remove the const method.

Same thing for HasNetworkResponseStarted

[Not at Google. Contact bengr, 2016/10/28 18:43:33]

Done. 

There is already a const method in this file above (WasStartedFromContextMenu).

clamy@: Any preference here?

+
   // Testing methods ----------------------------------------------------------
   //
   // The following methods should be used exclusively for writing unit tests.
 
   static std::unique_ptr<NavigationHandle> CreateNavigationHandleForTesting(
       const GURL& url,
       RenderFrameHost* render_frame_host,
       bool committed = false,
       net::Error error = net::OK);
 
@@ skipping 33 matching lines @@
 
   // The NavigationData that the embedder returned from
   // ResourceDispatcherHostDelegate::GetNavigationData during commit. This will
   // be a clone of the NavigationData.
   virtual NavigationData* GetNavigationData() = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NAVIGATION_HANDLE_H_