Precache should cancel when there is user traffic

components/precache/content/precache_manager_unittest.cc

Index: components/precache/content/precache_manager_unittest.cc
diff --git a/components/precache/content/precache_manager_unittest.cc b/components/precache/content/precache_manager_unittest.cc
index 27b167f31548ca8d25aba396be7d9e3e356e1944..d075231b4b0aa96e5f9110b285b9d80f426f5d69 100644
--- a/components/precache/content/precache_manager_unittest.cc
+++ b/components/precache/content/precache_manager_unittest.cc
@@ -147,6 +147,8 @@ class PrecacheManagerUnderTest : public PrecacheManager {
   bool control_group_;
 };
 
+}  // namespace
+
 class PrecacheManagerTest : public testing::Test {
  public:
   PrecacheManagerTest()
@@ -342,6 +344,8 @@ TEST_F(PrecacheManagerTest, StartAndCancelPrecachingAfterURLsReceived) {
   EXPECT_EQ(expected_requested_urls, url_callback_.requested_urls());
 }
 
+// TODO(rajendrant): Add unittests for
+// PrecacheUtil::UpdatePrecacheMetricsAndState() for more test coverage.
 TEST_F(PrecacheManagerTest, RecordStatsForFetchWithSizeZero) {
   // Fetches with size 0 should be ignored.
   precache_manager_->RecordStatsForFetch(GURL("http://url.com"), GURL(),
@@ -523,6 +527,4 @@ TEST_F(PrecacheManagerTest, DeleteExpiredPrecacheHistory) {
               ContainerEq(expected_histogram_count_map));
 }
 
-}  // namespace
-
 }  // namespace precache