Index: chrome/browser/page_load_metrics/observers/document_write_page_load_metrics_observer.cc |
diff --git a/chrome/browser/page_load_metrics/observers/document_write_page_load_metrics_observer.cc b/chrome/browser/page_load_metrics/observers/document_write_page_load_metrics_observer.cc |
index e9b54992953749fd657da326def12b5c6183031a..4bb01dd3fda49fb660f42c3027250adc122f34a9 100644 |
--- a/chrome/browser/page_load_metrics/observers/document_write_page_load_metrics_observer.cc |
+++ b/chrome/browser/page_load_metrics/observers/document_write_page_load_metrics_observer.cc |
@@ -175,8 +175,6 @@ void DocumentWritePageLoadMetricsObserver::OnLoadingBehaviorObserved( |
void DocumentWritePageLoadMetricsObserver::OnComplete( |
const page_load_metrics::PageLoadTiming& timing, |
const page_load_metrics::PageLoadExtraInfo& info) { |
- if (!info.time_to_commit || timing.IsEmpty()) |
Charlie Harrison
2016/07/18 20:02:18
Why remove the IsEmpty() check?
Bryan McQuade
2016/07/19 12:55:38
Yeah, you're right, we might as well return early
|
- return; |
if (info.metadata.behavior_flags & |
blink::WebLoadingBehaviorFlag::WebLoadingBehaviorDocumentWriteEvaluator) { |
LogDocumentWriteEvaluatorData(timing, info); |