Allow TimeTicks for serialized page load operations to be equal.

components/page_load_metrics/browser/page_load_metrics_observer.h

Index: components/page_load_metrics/browser/page_load_metrics_observer.h
diff --git a/components/page_load_metrics/browser/page_load_metrics_observer.h b/components/page_load_metrics/browser/page_load_metrics_observer.h
index c42f36d93ee467eab1375a7bad8abbb8b90a4441..b6dd353e467e205976e718817a1a31983000e49d 100644
--- a/components/page_load_metrics/browser/page_load_metrics_observer.h
+++ b/components/page_load_metrics/browser/page_load_metrics_observer.h
@@ -55,6 +55,7 @@ struct PageLoadExtraInfo {
                     base::TimeDelta first_foreground_time,
                     bool started_in_foreground,
                     const GURL& committed_url,
+                    bool committed,
                     base::TimeDelta time_to_commit,
                     UserAbortType abort_type,
                     base::TimeDelta time_to_abort);
@@ -74,8 +75,10 @@ struct PageLoadExtraInfo {
   // empty.
   const GURL committed_url;
 
-  // Time from navigation start until commit. If the page load did not commit,
-  // |time_to_commit| will be zero.
+  // Whether the navigation was committed.
+  const bool committed;

[Charlie Harrison, 2016/03/22 14:10:13]

Can you add:
TODO(csharrison): Consider removing this in favor of Option<time_to_commit>.

[Will Harris, 2016/03/22 18:13:17]

Done.

+
+  // Time from navigation start until commit. Only valid if |committed| is true.
   const base::TimeDelta time_to_commit;
 
   // The abort time and time to abort for this page load. If the page was not