Chrome should never attempt to launch in Metro if it is no longer default browser

chrome/browser/process_singleton_win.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.
 
 #include "chrome/browser/process_singleton.h"
 
 #include <shellapi.h>
 #include <shobjidl.h>
 
 #include "base/base_paths.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/metro.h"
 #include "base/win/registry.h"
 #include "base/win/scoped_com_initializer.h"
 #include "base/win/scoped_comptr.h"
 #include "base/win/scoped_handle.h"
 #include "base/win/win_util.h"
 #include "base/win/windows_version.h"
 #include "base/win/wrapped_window_proc.h"
+#include "chrome/browser/shell_integration.h"
 #include "chrome/browser/ui/simple_message_box.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_paths_internal.h"
 #include "chrome/installer/util/browser_distribution.h"
 #include "chrome/installer/util/install_util.h"
 #include "chrome/installer/util/shell_util.h"
 #include "chrome/installer/util/wmi.h"
 #include "content/public/common/result_codes.h"
 #include "grit/chromium_strings.h"
@@ skipping 129 matching lines @@
     NOTREACHED() << "Failed to activate metro chrome. Error: " << hr;
     return false;
   }
   return true;
 }
 
 // Returns true if Chrome needs to be relaunched into Windows 8 immersive mode.
 // Following conditions apply:-
 // 1. Windows 8 or greater.
 // 2. Not in Windows 8 immersive mode.
-// 3. Process integrity level is not high.
-// 4. The profile data directory is the default directory .
-// 5. Last used mode was immersive/machine is a tablet.
+// 3. Chrome is default browser.
+// 4. Process integrity level is not high.
+// 5. The profile data directory is the default directory.
+// 6. Last used mode was immersive/machine is a tablet.
 // TODO(ananta)
 // Move this function to a common place as the Windows 8 delegate_execute
 // handler can possibly use this.
 bool ShouldLaunchInWindows8ImmersiveMode(const FilePath& user_data_dir) {
 #if defined(USE_AURA)
   return false;
 #endif
 
   if (base::win::GetVersion() < base::win::VERSION_WIN8)
     return false;
 
   if (base::win::IsProcessImmersive(base::GetCurrentProcessHandle()))
     return false;
 
+  if (!ShellIntegration::IsDefaultBrowser())
+    return false;
+
   base::IntegrityLevel integrity_level = base::INTEGRITY_UNKNOWN;
   base::GetProcessIntegrityLevel(base::GetCurrentProcessHandle(),
                                  &integrity_level);
   if (integrity_level == base::HIGH_INTEGRITY)
     return false;
 
   FilePath default_user_data_dir;
   if (!chrome::GetDefaultUserDataDirectory(&default_user_data_dir))
     return false;
 
@@ skipping 46 matching lines @@
   }
   return false;
 }
 
 // Look for a Chrome instance that uses the same profile directory.
 // If there isn't one, create a message window with its title set to
 // the profile directory path.
 ProcessSingleton::ProcessSingleton(const FilePath& user_data_dir)
     : window_(NULL), locked_(false), foreground_window_(NULL),
     is_virtualized_(false), lock_file_(INVALID_HANDLE_VALUE) {
   FilePath default_user_data_dir;

[grt (UTC plus 2), 2012/10/11 03:26:06]

unused, please remove

[gab, 2012/10/11 03:57:08]

How did you see that far in this CL?! Eagle eyes code review :)!

Done.

   // For Windows 8 and above check if we need to relaunch into Windows 8
   // immersive mode.
   if (ShouldLaunchInWindows8ImmersiveMode(user_data_dir)) {
     bool immersive_chrome_launched = ActivateMetroChrome();
     if (!immersive_chrome_launched) {
       LOG(WARNING) << "Failed to launch immersive chrome";
     } else {
       // Sleep to allow the immersive chrome process to create its initial
       // message window.
       SleepEx(kImmersiveChromeInitTimeout, FALSE);
@@ skipping 251 matching lines @@
   switch (message) {
     case WM_COPYDATA:
       return OnCopyData(reinterpret_cast<HWND>(wparam),
                         reinterpret_cast<COPYDATASTRUCT*>(lparam));
     default:
       break;
   }
 
   return ::DefWindowProc(hwnd, message, wparam, lparam);
 }