| Index: chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.cc
|
| diff --git a/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.cc b/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.cc
|
| index 8b460cfb68e4f8f21181cdab31716b01b1d71204..e0a445497921a99852fed6905e4c743c7f78d0f7 100644
|
| --- a/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.cc
|
| +++ b/chrome/browser/page_load_metrics/observers/data_reduction_proxy_metrics_observer.cc
|
| @@ -91,7 +91,8 @@ DataReductionProxyMetricsObserver::DataReductionProxyMetricsObserver()
|
| DataReductionProxyMetricsObserver::~DataReductionProxyMetricsObserver() {}
|
|
|
| // Check if the NavigationData indicates anything about the DataReductionProxy.
|
| -void DataReductionProxyMetricsObserver::OnCommit(
|
| +page_load_metrics::PageLoadMetricsObserver::ObservePolicy
|
| +DataReductionProxyMetricsObserver::OnCommit(
|
| content::NavigationHandle* navigation_handle) {
|
| // This BrowserContext is valid for the lifetime of
|
| // DataReductionProxyMetricsObserver. BrowserContext is always valid and
|
| @@ -110,15 +111,16 @@ void DataReductionProxyMetricsObserver::OnCommit(
|
| static_cast<ChromeNavigationData*>(
|
| navigation_handle->GetNavigationData());
|
| if (!chrome_navigation_data)
|
| - return;
|
| + return STOP_OBSERVING;
|
| data_reduction_proxy::DataReductionProxyData* data =
|
| chrome_navigation_data->GetDataReductionProxyData();
|
| if (!data)
|
| - return;
|
| + return STOP_OBSERVING;
|
| data_ = data->DeepCopy();
|
| // DataReductionProxy page loads should only occur on HTTP navigations.
|
| DCHECK(!data_->used_data_reduction_proxy() ||
|
| !navigation_handle->GetURL().SchemeIsCryptographic());
|
| + return CONTINUE_OBSERVING;
|
| }
|
|
|
| void DataReductionProxyMetricsObserver::OnComplete(
|
|
|