[iOS] Clean-up NativeContextMenu Feature Flag

Bug: 1110944
Change-Id: Ie0c4809718959ca06a1eff8bc7d309644d1d98c2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617580
Commit-Queue: Sebastien Lalancette <seblalancette@chromium.org>
Auto-Submit: Sebastien Lalancette <seblalancette@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#842129}

chrome/browser/flag-metadata.json

@@ -1915,11 +1915,6 @@
     "owners": [ "dschuff", "native-client-dev@googlegroups.com" ],
     "expiry_milestone": 100
   },
-  {
-    "name": "enable-native-context-menus",
-    "owners": [ "seblalancette" ],
-    "expiry_milestone": 89
-  },
   {
     "name": "enable-native-notifications",
     "owners": [ "peter", "finnur", "knollr" ],

ios/chrome/browser/flags/about_flags.mm

@@ -485,10 +485,6 @@ const flags_ui::FeatureEntry kFeatureEntries[] = {
     {"illustrated-empty-states", flag_descriptions::kIllustratedEmptyStatesName,
      flag_descriptions::kIllustratedEmptyStatesDescription, flags_ui::kOsIos,
      FEATURE_VALUE_TYPE(kIllustratedEmptyStates)},
-    {"enable-native-context-menus",
-     flag_descriptions::kEnableNativeContextMenusName,
-     flag_descriptions::kEnableNativeContextMenusDescription, flags_ui::kOsIos,
-     FEATURE_VALUE_TYPE(kEnableNativeContextMenus)},
     {"expanded-tab-strip", flag_descriptions::kExpandedTabStripName,
      flag_descriptions::kExpandedTabStripDescription, flags_ui::kOsIos,
      FEATURE_VALUE_TYPE(kExpandedTabStrip)},

ios/chrome/browser/flags/ios_chrome_flag_descriptions.cc

@@ -194,12 +194,6 @@ const char kEnableIOSManagedSettingsUIDescription[] =
     "Enable showing a different UI when the setting is managed by an "
     "enterprise policy on iOS.";
 
-const char kEnableNativeContextMenusName[] =
-    "Enable Context Menus in Native UI";
-const char kEnableNativeContextMenusDescription[] =
-    "Enables the new iOS 13 context menus on various pieces of UI in native "
-    "Chrome (e.g. History, Bookmarks).";
-
 const char kExpandedTabStripName[] = "Enable expanded tabstrip";
 const char kExpandedTabStripDescription[] =
     "Enables the new expanded tabstrip. Activated by swiping down the tabstrip"

ios/chrome/browser/flags/ios_chrome_flag_descriptions.h

@@ -169,10 +169,6 @@ extern const char kEnableIncognitoModeAvailabilityIOSDescription[];
 extern const char kEnableIOSManagedSettingsUIName[];
 extern const char kEnableIOSManagedSettingsUIDescription[];
 
-// Title and description for the flag to enable new context menus for native UI.
-extern const char kEnableNativeContextMenusName[];
-extern const char kEnableNativeContextMenusDescription[];
-
 // Title and description for the flag to enable an expanded tab strip.
 extern const char kExpandedTabStripName[];
 extern const char kExpandedTabStripDescription[];

ios/chrome/browser/ui/ui_feature_flags.cc

@@ -28,9 +28,6 @@ const base::Feature kForceUnstackedTabstrip{"ForceUnstackedTabstrip",
 const base::Feature kTestFeature{"TestFeature",
                                  base::FEATURE_DISABLED_BY_DEFAULT};
 
-const base::Feature kEnableNativeContextMenus{"EnableNativeContextMenus",
-                                              base::FEATURE_ENABLED_BY_DEFAULT};
-
 const base::Feature kEnableIOSManagedSettingsUI{
     "EnableIOSManagedSettingsUI", base::FEATURE_ENABLED_BY_DEFAULT};
 

ios/chrome/browser/ui/ui_feature_flags.h

@@ -31,9 +31,6 @@ extern const base::Feature kForceUnstackedTabstrip;
 // features in tests works.
 extern const base::Feature kTestFeature;
 
-// Feature flag that enables the native UI Context Menus (not for Web content).
-extern const base::Feature kEnableNativeContextMenus;
-
 // Feature flag to enable showing a different UI when the setting is managed by
 // an enterprise policy.
 extern const base::Feature kEnableIOSManagedSettingsUI;

ios/chrome/browser/ui/util/menu_util.mm

@@ -12,6 +12,5 @@
 #endif
 
 bool IsNativeContextMenuEnabled() {
-  return base::ios::IsRunningOnIOS13OrLater() &&
-         base::FeatureList::IsEnabled(kEnableNativeContextMenus);
+  return base::ios::IsRunningOnIOS13OrLater();
 }