Move GetProcessIntegrityLevel to file_info.h and remove the handle argument.

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 "base/base_paths.h"
 #include "base/bind.h"
@@ skipping 168 matching lines @@
 // 1. Windows 8 or greater.
 // 2. Not in Windows 8 immersive mode.
 // 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 base::FilePath& user_data_dir) {
-#if defined(USE_AURA)

[rvargas (doing something else), 2015/02/12 22:47:12]

I still see 460 uses of this definition.

[cpu_(ooo_6.6-7.5), 2015/02/13 00:25:12]

Mac is not yet aura so I hope that is the bulk of it. :(

   // Returning false from this function doesn't mean we don't launch immersive
   // mode in Aura. This function is specifically called in case when we need
   // to relaunch desktop launched chrome into immersive mode through 'relaunch'
   // menu. In case of Aura, we will use delegate_execute to do the relaunch.
   return false;
-#else
-  if (base::win::GetVersion() < base::win::VERSION_WIN8)
-    return false;
-
-  if (base::win::IsProcessImmersive(base::GetCurrentProcessHandle()))

[cpu_(ooo_6.6-7.5), 2015/02/13 00:25:12]

I believe IsProcessImmersive can be removed from base altogether. 

[rvargas (doing something else), 2015/02/13 02:53:19]

There are still two callers and I'm not sure it is aura only.

-    return false;
-
-  if (ShellIntegration::GetDefaultBrowser() != ShellIntegration::IS_DEFAULT)
-    return false;
-
-  base::IntegrityLevel integrity_level = base::INTEGRITY_UNKNOWN;
-  base::GetProcessIntegrityLevel(base::GetCurrentProcessHandle(),
-                                 &integrity_level);
-  if (integrity_level == base::HIGH_INTEGRITY)
-    return false;
-
-  base::FilePath default_user_data_dir;
-  if (!chrome::GetDefaultUserDataDirectory(&default_user_data_dir))
-    return false;
-
-  if (default_user_data_dir != user_data_dir)
-    return false;
-
-  base::win::RegKey reg_key;
-  DWORD reg_value = 0;
-  if (reg_key.Create(HKEY_CURRENT_USER, chrome::kMetroRegistryPath,
-                     KEY_READ) == ERROR_SUCCESS &&
-      reg_key.ReadValueDW(chrome::kLaunchModeValue,
-                          &reg_value) == ERROR_SUCCESS) {
-    return reg_value == 1;
-  }
-  return false;
-#endif
 }
 
 }  // namespace
 
 // Microsoft's Softricity virtualization breaks the sandbox processes.
 // So, if we detect the Softricity DLL we use WMI Win32_Process.Create to
 // break out of the virtualization environment.
 // http://code.google.com/p/chromium/issues/detail?id=43650
 bool ProcessSingleton::EscapeVirtualization(
     const base::FilePath& user_data_dir) {
@@ skipping 216 matching lines @@
           ::SetEvent(metro_activation_event.Get());
       }
     }
   }
 
   return window_.hwnd() != NULL;
 }
 
 void ProcessSingleton::Cleanup() {
 }