| Index: chrome/renderer/page_load_histograms.cc
|
| diff --git a/chrome/renderer/page_load_histograms.cc b/chrome/renderer/page_load_histograms.cc
|
| index 0fccbd1ae41c1bf115638c76f1fe4a2e365aeed9..127451226117757318c0179d433fdd9783310072 100644
|
| --- a/chrome/renderer/page_load_histograms.cc
|
| +++ b/chrome/renderer/page_load_histograms.cc
|
| @@ -106,10 +106,13 @@ void PageLoadHistograms::Dump(WebFrame* frame) {
|
| // Record the new PLT times prior to the faulty abandon check below.
|
| // TODO(tonyg): There are many new details we can record, add them after the
|
| // basic metrics are evaluated.
|
| - TimeDelta nav_start_to_load_start = load_event_start - navigation_start;
|
| - TimeDelta nav_start_to_load_end = load_event_end - navigation_start;
|
| - PLT_HISTOGRAM("PLT.NavStartToLoadStart", nav_start_to_load_start);
|
| - PLT_HISTOGRAM("PLT.NavStartToLoadEnd", nav_start_to_load_end);
|
| + // TODO(simonjam): There is no way to distinguish between abandonment and
|
| + // intentional Javascript navigation before the load event fires.
|
| + if (!load_event_start.is_null())
|
| + PLT_HISTOGRAM("PLT.NavStartToLoadStart",
|
| + load_event_start - navigation_start);
|
| + if (!load_event_end.is_null())
|
| + PLT_HISTOGRAM("PLT.NavStartToLoadEnd", load_event_end - navigation_start);
|
|
|
| // We properly handle null values for the next 3 variables.
|
| Time request = navigation_state->request_time();
|
|
|