Expose GlobalRequestID in NavigationHandle and ResourceRequestInfo.

chrome/browser/data_use_measurement/data_use_web_contents_observer.h

 // Copyright 2016 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 CHROME_BROWSER_DATA_USE_MEASUREMENT_DATA_USE_WEB_CONTENTS_OBSERVER_H_
 #define CHROME_BROWSER_DATA_USE_MEASUREMENT_DATA_USE_WEB_CONTENTS_OBSERVER_H_
 
 #include "base/macros.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
@@ skipping 14 matching lines @@
 // observe all WebContents.
 class DataUseWebContentsObserver
     : public content::WebContentsObserver,
       public content::WebContentsUserData<DataUseWebContentsObserver> {
  public:
   // Creates a DataUseWebContentsObserver for the given WebContents.
   static void CreateForWebContents(content::WebContents* web_contents);
 
   ~DataUseWebContentsObserver() override;
 
   // WebContentsObserver:

[RyanSturm, 2016/11/07 19:33:19]

nit: s/WebContentsObserver/WebContentsObserver implmentation:/

The comments on the methods shouldn't be there as the WCO comments will be more
useful and accurate and these don't add any class specific information.

Remove the method comments and the vertical whitespace between methods.
Similar to
https://cs.chromium.org/chromium/src/chrome/browser/page_load_metrics/metrics...

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

Done.

   // Called when a render frame host is created.
   void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override;
 
   // Called when a render frame host is deleted.
   void RenderFrameDeleted(content::RenderFrameHost* render_frame_host) override;
 
   // Called when a navigation is started.
   void DidStartNavigation(
       content::NavigationHandle* navigation_handle) override;
 
-  // Called when a navigation is completed.
-  void DidFinishNavigation(
+  // Called when a navigation is ready to be committed in a renderer.
+  void ReadyToCommitNavigation(
       content::NavigationHandle* navigation_handle) override;
 
   // Called when a navigation is redirected.
   void DidRedirectNavigation(
       content::NavigationHandle* navigation_handle) override;
 
  private:
   friend class content::WebContentsUserData<DataUseWebContentsObserver>;
 
   DataUseWebContentsObserver(content::WebContents* web_contents,
                              ChromeDataUseAscriberService* service);
   ChromeDataUseAscriberService* const service_;
 
   DISALLOW_COPY_AND_ASSIGN(DataUseWebContentsObserver);
 };
 
 }  // namespace data_use_measurement
 
 #endif  // CHROME_BROWSER_DATA_USE_MEASUREMENT_DATA_USE_WEB_CONTENTS_OBSERVER_H_