Adding parse UMA and pingback metrics

components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.cc

Index: components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.cc
diff --git a/components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.cc b/components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.cc
index fdccecb0a1abded0581fa42d962428d5dc202a1c..8964edd165123dadf59c65ca98fadc19158348ae 100644
--- a/components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.cc
+++ b/components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.cc
@@ -4,26 +4,35 @@
 
 #include "components/data_reduction_proxy/core/common/data_reduction_proxy_page_load_timing.h"
 
 namespace data_reduction_proxy {
 
 DataReductionProxyPageLoadTiming::DataReductionProxyPageLoadTiming(
     const base::Time& navigation_start,
     const base::Optional<base::TimeDelta>& response_start,
     const base::Optional<base::TimeDelta>& load_event_start,
     const base::Optional<base::TimeDelta>& first_image_paint,
-    const base::Optional<base::TimeDelta>& first_contentful_paint)
+    const base::Optional<base::TimeDelta>& first_contentful_paint,
+    const base::Optional<base::TimeDelta>&
+        parse_blocked_on_script_load_duration,
+    const base::Optional<base::TimeDelta>& parse_stop)
     : navigation_start(navigation_start),
       response_start(response_start),
       load_event_start(load_event_start),
       first_image_paint(first_image_paint),
-      first_contentful_paint(first_contentful_paint) {}
+      first_contentful_paint(first_contentful_paint),
+      parse_blocked_on_script_load_duration(
+          parse_blocked_on_script_load_duration),
+      parse_stop(parse_stop) {}
 
 DataReductionProxyPageLoadTiming::DataReductionProxyPageLoadTiming(

[tbansal1, 2016/09/06 19:44:43]

nit: Why not use default copy constructor.

[tbansal1, 2016/09/06 19:44:43]

nit: May be add a default assignment operator.

[RyanSturm, 2016/09/06 20:02:57]

base::Time only doesn't have a a copy assignment operator, so deafult won't
work.

[RyanSturm, 2016/09/06 20:02:57]

Done.

     const DataReductionProxyPageLoadTiming& other)
     : navigation_start(other.navigation_start),
       response_start(other.response_start),
       load_event_start(other.load_event_start),
       first_image_paint(other.first_image_paint),
-      first_contentful_paint(other.first_contentful_paint) {}
+      first_contentful_paint(other.first_contentful_paint),
+      parse_blocked_on_script_load_duration(
+          other.parse_blocked_on_script_load_duration),
+      parse_stop(other.parse_stop) {}
 
 }  // namespace data_reduction_proxy