Support needed to measure user and service traffic in Chrome.

chrome/browser/net/chrome_network_delegate.h

Index: chrome/browser/net/chrome_network_delegate.h
diff --git a/chrome/browser/net/chrome_network_delegate.h b/chrome/browser/net/chrome_network_delegate.h
index 10ff0dad29eb66d9d86f1b4209dfd2a58b83a261..f7cf41b349048cfc0723b99d1f9a07099b575da0 100644
--- a/chrome/browser/net/chrome_network_delegate.h
+++ b/chrome/browser/net/chrome_network_delegate.h
@@ -13,6 +13,7 @@
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
+#include "chrome/browser/net/data_use_measurement.h"
 #include "components/data_reduction_proxy/core/browser/data_reduction_proxy_metrics.h"
 #include "net/base/network_delegate_impl.h"
 
@@ -199,6 +200,9 @@ class ChromeNetworkDelegate : public net::NetworkDelegateImpl {
   // When true, allow access to all file:// URLs.
   static bool g_allow_file_access_;
 
+  // Measuring component to measure data use of services.
+  scoped_refptr<DataUseMeasurement> data_use_measurer_;

[sclittle, 2015/08/19 23:32:39]

nit: please either rename the class to DataUseMeasurer or change the name here
to data_use_measurement_.

[sclittle, 2015/08/19 23:32:39]

Also, don't refcount this unless there's a really good reason to refcount it.

[amohammadkhan, 2015/08/21 23:06:31]

Basically I used all the refcounts because of the callback in this class,
because I was not sure is it okay to not use refcounts and handle the lifetime
manually. But after your heads up and more reading about
AppliactionStatusListener, it seems I was over cautious and it was not needed.
So I changed it to normal class.

[amohammadkhan, 2015/08/21 23:06:31]

Done.

+
   bool experimental_web_platform_features_enabled_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeNetworkDelegate);