Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2113)

Unified Diff: chrome/browser/metro_utils/metro_chrome_win.cc

Issue 1581473002: Remove base/win/metro.{cc|h} and some associated code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixes Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/metro_utils/metro_chrome_win.cc
diff --git a/chrome/browser/metro_utils/metro_chrome_win.cc b/chrome/browser/metro_utils/metro_chrome_win.cc
deleted file mode 100644
index 2bab9e17dd3e87dde7f16e06c6fe4f3954a1b1fb..0000000000000000000000000000000000000000
--- a/chrome/browser/metro_utils/metro_chrome_win.cc
+++ /dev/null
@@ -1,72 +0,0 @@
-// Copyright 2013 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/metro_utils/metro_chrome_win.h"
-
-#include <windows.h>
-#include <shobjidl.h>
-
-#include "base/files/file_path.h"
-#include "base/path_service.h"
-#include "base/win/metro.h"
-#include "base/win/scoped_com_initializer.h"
-#include "base/win/scoped_comptr.h"
-#include "base/win/windows_version.h"
-#include "chrome/installer/util/browser_distribution.h"
-#include "chrome/installer/util/install_util.h"
-#include "chrome/installer/util/shell_util.h"
-
-namespace chrome {
-
-bool ActivateMetroChrome() {
- // TODO(cpu): For Win7 we need to activate differently.
- if (base::win::GetVersion() < base::win::VERSION_WIN8)
- return true;
-
- base::FilePath chrome_exe;
- if (!PathService::Get(base::FILE_EXE, &chrome_exe)) {
- NOTREACHED() << "Failed to get chrome exe path";
- return false;
- }
-
- base::string16 app_id =
- ShellUtil::GetBrowserModelId(BrowserDistribution::GetDistribution(),
- InstallUtil::IsPerUserInstall(chrome_exe));
- if (app_id.empty()) {
- NOTREACHED() << "Failed to get chrome app user model id.";
- return false;
- }
-
- base::win::ScopedComPtr<IApplicationActivationManager> activation_manager;
- HRESULT hr = activation_manager.CreateInstance(
- CLSID_ApplicationActivationManager);
- if (!activation_manager.get()) {
- NOTREACHED() << "Failed to cocreate activation manager. Error: "
- << std::showbase << std::hex << hr;
- return false;
- }
-
- unsigned long pid = 0;
- hr = activation_manager->ActivateApplication(app_id.c_str(),
- L"open",
- AO_NONE,
- &pid);
- if (FAILED(hr)) {
- NOTREACHED() << "Failed to activate metro chrome. Error: "
- << std::showbase << std::hex << hr;
- return false;
- }
-
- return true;
-}
-
-Win8Environment GetWin8Environment(HostDesktopType desktop) {
- if (desktop == chrome::HOST_DESKTOP_TYPE_ASH)
- return WIN_8_ENVIRONMENT_METRO_AURA;
- else
- return WIN_8_ENVIRONMENT_DESKTOP_AURA;
-}
-
-
-} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698