Remove feature flags for default Chat/Meet apps

This feature has shipped in M92 and these flags are no longer needed.

Bug: 1203138
Change-Id: I188f1449aad03c4f81efd8b937c58cfb948d3e0c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3149802
Reviewed-by: Eric Willigers <[email protected]>
Commit-Queue: Eric Willigers <[email protected]>
Auto-Submit: Alan Cutter <[email protected]>
Cr-Commit-Position: refs/heads/main@{#920035}

Author: alancutter

chrome/browser/about_flags.cc

@@ -3669,12 +3669,6 @@ const FeatureEntry kFeatureEntries[] = {
      kOsCrOS,
      FEATURE_VALUE_TYPE(
          web_app::kAllowDefaultWebAppMigrationForChromeOsManagedUsers)},
-    {"enable-default-chat-web-app", flag_descriptions::kDefaultChatWebAppName,
-     flag_descriptions::kDefaultChatWebAppDescription, kOsCrOS,
-     FEATURE_VALUE_TYPE(web_app::kDefaultChatWebApp)},
-    {"enable-default-meet-web-app", flag_descriptions::kDefaultMeetWebAppName,
-     flag_descriptions::kDefaultMeetWebAppDescription, kOsCrOS,
-     FEATURE_VALUE_TYPE(web_app::kDefaultMeetWebApp)},
 #endif  // BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 #if BUILDFLAG(IS_CHROMEOS_ASH)
     {"enable-desktop-pwas-attention-badging-cros",

chrome/browser/flag-metadata.json

@@ -1744,22 +1744,6 @@
     "owners": [ "[email protected]", "[email protected]" ],
     "expiry_milestone": 99
   },
-  {
-    "name": "enable-default-chat-web-app",
-    "owners": [
-      "[email protected]",
-      "[email protected]"
-    ],
-    "expiry_milestone": 94
-  },
-  {
-    "name": "enable-default-meet-web-app",
-    "owners": [
-      "[email protected]",
-      "[email protected]"
-    ],
-    "expiry_milestone": 94
-  },
   {
     "name": "enable-defer-all-script",
     "owners": [ "//components/data_reduction_proxy/OWNERS" ],

chrome/browser/flag_descriptions.cc

@@ -5144,14 +5144,6 @@ const char kDefaultCalculatorWebAppName[] = "Default install Calculator PWA";
 const char kDefaultCalculatorWebAppDescription[] =
     "Enable default installing of the calculator PWA instead of the deprecated "
     "chrome app.";
-
-const char kDefaultChatWebAppName[] = "Default Chat web app";
-const char kDefaultChatWebAppDescription[] =
-    "Enables the Chat web app to be installed by default.";
-
-const char kDefaultMeetWebAppName[] = "Default Meet web app";
-const char kDefaultMeetWebAppDescription[] =
-    "Enables the Meet web app to be installed by default.";
 #endif  // BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 
 #if defined(OS_CHROMEOS)

chrome/browser/flag_descriptions.h

@@ -2978,12 +2978,6 @@ extern const char
 
 extern const char kDefaultCalculatorWebAppName[];
 extern const char kDefaultCalculatorWebAppDescription[];
-
-extern const char kDefaultChatWebAppName[];
-extern const char kDefaultChatWebAppDescription[];
-
-extern const char kDefaultMeetWebAppName[];
-extern const char kDefaultMeetWebAppDescription[];
 #endif  // BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 
 #if defined(OS_CHROMEOS)

chrome/browser/web_applications/preinstalled_app_install_features.cc

@@ -18,10 +18,6 @@ namespace {
 constexpr const base::Feature* kPreinstalledAppInstallFeatures[] = {
     &kMigrateDefaultChromeAppToWebAppsGSuite,
     &kMigrateDefaultChromeAppToWebAppsNonGSuite,
-#if BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
-    &kDefaultChatWebApp,
-    &kDefaultMeetWebApp,
-#endif  // BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 };
 
 bool g_always_enabled_for_testing = false;
@@ -67,13 +63,6 @@ const base::Feature kAllowDefaultWebAppMigrationForChromeOsManagedUsers{
     "AllowDefaultWebAppMigrationForChromeOsManagedUsers",
     base::FEATURE_DISABLED_BY_DEFAULT};
 
-// Enables default installing the Chat web app.
-const base::Feature kDefaultChatWebApp{"DefaultChatWebApp",
-                                       base::FEATURE_ENABLED_BY_DEFAULT};
-
-// Enables default installing the Meet web app.
-const base::Feature kDefaultMeetWebApp{"DefaultMeetWebApp",
-                                       base::FEATURE_ENABLED_BY_DEFAULT};
 #endif  // BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 
 bool IsPreinstalledAppInstallFeatureEnabled(base::StringPiece feature_name,

chrome/browser/web_applications/preinstalled_app_install_features.h

@@ -22,10 +22,6 @@ extern const base::Feature kDefaultCalculatorWebApp;
 
 #if BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 extern const base::Feature kAllowDefaultWebAppMigrationForChromeOsManagedUsers;
-
-extern const base::Feature kDefaultChatWebApp;
-
-extern const base::Feature kDefaultMeetWebApp;
 #endif  // BUILDFLAG(IS_CHROMEOS_ASH) || BUILDFLAG(IS_CHROMEOS_LACROS)
 
 // Returns the base::Feature in |kPreinstalledAppInstallFeatures| that

chrome/browser/web_applications/preinstalled_web_apps/google_chat.cc

@@ -4,8 +4,6 @@
 
 #include "chrome/browser/web_applications/preinstalled_web_apps/google_chat.h"
 
-#include "chrome/browser/web_applications/preinstalled_app_install_features.h"
-
 namespace web_app {
 
 ExternalInstallOptions GetConfigForGoogleChat() {
@@ -17,7 +15,6 @@ ExternalInstallOptions GetConfigForGoogleChat() {
 
   // Exclude managed users until we have a way for admins to block the app.
   options.user_type_allowlist = {"unmanaged"};
-  options.gate_on_feature = kDefaultChatWebApp.name;
   options.only_for_new_users = true;
   options.add_to_quick_launch_bar = false;
 

chrome/browser/web_applications/preinstalled_web_apps/google_meet.cc

@@ -4,8 +4,6 @@
 
 #include "chrome/browser/web_applications/preinstalled_web_apps/google_meet.h"
 
-#include "chrome/browser/web_applications/preinstalled_app_install_features.h"
-
 namespace web_app {
 
 ExternalInstallOptions GetConfigForGoogleMeet() {
@@ -17,7 +15,6 @@ ExternalInstallOptions GetConfigForGoogleMeet() {
 
   // Exclude managed users until we have a way for admins to block the app.
   options.user_type_allowlist = {"unmanaged", "child"};
-  options.gate_on_feature = kDefaultMeetWebApp.name;
   options.only_for_new_users = true;
   options.add_to_quick_launch_bar = false;