Instrument parser blocking script execution time.

chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc

Index: chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc
diff --git a/chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc b/chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc
index 153228f9b3f71447394cac069c0e656a8d414e14..a8805849f5021e5938c201c5acf601e8780f0b88 100644
--- a/chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc
+++ b/chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.cc
@@ -89,10 +89,18 @@ void PageLoadMetricsObserverTestHarness::PopulateRequiredTimingFields(
   if (inout_timing->parse_start) {
     if (!inout_timing->parse_blocked_on_script_load_duration)
       inout_timing->parse_blocked_on_script_load_duration = base::TimeDelta();
+    if (!inout_timing->parse_blocked_on_script_execution_duration)

[Charlie Harrison, 2016/09/07 14:57:37]

Optional nit: It makes me nervous to have multi-line declarations with a bare
"if" with no {}. This a personal style thing though and it's already throughout
this function so it's fine to keep it.

[Bryan McQuade, 2016/09/07 18:26:09]

Sure, makes sense. I fixed a few other cases as well.

[Charlie Harrison, 2016/09/07 18:32:50]

Thanks!

+      inout_timing->parse_blocked_on_script_execution_duration =
+          base::TimeDelta();
     if (!inout_timing
              ->parse_blocked_on_script_load_from_document_write_duration)
       inout_timing->parse_blocked_on_script_load_from_document_write_duration =
           base::TimeDelta();
+    if (!inout_timing
+             ->parse_blocked_on_script_execution_from_document_write_duration)
+      inout_timing
+          ->parse_blocked_on_script_execution_from_document_write_duration =
+          base::TimeDelta();
   }
 }