Commit cda7b03e authored by Henrique Nakashima's avatar Henrique Nakashima Committed by Commit Bot

Remove redundant tab FeatureFlag checks in FeatureUtilities.

Also remove DOWNLOAD_TAB_MANAGEMENT_MODULE since it becomes unused.

Bug: 1012975
Change-Id: I8a687bc395ebb1d0f775eb7b83321047ba4ecb52
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1863033Reviewed-by: default avatarWei-Yin Chen (陳威尹) <wychen@chromium.org>
Reviewed-by: default avatarYusuf Ozuysal <yusufo@chromium.org>
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#709045}
parent acbae27c
...@@ -240,7 +240,6 @@ public abstract class ChromeFeatureList { ...@@ -240,7 +240,6 @@ public abstract class ChromeFeatureList {
public static final String DOWNLOAD_OFFLINE_CONTENT_PROVIDER = public static final String DOWNLOAD_OFFLINE_CONTENT_PROVIDER =
"UseDownloadOfflineContentProvider"; "UseDownloadOfflineContentProvider";
public static final String DOWNLOADS_LOCATION_CHANGE = "DownloadsLocationChange"; public static final String DOWNLOADS_LOCATION_CHANGE = "DownloadsLocationChange";
public static final String DOWNLOAD_TAB_MANAGEMENT_MODULE = "DownloadTabManagementModule";
public static final String DRAW_VERTICALLY_EDGE_TO_EDGE = "DrawVerticallyEdgeToEdge"; public static final String DRAW_VERTICALLY_EDGE_TO_EDGE = "DrawVerticallyEdgeToEdge";
public static final String EPHEMERAL_TAB = "EphemeralTab"; public static final String EPHEMERAL_TAB = "EphemeralTab";
public static final String EPHEMERAL_TAB_USING_BOTTOM_SHEET = "EphemeralTabUsingBottomSheet"; public static final String EPHEMERAL_TAB_USING_BOTTOM_SHEET = "EphemeralTabUsingBottomSheet";
......
...@@ -516,12 +516,8 @@ public class FeatureUtilities { ...@@ -516,12 +516,8 @@ public class FeatureUtilities {
private static void cacheGridTabSwitcherEnabled() { private static void cacheGridTabSwitcherEnabled() {
ChromePreferenceManager.getInstance().writeBoolean(GRID_TAB_SWITCHER_ENABLED_KEY, ChromePreferenceManager.getInstance().writeBoolean(GRID_TAB_SWITCHER_ENABLED_KEY,
!DeviceClassManager.enableAccessibilityLayout() !DeviceClassManager.enableAccessibilityLayout()
&& (ChromeFeatureList.isEnabled( && ChromeFeatureList.isEnabled(ChromeFeatureList.TAB_GRID_LAYOUT_ANDROID)
ChromeFeatureList.DOWNLOAD_TAB_MANAGEMENT_MODULE) && TabManagementModuleProvider.getDelegate() != null);
|| ChromeFeatureList.isEnabled(
ChromeFeatureList.TAB_GRID_LAYOUT_ANDROID))
&& TabManagementModuleProvider.getDelegate() != null
&& ChromeFeatureList.isEnabled(ChromeFeatureList.TAB_GRID_LAYOUT_ANDROID));
} }
/** /**
...@@ -546,13 +542,8 @@ public class FeatureUtilities { ...@@ -546,13 +542,8 @@ public class FeatureUtilities {
private static void cacheTabGroupsAndroidEnabled() { private static void cacheTabGroupsAndroidEnabled() {
ChromePreferenceManager.getInstance().writeBoolean(TAB_GROUPS_ANDROID_ENABLED_KEY, ChromePreferenceManager.getInstance().writeBoolean(TAB_GROUPS_ANDROID_ENABLED_KEY,
!DeviceClassManager.enableAccessibilityLayout() !DeviceClassManager.enableAccessibilityLayout()
&& (ChromeFeatureList.isEnabled(
ChromeFeatureList.DOWNLOAD_TAB_MANAGEMENT_MODULE)
|| ChromeFeatureList.isEnabled(
ChromeFeatureList.TAB_GROUPS_ANDROID))
&& TabManagementModuleProvider.getDelegate() != null
&& ChromeFeatureList.isEnabled(ChromeFeatureList.TAB_GROUPS_ANDROID) && ChromeFeatureList.isEnabled(ChromeFeatureList.TAB_GROUPS_ANDROID)
&& isHighEndPhone()); && TabManagementModuleProvider.getDelegate() != null && isHighEndPhone());
} }
/** /**
......
...@@ -136,7 +136,6 @@ const base::Feature* kFeaturesExposedToJava[] = { ...@@ -136,7 +136,6 @@ const base::Feature* kFeaturesExposedToJava[] = {
&kDownloadNotificationBadge, &kDownloadNotificationBadge,
&kDownloadProgressInfoBar, &kDownloadProgressInfoBar,
&kDownloadRename, &kDownloadRename,
&kDownloadTabManagementModule,
&kDrawVerticallyEdgeToEdge, &kDrawVerticallyEdgeToEdge,
&kEphemeralTab, &kEphemeralTab,
&kEphemeralTabUsingBottomSheet, &kEphemeralTabUsingBottomSheet,
...@@ -417,9 +416,6 @@ const base::Feature kDownloadNotificationBadge{ ...@@ -417,9 +416,6 @@ const base::Feature kDownloadNotificationBadge{
const base::Feature kDownloadRename{"DownloadRename", const base::Feature kDownloadRename{"DownloadRename",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kDownloadTabManagementModule{
"DownloadTabManagementModule", base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kEphemeralTab{"EphemeralTab", const base::Feature kEphemeralTab{"EphemeralTab",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_DISABLED_BY_DEFAULT};
......
...@@ -69,7 +69,6 @@ extern const base::Feature kDownloadLocationShowImageInGallery; ...@@ -69,7 +69,6 @@ extern const base::Feature kDownloadLocationShowImageInGallery;
extern const base::Feature kDownloadNotificationBadge; extern const base::Feature kDownloadNotificationBadge;
extern const base::Feature kDownloadProgressInfoBar; extern const base::Feature kDownloadProgressInfoBar;
extern const base::Feature kDownloadRename; extern const base::Feature kDownloadRename;
extern const base::Feature kDownloadTabManagementModule;
extern const base::Feature kDrawVerticallyEdgeToEdge; extern const base::Feature kDrawVerticallyEdgeToEdge;
extern const base::Feature kEphemeralTab; extern const base::Feature kEphemeralTab;
extern const base::Feature kEphemeralTabUsingBottomSheet; extern const base::Feature kEphemeralTabUsingBottomSheet;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment