Optional <TimeDelta> since they may be non existent.

components/page_load_metrics/browser/page_load_metrics_observer.cc

Index: components/page_load_metrics/browser/page_load_metrics_observer.cc
diff --git a/components/page_load_metrics/browser/page_load_metrics_observer.cc b/components/page_load_metrics/browser/page_load_metrics_observer.cc
index 23a00fbaee3f6ce879bf04cf4539156b1f8207ab..e1ecf9c25c19590cb267efbf264adcf0062d4a33 100644
--- a/components/page_load_metrics/browser/page_load_metrics_observer.cc
+++ b/components/page_load_metrics/browser/page_load_metrics_observer.cc
@@ -7,13 +7,13 @@
 namespace page_load_metrics {
 
 PageLoadExtraInfo::PageLoadExtraInfo(
-    base::TimeDelta first_background_time,
-    base::TimeDelta first_foreground_time,
+    const base::Optional<base::TimeDelta>& first_background_time,
+    const base::Optional<base::TimeDelta>& first_foreground_time,
     bool started_in_foreground,
     const GURL& committed_url,
-    base::TimeDelta time_to_commit,
+    const base::Optional<base::TimeDelta>& time_to_commit,
     UserAbortType abort_type,
-    base::TimeDelta time_to_abort)
+    const base::Optional<base::TimeDelta>& time_to_abort)
     : first_background_time(first_background_time),
       first_foreground_time(first_foreground_time),
       started_in_foreground(started_in_foreground),
@@ -22,4 +22,7 @@ PageLoadExtraInfo::PageLoadExtraInfo(
       abort_type(abort_type),
       time_to_abort(time_to_abort) {}
 
+// Using Optional <T> for the fields made clang complaint about not having
+// an explicit out-of-line destructor
+PageLoadExtraInfo::~PageLoadExtraInfo() {}
 }  // namespace page_load_metrics