Fix RegisterChromeBrowser definition and usage in shell_util

chrome/installer/util/shell_util.h

Index: chrome/installer/util/shell_util.h
diff --git a/chrome/installer/util/shell_util.h b/chrome/installer/util/shell_util.h
index 4215ce83e5ddf68d87870812253bb90f4e46bcef..055e95b640287782f52fc480afce4b6f659ec929 100644
--- a/chrome/installer/util/shell_util.h
+++ b/chrome/installer/util/shell_util.h
@@ -253,27 +253,31 @@ class ShellUtil {
                                               const string16& chrome_exe,
                                               const string16& protocol);
 
-  // This method adds Chrome to the list that shows up in Add/Remove Programs->
-  // Set Program Access and Defaults and also creates Chrome ProgIds under
-  // Software\Classes. This method requires write access to HKLM so is just
-  // best effort deal. If write to HKLM fails and elevate_if_not_admin is true,
-  // this method will:
-  // - add the ProgId entries to HKCU on XP. HKCU entries will not make
-  //   Chrome show in Set Program Access and Defaults but they are still useful
-  //   because we can make Chrome run when user clicks on http link or html
-  //   file.
-  // - will try to launch setup.exe with admin priviledges on Vista to do
-  //   these tasks. Users will see standard Vista elevation prompt and if they
-  //   enter the right credentials, the write operation will work.
-  // Currently elevate_if_not_admin is true only when user tries to make Chrome
-  // default browser (through the UI or through installer options) and Chrome
-  // is not registered on the machine.
+  // Registers Chrome as a potential default browser and handler for filetypes
+  // and protocols.
+  // If Chrome is already registered, this method is a no-op.
+  // This method requires write access to HKLM so is just a best effort deal.
+  // If write to HKLM fails and:
+  // - |elevate_if_not_admin| is true (and OS is Vista or above):
+  //   tries to launch setup.exe with admin priviledges (by prompting the user
+  //   with a UAC) to do these tasks.
+  // - |elevate_if_not_admin| is false (or OS is XP):
+  //   adds the ProgId entries to HKCU. These entries will not make Chrome show
+  //   in Default Programs but they are still useful because Chrome can be
+  //   registered to run when  the user clicks on an http link or an html file.

[grt (UTC plus 2), 2012/06/11 17:57:38]

nit: "when   the" -> "when the"

[gab, 2012/06/11 19:38:11]

Done.

   //
+  // |elevate_if_not_admin| should only be true when following a user action

[grt (UTC plus 2), 2012/06/11 17:57:38]

merge this in with the documentation for |elevate_if_not_admin| on line 277

[gab, 2012/06/11 19:38:11]

Ah good catch! Done.

+  // (e.g. "Make Chrome Default") as it allows this method to UAC.
   // |chrome_exe| full path to chrome.exe.
   // |unique_suffix| Optional input. If given, this function appends the value
   // to default browser entries names that it creates in the registry.
+  // Currently, this is only used to continue an install with the same suffix
+  // when elevating and calling setup.exe with admin privileges as described
+  // above.
   // |elevate_if_not_admin| if true will make this method try alternate methods
   // as described above.
+  //
+  // Returns true if Chrome is successfully registered (or already registered).
   static bool RegisterChromeBrowser(BrowserDistribution* dist,
                                     const string16& chrome_exe,
                                     const string16& unique_suffix,