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/navigation_throttle.h"
 #include "content/public/common/referrer.h"
 #include "net/base/net_errors.h"
 #include "net/http/http_response_info.h"
 #include "ui/base/page_transition_types.h"
 
 class GURL;
 
 namespace net {
 class HttpResponseHeaders;
 }  // namespace net
 
 namespace content {
+struct GlobalRequestID;
 class NavigationData;
 class NavigationThrottle;
 class RenderFrameHost;
 class SiteInstance;
 class WebContents;
 
 // A NavigationHandle tracks information related to a single navigation.
 // NavigationHandles are provided to several WebContentsObserver methods to
 // allow observers to track specific navigations. Observers should clear any
 // references to a NavigationHandle at the time of
@@ skipping 142 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;
 
+  // Returns the ID of the MAIN_FRAME URLRequest associated with this

[clamy, 2016/11/09 16:06:53]

This is valid for subframes as well. I think we can remove "MAIN_FRAME" from the
comment.

[Not at Google. Contact bengr, 2016/11/09 22:30:01]

Done.

+  // navigation. Can only be called from NavigationThrottle::WillProcessResponse
+  // and WebContentsObserver::ReadyToCommitNavigation.

[clamy, 2016/11/09 16:06:53]

Can you add something like:
"Note: in the case of transfer navigations, this is the ID of the first request
made. The transferred request's ID will not be tracked by the NavigationHandle."

[Not at Google. Contact bengr, 2016/11/09 22:30:01]

Done.

+  virtual const GlobalRequestID& GetGlobalRequestID() = 0;
+
   // 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_