Tweaks to the precache triggering code.

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 103 matching lines @@
   }
 
   bool was_on_done_called() const {
     return was_on_done_called_;
   }
 
  private:
   bool was_on_done_called_;
 };
 
+class PrecacheManagerUnderTest : public PrecacheManager {
+ public:
+  PrecacheManagerUnderTest(content::BrowserContext* browser_context,
+                           const sync_driver::SyncService* const sync_service)
+      : PrecacheManager(browser_context, sync_service) {}
+  bool ShouldRun() const override { return true; }
+  bool WouldRun() const override { return true; }
+};
+
 class PrecacheManagerTest : public testing::Test {
  public:
   PrecacheManagerTest()
       : precache_manager_(&browser_context_, nullptr /* sync_service */),
         factory_(nullptr,
                  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);
 
     // Make the fetch of the precache configuration settings fail. Precaching
     // should still complete normally in this case.
     factory_.SetFakeResponse(GURL(kConfigURL), "",
                              net::HTTP_INTERNAL_SERVER_ERROR,
                              net::URLRequestStatus::FAILED);
   }
 
   content::TestBrowserThreadBundle test_browser_thread_bundle_;
   content::TestBrowserContext browser_context_;
-  PrecacheManager precache_manager_;
+  PrecacheManagerUnderTest precache_manager_;
   TestURLFetcherCallback url_callback_;
   net::FakeURLFetcherFactory factory_;
   TestPrecacheCompletionCallback precache_callback_;
 };
 
 TEST_F(PrecacheManagerTest, StartAndFinishPrecaching) {
   EXPECT_FALSE(precache_manager_.IsPrecaching());
 
   MockHistoryService history_service;
   MockHistoryService::TopHostsCallback top_hosts_callback;
@@ skipping 200 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