Removing unused previews_decider and associated setter

DataReductionProxyIOData historically added previews_decider_ member to
support AMP redirection, but this is not longer used.

Bug: 842233
Change-Id: If4b5a9a9a7c259fc0552de20a15e4d723a52e773
Reviewed-on: https://chromium-review.googlesource.com/1211842Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Stefan Kuhne <skuhne@chromium.org>
Commit-Queue: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#589361}

chrome/browser/profiles/profile_impl_io_data.cc

@@ -469,7 +469,6 @@ void ProfileImplIOData::InitializeInternal(
       data_reduction_proxy_io_data()->CreateInterceptor());
   data_reduction_proxy_io_data()->SetDataUseAscriber(
       io_thread_globals->data_use_ascriber.get());
-  data_reduction_proxy_io_data()->SetPreviewsDecider(previews_decider_impl());
   SetUpJobFactoryDefaultsForBuilder(
       builder, std::move(request_interceptors),
       std::move(profile_params->protocol_handler_interceptor));

components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data.cc

@@ -462,13 +462,6 @@ void DataReductionProxyIOData::SetDataUseAscriber(
   }
 }
 
-void DataReductionProxyIOData::SetPreviewsDecider(
-    previews::PreviewsDecider* previews_decider) {
-  DCHECK(io_task_runner_->BelongsToCurrentThread());
-  DCHECK(previews_decider);
-  previews_decider_ = previews_decider;
-}
-
 void DataReductionProxyIOData::UpdateProxyRequestHeaders(
     net::HttpRequestHeaders headers) {
   ui_task_runner_->PostTask(

components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data.h

@@ -85,8 +85,6 @@ class DataReductionProxyIOData : public DataReductionProxyEventStorageDelegate {
   virtual void SetDataReductionProxyService(
       base::WeakPtr<DataReductionProxyService> data_reduction_proxy_service);
 
-  void SetPreviewsDecider(previews::PreviewsDecider* previews_decider);
-
   // Creates an interceptor suitable for following the Data Reduction Proxy
   // bypass protocol.
   std::unique_ptr<net::URLRequestInterceptor> CreateInterceptor();
@@ -229,10 +227,6 @@ class DataReductionProxyIOData : public DataReductionProxyEventStorageDelegate {
     resource_type_provider_ = std::move(resource_type_provider);
   }
 
-  previews::PreviewsDecider* previews_decider() const {
-    return previews_decider_;
-  }
-
   // The production channel of this build.
   std::string channel() const { return channel_; }
 
@@ -319,10 +313,6 @@ class DataReductionProxyIOData : public DataReductionProxyEventStorageDelegate {
   // Observes pageload events and records per host data use.
   std::unique_ptr<DataReductionProxyDataUseObserver> data_use_observer_;
 
-  // Previews IO data that is owned by Profile IO data. Deleted at the same time
-  // as |this|.
-  previews::PreviewsDecider* previews_decider_;
-
   // Whether the Data Reduction Proxy has been enabled or not by the user. In
   // practice, this can be overridden by the command line.
   bool enabled_;