Move failed URLRequest checking to OnUrlRequestCompleted

chrome/browser/data_use_measurement/chrome_data_use_ascriber.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_CHROME_DATA_USE_ASCRIBER_H_
 #define CHROME_BROWSER_DATA_USE_MEASUREMENT_CHROME_DATA_USE_ASCRIBER_H_
 
 #include <list>
 #include <memory>
 #include <string>
@@ skipping 41 matching lines @@
  public:
   ChromeDataUseAscriber();
 
   ~ChromeDataUseAscriber() override;
 
   // DataUseAscriber implementation:
   ChromeDataUseRecorder* GetOrCreateDataUseRecorder(
       net::URLRequest* request) override;
   ChromeDataUseRecorder* GetDataUseRecorder(
       const net::URLRequest& request) override;
+  void OnUrlRequestCompleted(net::URLRequest* request, bool started) override;

[tbansal1, 2017/05/09 01:14:43]

pass const ref instead of raw ptr?

[rajendrant, 2017/05/11 20:31:39]

Done.

   void OnUrlRequestDestroyed(net::URLRequest* request) override;
   std::unique_ptr<URLRequestClassifier> CreateURLRequestClassifier()
       const override;
 
   // Called when a render frame host is created. When the render frame is a main
   // frame, |main_render_process_id| and |main_render_frame_id| should be -1.
   void RenderFrameCreated(int render_process_id,
                           int render_frame_id,
                           int main_render_process_id,
                           int main_render_frame_id);
@@ skipping 102 matching lines @@
 
   // Contains the mainframe IDs that are currently visible.
   base::hash_set<RenderFrameHostID> visible_main_render_frames_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeDataUseAscriber);
 };
 
 }  // namespace data_use_measurement
 
 #endif  // CHROME_BROWSER_DATA_USE_MEASUREMENT_CHROME_DATA_USE_ASCRIBER_H_