Allow TimeTicks for serialized page load operations to be equal.

components/page_load_metrics/browser/page_load_metrics_observer.h

 // Copyright 2015 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 COMPONENTS_PAGE_LOAD_METRICS_BROWSER_PAGE_LOAD_METRICS_OBSERVER_H_
 #define COMPONENTS_PAGE_LOAD_METRICS_BROWSER_PAGE_LOAD_METRICS_OBSERVER_H_
 
 #include "base/macros.h"
 #include "components/page_load_metrics/common/page_load_timing.h"
 #include "content/public/browser/navigation_handle.h"
@@ skipping 37 matching lines @@
 
   // Add values before this final count.
   ABORT_LAST_ENTRY
 };
 
 struct PageLoadExtraInfo {
   PageLoadExtraInfo(base::TimeDelta first_background_time,
                     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);
 
   // The first time that the page was backgrounded since the navigation started.
   // If the page has not been backgrounded this will be base::TimeDelta().
   const base::TimeDelta first_background_time;
 
   // The first time that the page was foregrounded since the navigation started.
   // If the page has not been foregrounded this will be base::TimeDelta().
   const base::TimeDelta first_foreground_time;
 
   // True if the page load started in the foreground.
   const bool started_in_foreground;
 
   // Committed URL. If the page load did not commit, |committed_url| will be
   // 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
   // aborted, |abort_type| will be |ABORT_NONE| and |time_to_abort| will be
   // |base::TimeDelta()|.
   const UserAbortType abort_type;
   const base::TimeDelta time_to_abort;
 };
 
 // Interface for PageLoadMetrics observers. All instances of this class are
@@ skipping 31 matching lines @@
   // data collected over the course of the page load. If the load did not
   // receive any timing information, |timing.IsEmpty()| will be true.
   // After this call, the object will be deleted.
   virtual void OnComplete(const PageLoadTiming& timing,
                           const PageLoadExtraInfo& extra_info) {}
 };
 
 }  // namespace page_load_metrics
 
 #endif  // COMPONENTS_PAGE_LOAD_METRICS_BROWSER_PAGE_LOAD_METRICS_OBSERVER_H_