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

Side by Side Diff: chrome/installer/util/chrome_app_host_distribution.cc

Issue 14031025: Implementing unified Chrome / App Launcher flow, and migrating old stand-alone App Launcher. (Closed) Base URL: http://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This file defines a specific implementation of BrowserDistribution class for 5 // This file defines a specific implementation of BrowserDistribution class for
6 // Chrome App Host. It overrides the bare minimum of methods necessary to get a 6 // Chrome App Host. It overrides the bare minimum of methods necessary to get a
7 // Chrome App Host installer that does not interact with Google Chrome or 7 // Chrome App Host installer that does not interact with Google Chrome or
8 // Chromium installations. 8 // Chromium installations.
9 9
10 #include "chrome/installer/util/chrome_app_host_distribution.h" 10 #include "chrome/installer/util/chrome_app_host_distribution.h"
11 11
12 #include "base/string_util.h" 12 #include "base/string_util.h"
13 #include "base/version.h"
13 #include "chrome/common/net/test_server_locations.h" 14 #include "chrome/common/net/test_server_locations.h"
14 #include "chrome/installer/util/channel_info.h" 15 #include "chrome/installer/util/channel_info.h"
15 #include "chrome/installer/util/google_update_constants.h" 16 #include "chrome/installer/util/google_update_constants.h"
16 #include "chrome/installer/util/google_update_settings.h" 17 #include "chrome/installer/util/google_update_settings.h"
17 #include "chrome/installer/util/helper.h" 18 #include "chrome/installer/util/helper.h"
18 #include "chrome/installer/util/install_util.h" 19 #include "chrome/installer/util/install_util.h"
19 #include "chrome/installer/util/l10n_string_util.h" 20 #include "chrome/installer/util/l10n_string_util.h"
20 21
21 #include "installer_util_strings.h" // NOLINT 22 #include "installer_util_strings.h" // NOLINT
22 23
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 return chrome_common_net::kPipelineTestServerBaseUrl; 109 return chrome_common_net::kPipelineTestServerBaseUrl;
109 } 110 }
110 111
111 string16 ChromeAppHostDistribution::GetUninstallLinkName() { 112 string16 ChromeAppHostDistribution::GetUninstallLinkName() {
112 const string16& link_name = 113 const string16& link_name =
113 installer::GetLocalizedString(IDS_UNINSTALL_APP_LAUNCHER_BASE); 114 installer::GetLocalizedString(IDS_UNINSTALL_APP_LAUNCHER_BASE);
114 return link_name; 115 return link_name;
115 } 116 }
116 117
117 string16 ChromeAppHostDistribution::GetUninstallRegPath() { 118 string16 ChromeAppHostDistribution::GetUninstallRegPath() {
118 return L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\" 119 NOTREACHED();
119 L"Google Chrome App Launcher"; 120 return string16();
120 } 121 }
121 122
122 string16 ChromeAppHostDistribution::GetVersionKey() { 123 string16 ChromeAppHostDistribution::GetVersionKey() {
123 string16 key(google_update::kRegPathClients); 124 string16 key(google_update::kRegPathClients);
124 key.append(L"\\"); 125 key.append(L"\\");
125 key.append(kChromeAppHostGuid); 126 key.append(kChromeAppHostGuid);
126 return key; 127 return key;
127 } 128 }
128 129
129 bool ChromeAppHostDistribution::CanSetAsDefault() { 130 bool ChromeAppHostDistribution::CanSetAsDefault() {
(...skipping 15 matching lines...) Expand all
145 146
146 void ChromeAppHostDistribution::UpdateInstallStatus(bool system_install, 147 void ChromeAppHostDistribution::UpdateInstallStatus(bool system_install,
147 installer::ArchiveType archive_type, 148 installer::ArchiveType archive_type,
148 installer::InstallStatus install_status) { 149 installer::InstallStatus install_status) {
149 #if defined(GOOGLE_CHROME_BUILD) 150 #if defined(GOOGLE_CHROME_BUILD)
150 GoogleUpdateSettings::UpdateInstallStatus(system_install, 151 GoogleUpdateSettings::UpdateInstallStatus(system_install,
151 archive_type, InstallUtil::GetInstallReturnCode(install_status), 152 archive_type, InstallUtil::GetInstallReturnCode(install_status),
152 kChromeAppHostGuid); 153 kChromeAppHostGuid);
153 #endif 154 #endif
154 } 155 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698