Support for interactive set-chrome-as-default in Windows.

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 85e5fbd7d0f9e8123a5e2b2157810f3ad4ede95f..6cc85f6d3743bb56028b048a05a827e9c6954af4 100644
--- a/chrome/installer/util/shell_util.h
+++ b/chrome/installer/util/shell_util.h
@@ -237,6 +237,16 @@ class ShellUtil {
                                 const string16& chrome_exe,
                                 bool elevate_if_not_admin);
 
+  // Shows to the user a system dialog where Chrome can be set as the
+  // default browser. This is intended for Windows 8, but is known to work
+  // on Windows 7, too. This is a blocking call.

[gab, 2012/05/31 22:17:20]

Please change "This is intended for Windows 8, but is known to work on Windows
7, too."
to "This is intended for Windows 8 and above only."

So that the contract in the comment matches the implementation.

[grt (UTC plus 2), 2012/06/01 14:36:39]

Agreed.  If you'd like to leave a note for posterity indicating that it
technically does work in Win7 as well, please put it in the implementation since
it's not relevant to consumers of this function.

[motek., 2012/06/01 20:50:17]

Done.

[motek., 2012/06/01 20:50:17]

Ack.

+  //
+  // Parameters as MakeChromeDefault.
+  static bool ShowMakeChromeDefaultSystemUI(BrowserDistribution* dist,
+                                            int shell_change,

[grt (UTC plus 2), 2012/06/01 14:36:39]

is this parameter needed?  my understanding is that this function is purely a
per-user operation.

[motek., 2012/06/01 20:50:17]

Done.

+                                            const string16& chrome_exe,
+                                            bool elevate_if_not_admin);
+
   // Make Chrome the default application for a protocol.
   // chrome_exe: The chrome.exe path to register as default browser.
   // protocol: The protocol to register as the default handler for.