Framework to ascribe all network data use to a source.

chrome/browser/data_use_measurement/chrome_data_use_ascriber.h

Index: chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
diff --git a/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
new file mode 100644
index 0000000000000000000000000000000000000000..57adb0b05cfbd3251b567217dfe10a453a75cc99
--- /dev/null
+++ b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.h
@@ -0,0 +1,74 @@
+// 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 "components/data_use_measurement/core/data_use_ascriber.h"
+#include "net/base/layered_network_delegate.h"
+#include "url/gurl.h"
+
+namespace content {
+class NavigationHandle;
+class RenderFrameHost;
+}
+
+namespace data_use_measurement {
+// Browser implementation of |DataUseAscriber|. Maintains a list of
+// |DataUseRecorder| instances, one for each source of data, such as a page
+// load.
+//
+// A page includes all resources loaded in response to a main page navigation.
+// The scope of a page load ends either when the tab is closed or a new page
+// load is initiated by clicking on a link, entering a new URL, window location
+// change, etc
+//
+// For URLRequests initiated outside the context of a page load, such as
+// Service Workers, Chrome Services, etc, a new instance of |DataUseRecorder|
+// will be created for each |URLRequest|.
+//
+// Each page load will be associated with an instance of |DataUseRecorder|.
+// Each URLRequest initiated in the context of a page load will be mapped to
+// the |DataUseRecorder| instance for page load.
+//
+// This class lives entirely on the IO thread. It maintains a copy of frame and
+// navigation information on the IO thread.
+class ChromeDataUseAscriber : public DataUseAscriber {
+ public:
+  ChromeDataUseAscriber();
+
+  ~ChromeDataUseAscriber() override;
+
+  DataUseRecorder* GetDataUseRecorder(const net::URLRequest* request) override;
+
+  void RenderFrameCreated(int render_process_id,

[bengr, 2016/09/01 00:16:55]

All of these public methods need comments.

[Not at Google. Contact bengr, 2016/09/07 23:38:37]

Done.

+                          int render_frame_id,
+                          int parent_render_process_id,
+                          int parent_render_frame_id);
+
+  void RenderFrameDeleted(int render_process_id,
+                          int render_frame_id,
+                          int parent_render_process_id,
+                          int parent_render_frame_id);
+
+  void DidStartMainFrameNavigation(GURL gurl,
+                                   int render_process_id,
+                                   int render_frame_id,
+                                   void* navigation_handle);
+
+  void DidFinishMainFrameNavigation(GURL gurl,
+                                    int render_process_id,
+                                    int render_frame_id,
+                                    bool is_same_page_navigation,
+                                    void* navigation_handle);
+
+  void DidRedirectMainFrameNavigation(GURL gurl,
+                                      int render_process_id,
+                                      int render_frame_id,
+                                      void* navigation_handle);
+};
+
+}  // namespace data_use_measurement

[bengr, 2016/09/01 00:16:55]

DISALLOW_COPY_AND_ASSIGN?

[Not at Google. Contact bengr, 2016/09/07 23:38:37]

Done.

+
+#endif  // CHROME_BROWSER_DATA_USE_MEASUREMENT_CHROME_DATA_USE_ASCRIBER_H_