Enable offline pages prefetching flag

Sets the offline pages prefetching feature flag to be enabled by
default.

Bug: 881120
Change-Id: I735723193fbb19100e6962b8861d4fd04933e4c2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1562295Reviewed-by: Dan H <harringtond@chromium.org>
Commit-Queue: Ian Wells <iwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#654675}

components/offline_pages/core/client_policy_controller_unittest.cc

@@ -226,7 +226,7 @@ TEST_F(ClientPolicyControllerTest, CheckSuggestedArticlesDefined) {
   EXPECT_TRUE(isTemporary(policy));
   EXPECT_TRUE(controller()->IsRemovedOnCacheReset(kSuggestedArticlesNamespace));
   ExpectRemovedOnCacheReset(kSuggestedArticlesNamespace, true);
-  ExpectDownloadSupport(kSuggestedArticlesNamespace, false);
+  ExpectDownloadSupport(kSuggestedArticlesNamespace, true);
   ExpectUserRequestedDownloadSupport(kSuggestedArticlesNamespace, false);
   ExpectRecentTab(kSuggestedArticlesNamespace, false);
   ExpectRestrictedToTabFromClientId(kSuggestedArticlesNamespace, false);

components/offline_pages/core/offline_page_feature.cc

@@ -39,7 +39,7 @@ const base::Feature kOfflinePagesResourceBasedSnapshotFeature{
     "OfflinePagesResourceBasedSnapshot", base::FEATURE_DISABLED_BY_DEFAULT};
 
 const base::Feature kPrefetchingOfflinePagesFeature{
-    "OfflinePagesPrefetching", base::FEATURE_DISABLED_BY_DEFAULT};
+    "OfflinePagesPrefetching", base::FEATURE_ENABLED_BY_DEFAULT};
 
 const base::Feature kOfflinePagesCTV2Feature{"OfflinePagesCTV2",
                                              base::FEATURE_DISABLED_BY_DEFAULT};

components/offline_pages/core/offline_page_feature_unittest.cc

@@ -43,13 +43,13 @@ TEST(OfflinePageFeatureTest, OfflinePagesLoadSignalCollecting) {
 }
 
 TEST(OfflinePageFeatureTest, OfflinePagesPrefetching) {
-  // Disabled by default.
-  EXPECT_FALSE(offline_pages::IsPrefetchingOfflinePagesEnabled());
+  // Enabled by default.
+  EXPECT_TRUE(offline_pages::IsPrefetchingOfflinePagesEnabled());
 
-  // Check if helper method works correctly when the features is enabled.
+  // Check if helper method works correctly when the features is disabled.
   base::test::ScopedFeatureList scoped_feature_list;
-  scoped_feature_list.InitAndEnableFeature(kPrefetchingOfflinePagesFeature);
-  EXPECT_TRUE(offline_pages::IsPrefetchingOfflinePagesEnabled());
+  scoped_feature_list.InitAndDisableFeature(kPrefetchingOfflinePagesFeature);
+  EXPECT_FALSE(offline_pages::IsPrefetchingOfflinePagesEnabled());
 }
 
 TEST(OfflinePageFeatureTest, OfflinePagesInDownloadHomeOpenInCct) {