Fix IsPrecachingAllowed() to check Sync instead of Data Saver.

components/precache/content/precache_manager_unittest.cc

 // Copyright 2013 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/precache/content/precache_manager.h"
 
 #include <map>
 #include <set>
 #include <string>
 
@@ skipping 106 matching lines @@
     return was_on_done_called_;
   }
 
  private:
   bool was_on_done_called_;
 };
 
 class PrecacheManagerTest : public testing::Test {
  public:
   PrecacheManagerTest()
-      : precache_manager_(&browser_context_),
-        factory_(NULL, base::Bind(&TestURLFetcherCallback::CreateURLFetcher,
-                                  base::Unretained(&url_callback_))) {}
+      : precache_manager_(&browser_context_, nullptr /* sync_service */),
+        factory_(NULL,

[bengr, 2015/06/23 22:16:19]

Mind changing this to nullptr too?

[twifkak, 2015/06/23 23:54:51]

Not at all. Done.

+                 base::Bind(&TestURLFetcherCallback::CreateURLFetcher,
+                            base::Unretained(&url_callback_))) {}
 
  protected:
   void SetUp() override {
     base::StatisticsRecorder::Initialize();
 
     base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
         switches::kPrecacheConfigSettingsURL, kConfigURL);
     base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
         switches::kPrecacheManifestURLPrefix, kManifestURLPrefix);
 
@@ skipping 220 matching lines @@
       kCurrentTime, 1000, true);
   expected_histogram_count_map["Precache.Saved"] += 2;
 
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_EQ(expected_histogram_count_map, GetHistogramCountMap());
 }
 
 }  // namespace
 
 }  // namespace precache