| Index: chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
|
| diff --git a/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc b/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
|
| index 059653cbaf9ebc27469efbaf6d40328aa3d5bb12..10cc4b0a2497ae3c44e446de378cc2eb9a3f871b 100644
|
| --- a/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
|
| +++ b/chrome/renderer/page_load_metrics/metrics_render_frame_observer.cc
|
| @@ -38,9 +38,7 @@ MetricsRenderFrameObserver::MetricsRenderFrameObserver(
|
| MetricsRenderFrameObserver::~MetricsRenderFrameObserver() {}
|
|
|
| void MetricsRenderFrameObserver::DidChangePerformanceTiming() {
|
| - // Only track timing metrics for main frames.
|
| - if (IsMainFrame())
|
| - SendMetrics();
|
| + SendMetrics();
|
| }
|
|
|
| void MetricsRenderFrameObserver::DidObserveLoadingBehavior(
|
| @@ -72,14 +70,8 @@ void MetricsRenderFrameObserver::DidCommitProvisionalLoad(
|
| // non-null, we will send metrics for the current page at some later time, as
|
| // those metrics become available.
|
| if (ShouldSendMetrics()) {
|
| - PageLoadTiming timing;
|
| - if (IsMainFrame()) {
|
| - // Only populate PageLoadTiming for the main frame.
|
| - timing = GetTiming();
|
| - DCHECK(!timing.navigation_start.is_null());
|
| - }
|
| - page_timing_metrics_sender_.reset(
|
| - new PageTimingMetricsSender(this, routing_id(), CreateTimer(), timing));
|
| + page_timing_metrics_sender_ = base::MakeUnique<PageTimingMetricsSender>(
|
| + this, routing_id(), CreateTimer(), GetTiming());
|
| }
|
| }
|
|
|
| @@ -185,8 +177,4 @@ void MetricsRenderFrameObserver::OnDestruct() {
|
| delete this;
|
| }
|
|
|
| -bool MetricsRenderFrameObserver::IsMainFrame() const {
|
| - return render_frame()->IsMainFrame();
|
| -}
|
| -
|
| } // namespace page_load_metrics
|
|
|