Moved data reduction proxy initialization logic to ProfileImplIOData

components/data_reduction_proxy/browser/data_reduction_proxy_config_service_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/data_reduction_proxy/browser/data_reduction_proxy_config_service.h"
 
 #include <string>
 
+#include "base/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop/message_loop.h"
 #include "base/test/test_simple_task_runner.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using testing::Mock;
 
 namespace {
 
@@ skipping 171 matching lines @@
 
   config_service_->RemoveObserver(&observer);
 }
 
 TEST_F(DataReductionProxyConfigServiceTest, TrackerEnable) {
   MockObserver observer;
   //base::MessageLoopForUI loop;
   config_service_->AddObserver(&observer);
   scoped_refptr<base::TestSimpleTaskRunner> task_runner_(
       new base::TestSimpleTaskRunner());
-  DataReductionProxyConfigTracker tracker(config_service_.get(),
-                                          task_runner_.get());
+  DataReductionProxyConfigTracker tracker(
+      base::Bind(&data_reduction_proxy::DataReductionProxyConfigService::
+                     UpdateProxyConfig,
+                 base::Unretained(config_service_.get())),
+      task_runner_.get());
   net::ProxyConfig expected_config;
   expected_config.proxy_rules().ParseFromString(kDataReductionProxyRules);
   EXPECT_CALL(observer, OnProxyConfigChanged(
                   ProxyConfigMatches(expected_config),
                   net::ProxyConfigService::CONFIG_VALID)).Times(1);
   tracker.Enable(false,
                  false,
                  "https://foo.com:443",
                  "http://bar.com:80",
                  "");
   task_runner_->RunUntilIdle();
   Mock::VerifyAndClearExpectations(&observer);
 
   config_service_->RemoveObserver(&observer);
 }
 
 TEST_F(DataReductionProxyConfigServiceTest, TrackerEnableRestricted) {
   MockObserver observer;
   //base::MessageLoopForUI loop;
   config_service_->AddObserver(&observer);
   scoped_refptr<base::TestSimpleTaskRunner> task_runner_(
       new base::TestSimpleTaskRunner());
-  DataReductionProxyConfigTracker tracker(config_service_.get(),
-                                          task_runner_.get());
+  DataReductionProxyConfigTracker tracker(
+      base::Bind(&data_reduction_proxy::DataReductionProxyConfigService::
+                     UpdateProxyConfig,
+                 base::Unretained(config_service_.get())),
+      task_runner_.get());
   net::ProxyConfig expected_config;
   expected_config.proxy_rules().ParseFromString(
       kDataReductionProxyRestrictedRules);
   EXPECT_CALL(observer, OnProxyConfigChanged(
                   ProxyConfigMatches(expected_config),
                   net::ProxyConfigService::CONFIG_VALID)).Times(1);
   tracker.Enable(true,
                  false,
                  "https://foo.com:443",
                  "http://bar.com:80",
                  "");
   task_runner_->RunUntilIdle();
   Mock::VerifyAndClearExpectations(&observer);
 
   config_service_->RemoveObserver(&observer);
 }
 
 TEST_F(DataReductionProxyConfigServiceTest, TrackerDisable) {
   MockObserver observer;
   //base::MessageLoopForUI loop;
   config_service_->AddObserver(&observer);
   scoped_refptr<base::TestSimpleTaskRunner> task_runner_(
       new base::TestSimpleTaskRunner());
-  DataReductionProxyConfigTracker tracker(config_service_.get(),
-                                          task_runner_.get());
+  DataReductionProxyConfigTracker tracker(
+      base::Bind(&data_reduction_proxy::DataReductionProxyConfigService::
+                     UpdateProxyConfig,
+                 base::Unretained(config_service_.get())),
+      task_runner_.get());
   net::ProxyConfig expected_config;
   expected_config.proxy_rules().ParseFromString(kSystemProxyRules);
   EXPECT_CALL(observer, OnProxyConfigChanged(
                   ProxyConfigMatches(expected_config),
                   net::ProxyConfigService::CONFIG_VALID)).Times(1);
   tracker.Disable();
   task_runner_->RunUntilIdle();
   //loop.RunUntilIdle();
   Mock::VerifyAndClearExpectations(&observer);
 
   config_service_->RemoveObserver(&observer);
 }
 
 
 TEST_F(DataReductionProxyConfigServiceTest, TrackerBypassList) {
   base::MessageLoopForUI loop;
   scoped_refptr<base::TestSimpleTaskRunner> task_runner_(
       new base::TestSimpleTaskRunner());
-  DataReductionProxyConfigTracker tracker(config_service_.get(),
-                                          task_runner_.get());
+  DataReductionProxyConfigTracker tracker(
+      base::Bind(&data_reduction_proxy::DataReductionProxyConfigService::
+                     UpdateProxyConfig,
+                 base::Unretained(config_service_.get())),
+      task_runner_.get());
   tracker.AddHostPatternToBypass("http://www.google.com");
   tracker.AddHostPatternToBypass("fefe:13::abc/33");
   tracker.AddURLPatternToBypass("foo.org/images/*");
   tracker.AddURLPatternToBypass("http://foo.com/*");
   tracker.AddURLPatternToBypass("http://baz.com:22/bar/*");
   tracker.AddURLPatternToBypass("http://*bat.com/bar/*");
 
   std::string expected[] = {
     "http://www.google.com",
     "fefe:13::abc/33",
     "foo.org",
     "http://foo.com",
     "http://baz.com:22",
     "http://*bat.com"
   };
 
   ASSERT_EQ(tracker.bypass_rules_.size(), 6u);
   int i = 0;
   for (std::vector<std::string>::iterator it = tracker.bypass_rules_.begin();
        it != tracker.bypass_rules_.end(); ++it) {
     EXPECT_EQ(expected[i++], *it);
   }
 }
 
 }  // namespace data_reduction_proxy