[go: nahoru, domu]

win: fix set as default for branded builds

On Win11, navigating to the default app settings sub-page for Chrome
didn't work for dev and beta channels. This CL corrects the app name
appended to the ms-settings URI to be the application name instead
of the application id.

(cherry picked from commit 90ff8d38b36678b1d82e445c0f37c1a896ce315f)

Bug: 326283383
Change-Id: I04169600344fbbbe3beef2f9d8f866bbff8f76f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5323322
Reviewed-by: Greg Thompson <grt@chromium.org>
Commit-Queue: David Bienvenu <davidbienvenu@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1266387}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5350819
Auto-Submit: David Bienvenu <davidbienvenu@chromium.org>
Reviewed-by: Jesse McKenna <jessemckenna@google.com>
Commit-Queue: Jesse McKenna <jessemckenna@google.com>
Cr-Commit-Position: refs/branch-heads/6312@{#438}
Cr-Branched-From: 6711dcdae48edaf98cbc6964f90fac85b7d9986e-refs/heads/main@{#1262506}
diff --git a/chrome/installer/util/shell_util.cc b/chrome/installer/util/shell_util.cc
index 679c1af..746a1d8f 100644
--- a/chrome/installer/util/shell_util.cc
+++ b/chrome/installer/util/shell_util.cc
@@ -2090,7 +2090,7 @@
           base::StrCat({L"ms-settings:defaultapps?",
                         is_per_user_install ? L"registeredAppUser="
                                             : L"registeredAppMachine=",
-                        ShellUtil::GetBrowserModelId(is_per_user_install)});
+                        GetApplicationName(chrome_exe)});
       succeeded = reinterpret_cast<intptr_t>(
                       ShellExecute(nullptr, L"open", settings_url.c_str(),
                                    nullptr, nullptr, SW_SHOWNORMAL)) > 32;