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

chrome/installer/util/shell_util.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // This file defines functions that integrate Chrome in Windows shell. These
 // functions can be used by Chrome as well as Chrome installer. All of the
 // work is done by the local functions defined in anonymous namespace in
 // this class.
 
 #include "chrome/installer/util/shell_util.h"
@@ skipping 596 matching lines @@
       else
         one_mismatch = true;
     }
   }
   return true;
 }
 
 // Launches the Windows 7 and Windows 8 application association dialog, which
 // is the only documented way to make a browser the default browser on
 // Windows 8.
-bool LaunchApplicationAssociationDialog(const string16& app_id) {
-  base::win::ScopedComPtr<IApplicationAssociationRegistrationUI> aarui;
-  HRESULT hr = aarui.CreateInstance(CLSID_ApplicationAssociationRegistrationUI);
-  if (FAILED(hr))
-    return false;
-  hr = aarui->LaunchAdvancedAssociationUI(app_id.c_str());
+bool LaunchApplicationAssociationDialog() {

[grt (UTC plus 2), 2012/05/28 20:47:02]

please update the comment and function name since this now brings up the "How do
you want to open web pages dialog" rather than the advanced association UI.

[gab, 2012/05/29 16:05:19]

Also, can you leave the old call around? It will need to be invoked if we are to
have a button in settings to pop this "all defaults" dialog.

[motek., 2012/05/31 21:59:04]

Done.

[motek., 2012/05/31 21:59:04]

Done.

+  const wchar_t* protocol = L"http";

[grt (UTC plus 2), 2012/05/28 20:47:02]

static const wchar_t protocol[] = L"http";
to be sure it's all in .rdata

[motek., 2012/05/31 21:59:04]

Done.

+  OPENASINFO open_as_info = {};
+  open_as_info.pcszFile = protocol;
+  open_as_info.oaifInFlags =
+      OAIF_URL_PROTOCOL | OAIF_FORCE_REGISTRATION | OAIF_REGISTER_EXT;
+  HRESULT hr = SHOpenWithDialog(NULL, &open_as_info);
   return SUCCEEDED(hr);

[grt (UTC plus 2), 2012/05/28 20:47:02]

how about adding this just before the return:
DLOG_IF(WARNING, FAILED(hr)) << "Failed to set as default http handler; hr=0x"
<< std::hex << hr;

[motek., 2012/05/31 21:59:04]

Done.

 }
 
 uint32 ConvertShellUtilShortcutOptionsToFileUtil(uint32 options) {
   uint32 converted_options = 0;
   if (options & ShellUtil::SHORTCUT_DUAL_MODE)
     converted_options |= file_util::SHORTCUT_DUAL_MODE;
   if (options & ShellUtil::SHORTCUT_CREATE_ALWAYS)
     converted_options |= file_util::SHORTCUT_CREATE_ALWAYS;
   return converted_options;
 }
@@ skipping 304 matching lines @@
                 KEY_READ).Valid();
 }
 
 bool ShellUtil::MakeChromeDefault(BrowserDistribution* dist,
                                   int shell_change,
                                   const string16& chrome_exe,
                                   bool elevate_if_not_admin) {
   if (!dist->CanSetAsDefault())
     return false;
 
+  // Windows 8 does not permit making a browser default just like that.
+  // This process needs to be routed through the system's UI. Use
+  // ShowMakeChromeDefaultSystemUI instead (below).
+  if (base::win::GetVersion() >= base::win::VERSION_WIN8)
+    return false;

[grt (UTC plus 2), 2012/05/28 20:47:02]

should there be a NOTREACHED() here?

[motek., 2012/05/31 21:59:04]

Done.

+
   ShellUtil::RegisterChromeBrowser(dist, chrome_exe, L"", elevate_if_not_admin);
 
   bool ret = true;
   // First use the new "recommended" way on Vista to make Chrome default
   // browser.
   string16 app_name = dist->GetApplicationName();
   string16 app_suffix;
   if (ShellUtil::GetUserSpecificDefaultBrowserSuffix(dist, &app_suffix))
     app_name += app_suffix;
 
-  if (base::win::GetVersion() >= base::win::VERSION_WIN8) {
-    // On Windows 8, you can't set yourself as the default handler
-    // programatically. In other words IApplicationAssociationRegistration
-    // has been rendered useless. What you can do is to launch
-    // "Set Program Associations" section of the "Default Programs"
-    // control panel. This action does not require elevation and we
-    // don't get to control window activation. More info at:
-    // http://msdn.microsoft.com/en-us/library/cc144154(VS.85).aspx
-    return LaunchApplicationAssociationDialog(app_name.c_str());
-
-  } else if (base::win::GetVersion() >= base::win::VERSION_VISTA) {
+  if (base::win::GetVersion() >= base::win::VERSION_VISTA) {
     // On Windows Vista and Win7 we still can set ourselves via the
     // the IApplicationAssociationRegistration interface.
     VLOG(1) << "Registering Chrome as default browser on Vista.";
     base::win::ScopedComPtr<IApplicationAssociationRegistration> pAAR;
     HRESULT hr = pAAR.CreateInstance(CLSID_ApplicationAssociationRegistration,
         NULL, CLSCTX_INPROC);
     if (SUCCEEDED(hr)) {
       for (int i = 0; ShellUtil::kBrowserProtocolAssociations[i] != NULL; i++) {
         hr = pAAR->SetAppAsDefault(app_name.c_str(),
             ShellUtil::kBrowserProtocolAssociations[i], AT_URLPROTOCOL);
@@ skipping 41 matching lines @@
     ret = false;
     LOG(ERROR) << "Could not make Chrome default browser (XP/system level).";
   }
 
   // Send Windows notification event so that it can update icons for
   // file associations.
   SHChangeNotify(SHCNE_ASSOCCHANGED, SHCNF_IDLIST, NULL, NULL);
   return ret;
 }
 
+bool ShellUtil::ShowMakeChromeDefaultSystemUI(
+    BrowserDistribution* dist, int shell_change, const string16& chrome_exe,

[grt (UTC plus 2), 2012/05/28 20:47:02]

put each parameters on its own line, indented four spaces

[motek., 2012/05/31 21:59:04]

Done.

+        bool elevate_if_not_admin) {
+  if (!dist->CanSetAsDefault())
+    return false;
+

[gab, 2012/05/29 16:05:19]

This is only for Win8+ right? If so, can you add a DCHECK here to enforce the
contract? (and add it in the comment in the header)

[motek., 2012/05/31 21:59:04]

I don't know. It is not only Windows 8. This dialog functions in earlier
versions and this is how I debug it for the most part, too. There is no good
reason to use it, so I have done what you suggest, but I have my doubts...

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

Thanks, my point is that it will never be used on Win7 and forcing Win8+ will
allow me to remove unnecessary extra parameters (i.e. elevate_if_not_admin) once
my HKCU only patch goes in. I wouldn't like to have to keep parameters around
just for the sake of being compatible on an OS we will never use it on.

+  ShellUtil::RegisterChromeBrowser(dist, chrome_exe, L"", elevate_if_not_admin);

[gab, 2012/05/29 16:05:19]

Note to self (and reminder to my future reviewers!): once my UAC patch goes in
(and provided this function is only for Win8+), elevate_if_not_admin can always
be false and there is thus no need to have this parameter propagated through the
call stack leading here.

For now (as this will go in before my UAC patch) it makes sense to leave it here
(so that we do get UAC on first run to be able to get this flow: as otherwise
chrome's capabilities are not registered in the current registration flow...)

[motek., 2012/05/31 21:59:04]

Well, OK.

+
+  // On Windows 8, you can't set yourself as the default handler
+  // programatically. In other words IApplicationAssociationRegistration
+  // has been rendered useless. What you can do is to launch
+  // "Set Program Associations" section of the "Default Programs"
+  // control panel. This action does not require elevation and we

[grt (UTC plus 2), 2012/05/28 20:47:02]

i suggest replacing the last few lines of this comment with:
  // control panel, which is a mess, or pop the concise "How you want to open
  // webpages"? dialog.  We choose the latter.

[motek., 2012/05/31 21:59:04]

Done.

+  // don't get to control window activation. More info at:
+  // http://msdn.microsoft.com/en-us/library/cc144154(VS.85).aspx
+  return LaunchApplicationAssociationDialog();
+}
+
 bool ShellUtil::MakeChromeDefaultProtocolClient(BrowserDistribution* dist,
                                                 const string16& chrome_exe,
                                                 const string16& protocol) {
   if (!dist->CanSetAsDefault())
     return false;
 
   ShellUtil::RegisterChromeForProtocol(dist, chrome_exe, L"", protocol, true);
 
   bool ret = true;
   // First use the new "recommended" way on Vista to make Chrome default
@@ skipping 251 matching lines @@
       chrome_exe.c_str(),
       shortcut.c_str(),
       chrome_path.c_str(),
       arguments.c_str(),
       description.c_str(),
       icon_path.c_str(),
       icon_index,
       dist->GetBrowserAppId().c_str(),
       ConvertShellUtilShortcutOptionsToFileUtil(options));
 }