| Index: chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
|
| diff --git a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
|
| index 29ce7bc4b0fcdad737f236eba370a5b5a615014b..4f1d8d3ae07e1d5d7cf1f72e232076ab0e0b3bb3 100644
|
| --- a/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
|
| +++ b/chrome/browser/page_load_metrics/metrics_web_contents_observer.cc
|
| @@ -9,7 +9,6 @@
|
| #include <string>
|
| #include <utility>
|
|
|
| -#include "base/feature_list.h"
|
| #include "base/location.h"
|
| #include "base/memory/ptr_util.h"
|
| #include "base/metrics/histogram_macros.h"
|
| @@ -17,7 +16,6 @@
|
| #include "chrome/browser/page_load_metrics/page_load_metrics_embedder_interface.h"
|
| #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
|
| #include "chrome/browser/page_load_metrics/page_load_tracker.h"
|
| -#include "chrome/common/chrome_features.h"
|
| #include "chrome/common/page_load_metrics/page_load_metrics_messages.h"
|
| #include "chrome/common/page_load_metrics/page_load_timing.h"
|
| #include "content/public/browser/browser_thread.h"
|
| @@ -72,8 +70,7 @@
|
| : content::WebContentsObserver(web_contents),
|
| in_foreground_(false),
|
| embedder_interface_(std::move(embedder_interface)),
|
| - has_navigated_(false),
|
| - page_load_metrics_binding_(web_contents, this) {
|
| + has_navigated_(false) {
|
| RegisterInputEventObserver(web_contents->GetRenderViewHost());
|
| }
|
|
|
| @@ -140,7 +137,7 @@
|
| bool handled = true;
|
| IPC_BEGIN_MESSAGE_MAP_WITH_PARAM(MetricsWebContentsObserver, message,
|
| render_frame_host)
|
| - IPC_MESSAGE_HANDLER(PageLoadMetricsMsg_TimingUpdated, OnUpdateTimingOverIPC)
|
| + IPC_MESSAGE_HANDLER(PageLoadMetricsMsg_TimingUpdated, OnTimingUpdated)
|
| IPC_MESSAGE_UNHANDLED(handled = false)
|
| IPC_END_MESSAGE_MAP()
|
| return handled;
|
| @@ -620,29 +617,6 @@
|
| observer.OnTimingUpdated(is_main_frame, timing, metadata);
|
| }
|
|
|
| -void MetricsWebContentsObserver::OnUpdateTimingOverIPC(
|
| - content::RenderFrameHost* render_frame_host,
|
| - const mojom::PageLoadTiming& timing,
|
| - const mojom::PageLoadMetadata& metadata) {
|
| - DCHECK(!base::FeatureList::IsEnabled(features::kPageLoadMetricsMojofication));
|
| - OnTimingUpdated(render_frame_host, timing, metadata);
|
| -
|
| - for (auto& observer : testing_observers_)
|
| - observer.DidReceiveTimingUpdate(TestingObserver::IPCType::LEGACY);
|
| -}
|
| -
|
| -void MetricsWebContentsObserver::UpdateTiming(
|
| - const mojom::PageLoadTimingPtr timing,
|
| - const mojom::PageLoadMetadataPtr metadata) {
|
| - content::RenderFrameHost* render_frame_host =
|
| - page_load_metrics_binding_.GetCurrentTargetFrame();
|
| - DCHECK(base::FeatureList::IsEnabled(features::kPageLoadMetricsMojofication));
|
| - OnTimingUpdated(render_frame_host, *timing, *metadata);
|
| -
|
| - for (auto& observer : testing_observers_)
|
| - observer.DidReceiveTimingUpdate(TestingObserver::IPCType::MOJO);
|
| -}
|
| -
|
| bool MetricsWebContentsObserver::ShouldTrackNavigation(
|
| content::NavigationHandle* navigation_handle) const {
|
| DCHECK(navigation_handle->IsInMainFrame());
|
|
|