[go: nahoru, domu]

iwa: Remove "Manage your Chrome profiles" link

This isn't needed as we're not letting users change their profile
during installation.

Bug: 1479140
Change-Id: I5547d2d25bdcfdd7fbc3290724ca51e9b413e6e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5194859
Reviewed-by: Zelin Liu <zelin@chromium.org>
Commit-Queue: Robbie McElrath <rmcelrath@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1247680}
diff --git a/chrome/app/generated_resources.grd b/chrome/app/generated_resources.grd
index 66d0c62..38fdd96 100644
--- a/chrome/app/generated_resources.grd
+++ b/chrome/app/generated_resources.grd
@@ -16741,10 +16741,7 @@
       Installing...
     </message>
     <message name="IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE" desc="Subtitle of the pre-installation app info summary screen of the Isolated Web App installer">
-      This application will be installed on your currently used Chrome profile. <ph name="MANAGE_PROFILES">$1<ex>Manage your Chrome profiles</ex></ph>
-    </message>
-    <message name="IDS_IWA_INSTALLER_SHOW_METADATA_MANAGE_PROFILES" desc="Link to a settings page that lets the user manage their active profiles">
-      Manage your Chrome profiles
+      This application will be installed on your currently used Chrome profile
     </message>
     <message name="IDS_IWA_INSTALLER_SHOW_METADATA_APP_NAME_LABEL" desc="Label before the Isolated Web App name in the pre-install app info summary screen">
       Application: <ph name="APP_NAME">$1<ex>Google Maps</ex></ph>
diff --git a/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_MANAGE_PROFILES.png.sha1 b/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_MANAGE_PROFILES.png.sha1
deleted file mode 100644
index 838f4ab..0000000
--- a/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_MANAGE_PROFILES.png.sha1
+++ /dev/null
@@ -1 +0,0 @@
-3957fd916535b10ed204bcccffaf1338b620b8f4
\ No newline at end of file
diff --git a/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE.png.sha1 b/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE.png.sha1
index 38c678f..9478cde5 100644
--- a/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE.png.sha1
+++ b/chrome/app/generated_resources_grd/IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE.png.sha1
@@ -1 +1 @@
-194f8c0363fdcf8fea4d402d30559b6343706e9f
\ No newline at end of file
+1a8b1e49bcc6ff3736d1548c61cf6a718615f0f7
\ No newline at end of file
diff --git a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view.h b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view.h
index ddfddd3..4f38bb6d 100644
--- a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view.h
+++ b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view.h
@@ -44,7 +44,6 @@
   class Delegate {
    public:
     virtual void OnSettingsLinkClicked() = 0;
-    virtual void OnManageProfilesLinkClicked() = 0;
     virtual void OnChildDialogCanceled() = 0;
     virtual void OnChildDialogAccepted() = 0;
   };
diff --git a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.cc b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.cc
index cad26c8..77145e6 100644
--- a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.cc
+++ b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.cc
@@ -405,10 +405,6 @@
 #endif  // BUILDFLAG(IS_CHROMEOS_LACROS)
 }
 
-void IsolatedWebAppInstallerViewController::OnManageProfilesLinkClicked() {
-  // TODO(crbug.com/1479140): Implement
-}
-
 void IsolatedWebAppInstallerViewController::OnChildDialogCanceled() {
   // Currently all child dialogs should close the installer when closed.
   Close();
diff --git a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.h b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.h
index 5dd510e..89fb808 100644
--- a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.h
+++ b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_controller.h
@@ -98,7 +98,6 @@
 
   // `IsolatedWebAppInstallerView::Delegate`:
   void OnSettingsLinkClicked() override;
-  void OnManageProfilesLinkClicked() override;
   void OnChildDialogCanceled() override;
   void OnChildDialogAccepted() override;
 
diff --git a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_impl.cc b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_impl.cc
index a612066..5a08df2 100644
--- a/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_impl.cc
+++ b/chrome/browser/ui/views/web_apps/isolated_web_apps/isolated_web_app_installer_view_impl.cc
@@ -317,11 +317,7 @@
             CreateImageModelFromVector(kFingerprintIcon, ui::kColorAccent),
             // The title will be updated to the app name when available.
             IDS_IWA_INSTALLER_VERIFICATION_TITLE,
-            IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE,
-            IDS_IWA_INSTALLER_SHOW_METADATA_MANAGE_PROFILES,
-            base::BindRepeating(&IsolatedWebAppInstallerView::Delegate::
-                                    OnManageProfilesLinkClicked,
-                                base::Unretained(delegate))) {
+            IDS_IWA_INSTALLER_SHOW_METADATA_SUBTITLE) {
     // Initialize the View with dummy data so the initial height calculation
     // will be roughly accurate.
     std::vector<std::pair<int, std::u16string>> info = {