Minor refactor to get rid of a friend association

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 134 matching lines @@
   void NotifyDataUseCompleted(DataUseRecorderEntry entry);
 
   DataUseRecorderEntry CreateNewDataUseRecorder(
       net::URLRequest* request,
       DataUse::TrafficType traffic_type);
 
   bool IsRecorderInPendingNavigationMap(net::URLRequest* request);
 
   bool IsRecorderInRenderFrameMap(net::URLRequest* request);
 
+  void AscribeRecorderWithRequest(net::URLRequest* request,
+                                  DataUseRecorderEntry recorder);

[tbansal1, 2017/06/20 18:06:47]

s/recorder/entry/
for consistency with name used in other places (e.g., Line 144, 145 above).

+
   // Owner for all instances of DataUseRecorder. An instance is kept in this
   // list if any entity (render frame hosts, URLRequests, pending navigations)
   // that ascribe data use to the instance exists, and deleted when all
   // ascribing entities go away.
   DataUseRecorderList data_use_recorders_;
 
   // TODO(rajendrant): Combine the multiple hash_maps keyed by
   // RenderFrameHostID. Map from RenderFrameHost to the DataUseRecorderEntry in
   // |data_use_recorders_| that the main frame ascribes data use to.
   base::hash_map<RenderFrameHostID, DataUseRecorderEntry>
@@ skipping 20 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_