Add flag for Read Later feature.

Bug: 1109316
Change-Id: I3dec74d1f9a8448602c5a5a5b998f344870605c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317836Reviewed-by: Peter Boström <pbos@chromium.org>
Commit-Queue: Caroline Rising <corising@chromium.org>
Cr-Commit-Position: refs/heads/master@{#791476}

chrome/browser/about_flags.cc

@@ -3932,6 +3932,10 @@ const FeatureEntry kFeatureEntries[] = {
      FEATURE_VALUE_TYPE(features::kDoubleTapToZoomInTabletMode)},
 #endif  // defined(OS_CHROMEOS)
 
+    {"read-later", flag_descriptions::kReadLaterName,
+     flag_descriptions::kReadLaterDescription, kOsDesktop,
+     FEATURE_VALUE_TYPE(features::kReadLater)},
+
     {"tab-groups", flag_descriptions::kTabGroupsName,
      flag_descriptions::kTabGroupsDescription, kOsDesktop,
      FEATURE_VALUE_TYPE(features::kTabGroups)},

chrome/browser/flag-metadata.json

@@ -3650,6 +3650,11 @@
     "owners": [ "huangdarwin", "pwnall" ],
     "expiry_milestone": 88
   },
+  {
+    "name": "read-later",
+    "owners": [ "chrome-desktop-ui-seattle@google.com", "corising" ],
+    "expiry_milestone": 90
+  },
   {
     "name": "reader-mode-heuristics",
     "owners": [ "mdjones", "wychen" ],

chrome/browser/flag_descriptions.cc

@@ -1874,6 +1874,11 @@ const char kRawClipboardDescription[] =
     "Allows raw / unsanitized clipboard content to be read and written. "
     "See https://github.com/WICG/raw-clipboard-access.";
 
+const char kReadLaterName[] = "Read Later";
+const char kReadLaterDescription[] =
+    "Allow users to save tabs for later. Enables a new button and menu for "
+    "accessing tabs saved for later.";
+
 const char kReducedReferrerGranularityName[] =
     "Reduce default 'referer' header granularity.";
 const char kReducedReferrerGranularityDescription[] =

chrome/browser/flag_descriptions.h

@@ -1089,6 +1089,9 @@ extern const char kQuietNotificationPromptsDescription[];
 extern const char kRawClipboardName[];
 extern const char kRawClipboardDescription[];
 
+extern const char kReadLaterName[];
+extern const char kReadLaterDescription[];
+
 extern const char kReducedReferrerGranularityName[];
 extern const char kReducedReferrerGranularityDescription[];
 

chrome/browser/ui/ui_features.cc

@@ -47,6 +47,10 @@ const base::Feature kPermissionChip{"PermissionChip",
 const base::Feature kProminentDarkModeActiveTabTitle{
     "ProminentDarkModeActiveTabTitle", base::FEATURE_DISABLED_BY_DEFAULT};
 
+// Allow users to save tabs for later. Enables a new button and menu for
+// accessing tabs saved for later. https://crbug.com/1109316
+const base::Feature kReadLater{"ReadLater", base::FEATURE_DISABLED_BY_DEFAULT};
+
 // Enables tabs to scroll in the tabstrip. https://crbug.com/951078
 const base::Feature kScrollableTabStrip{"ScrollableTabStrip",
                                         base::FEATURE_DISABLED_BY_DEFAULT};

chrome/browser/ui/ui_features.h

@@ -36,6 +36,8 @@ extern const base::Feature kPermissionChip;
 
 extern const base::Feature kProminentDarkModeActiveTabTitle;
 
+extern const base::Feature kReadLater;
+
 extern const base::Feature kScrollableTabStrip;
 
 extern const base::Feature kTabGroups;

tools/metrics/histograms/enums.xml

@@ -40276,6 +40276,7 @@ from previous Chrome versions.
   <int value="-1532014193" label="disable-encryption-migration"/>
   <int value="-1530711962" label="PrivacyElevatedAndroid:disabled"/>
   <int value="-1529907580" label="ImeServiceConnectable:disabled"/>
+  <int value="-1529140321" label="ReadLater:enabled"/>
   <int value="-1528455406" label="OmniboxPedalSuggestions:enabled"/>
   <int value="-1525129540"
       label="OmniboxUIExperimentRevealSteadyStateUrlPathQueryAndRefOnHover:disabled"/>
@@ -41539,6 +41540,7 @@ from previous Chrome versions.
       label="AutofillEnforceMinRequiredFieldsForHeuristics:enabled"/>
   <int value="-181279574" label="NativeFilesystemAPI:enabled"/>
   <int value="-181093956" label="ScrollAnchoring:enabled"/>
+  <int value="-180658998" label="ReadLater:disabled"/>
   <int value="-180481252" label="CompositorThreadedScrollbarScrolling:enabled"/>
   <int value="-179580377" label="CrossOriginOpenerPolicy:enabled"/>
   <int value="-178364053" label="TerminalSystemAppLegacySettings:enabled"/>