Adding UMA for data reduction proxy page load size/savings

chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_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_PAGE_LOAD_METRICS_OBSERVERS_DATA_REDUCTION_PROXY_METRICS_OBSERVER_H_
 #define CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_DATA_REDUCTION_PROXY_METRICS_OBSERVER_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 72 matching lines @@
   void OnFirstContentfulPaint(
       const page_load_metrics::PageLoadTiming& timing,
       const page_load_metrics::PageLoadExtraInfo& info) override;
   void OnFirstMeaningfulPaint(
       const page_load_metrics::PageLoadTiming& timing,
       const page_load_metrics::PageLoadExtraInfo& info) override;
   void OnParseStart(const page_load_metrics::PageLoadTiming& timing,
                     const page_load_metrics::PageLoadExtraInfo& info) override;
   void OnParseStop(const page_load_metrics::PageLoadTiming& timing,
                    const page_load_metrics::PageLoadExtraInfo& info) override;
+  void OnLoadedResource(
+      const page_load_metrics::ExtraRequestInfo& extra_request_info) override;
 
  private:
   // Sends the page load information to the pingback client.
   void SendPingback(const page_load_metrics::PageLoadTiming& timing,
                     const page_load_metrics::PageLoadExtraInfo& info);
 
+  // Record UMA of page size when the observer is about to be deleted.
+  void RecordPageSizeUMA();

[tbansal1, 2017/01/18 22:35:20]

s/Record/Records/

[RyanSturm, 2017/01/18 22:57:49]

Done.

+
   // Gets the default DataReductionProxyPingbackClient. Overridden in testing.
   virtual DataReductionProxyPingbackClient* GetPingbackClient() const;
 
   // Data related to this navigation.
   std::unique_ptr<DataReductionProxyData> data_;
 
   // The browser context this navigation is operating in.
   content::BrowserContext* browser_context_;
 
+  // The number of requests that used data reduction proxy.
+  int num_data_reduction_proxy_requests_;
+
+  // The number of request that used the network.

[tbansal1, 2017/01/18 22:35:20]

How is this determined? What happens if the cached resource is validated?

[RyanSturm, 2017/01/18 22:57:49]

Done.

+  int num_network_requests_;
+
+  // The total content network bytes that the user would have downloaded if they
+  // were not using data reduction proxy.
+  int64_t original_network_bytes_;
+
+  // The total network bytes loaded through data reduction proxy

[tbansal1, 2017/01/18 22:35:20]

missing period at end.

+  int64_t network_bytes_proxied_;
+
+  // The total network bytes used.
+  int64_t network_bytes_;
+
   DISALLOW_COPY_AND_ASSIGN(DataReductionProxyMetricsObserver);
 };
 
 }  // namespace data_reduction_proxy
 
 #endif  // CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_DATA_REDUCTION_PROXY_METRICS_OBSERVER_H_