Always suffix ChromeHTML entries on Windows for user-level installs.

chrome/browser/shell_integration_win.cc

Index: chrome/browser/shell_integration_win.cc
diff --git a/chrome/browser/shell_integration_win.cc b/chrome/browser/shell_integration_win.cc
index 869b5b51311f5ff8019a259bc8d3301c07c44629..69483977a8017e93031b2e36dc459bbb161b7dc4 100644
--- a/chrome/browser/shell_integration_win.cc
+++ b/chrome/browser/shell_integration_win.cc
@@ -74,14 +74,7 @@ ShellIntegration::DefaultWebClientState ProbeCurrentDefaultHandlers(
     return ShellIntegration::UNKNOWN_DEFAULT_WEB_CLIENT;
 
   string16 prog_id(ShellUtil::kChromeHTMLProgId);
-
-  // If a user specific default browser entry exists, we check for that ProgID
-  // being default. If not, then the ProgID is ChromeHTML or ChromiumHTML so we
-  // do not append a suffix to the ProgID.
-  string16 suffix;
-  BrowserDistribution* dist = BrowserDistribution::GetDistribution();
-  if (ShellUtil::GetUserSpecificDefaultBrowserSuffix(dist, &suffix))
-    prog_id += suffix;
+  prog_id += ShellUtil::GetCurrentInstallationSuffix();

[grt (UTC plus 2), 2012/05/30 20:37:40]

It would be nice to abstract the whole idea of the suffix away from consumers of
ShellUtil.  How about adding these method to ShellUtil for use by
shell_integration_win and others:

  static string16 GetProgId(BrowserDistribution*);
  static string16 GetApplicationName(BrowserDistribution*);

[gab, 2012/05/31 06:36:01]

Yea, I'm not sure functions in ShellUtil would serve that purpose though as the
consumers could still call the functions from BrowserDistribution not knowing
about our extra function.

I think the best way to fix this is through the template design pattern in
browser_distribution.h/cc:

Have browser_distribution.h/cc define and implement two public functions:
GetSuffixedAppId()
GetSuffixedApplicationName()
which are final (i.e. non-overridable, I used to do this all-the-time in Java,
not sure about C++)

Then move the current two functions to private.

And have the implementation in browser_distribution.cc of
GetSuffixedAppId()+GetSuffixedApplicationName() do the suffixing of the appid it
gets from the privately overriden non-suffixed getters (i.e. all the actual
browser distributions still return static strings and "do no work" as desired).

[gab, 2012/06/01 00:16:01]

Ok, I added this in http://codereview.chromium.org/10446111/, which I'd like to
checkin after this patch (I don't want to bloat up this CL with that change, but
on the other hand it's easier to base it on top of the new suffixing code than
making it work with the old one, checking it in, and adapt this CL to it
after...)

Let me know if that works for you.

 
   for (size_t i = 0; i < num_protocols; ++i) {
     base::win::ScopedCoMem<wchar_t> current_app;
@@ -107,13 +100,7 @@ ShellIntegration::DefaultWebClientState ProbeAppIsDefaultHandlers(
 
   BrowserDistribution* dist = BrowserDistribution::GetDistribution();
   string16 app_name(dist->GetApplicationName());
-
-  // If a user specific default browser entry exists, we check for that
-  // app name being default. If not, then default browser is just called
-  // Google Chrome or Chromium so we do not append a suffix to the app name.
-  string16 suffix;
-  if (ShellUtil::GetUserSpecificDefaultBrowserSuffix(dist, &suffix))
-    app_name += suffix;
+  app_name += ShellUtil::GetCurrentInstallationSuffix();
 
   BOOL result;
   for (size_t i = 0; i < num_protocols; ++i) {