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

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

Issue 12321061: Pulling user experiment code from BrowserDistribution to a new class. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Adding GetUserDataPaths() to BrowserDistribution; replacing Product with other params in LaunchUser… Created 7 years, 9 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 class that contains various method related to branding. 5 // This file defines a class that contains various method related to branding.
6 // It provides only default implementations of these methods. Usually to add 6 // It provides only default implementations of these methods. Usually to add
7 // specific branding, we will need to extend this class with a custom 7 // specific branding, we will need to extend this class with a custom
8 // implementation. 8 // implementation.
9 9
10 #include "chrome/installer/util/browser_distribution.h" 10 #include "chrome/installer/util/browser_distribution.h"
11 11
12 #include "base/atomicops.h" 12 #include "base/atomicops.h"
13 #include "base/command_line.h" 13 #include "base/command_line.h"
14 #include "base/files/file_path.h" 14 #include "base/files/file_path.h"
15 #include "base/logging.h" 15 #include "base/logging.h"
16 #include "base/path_service.h" 16 #include "base/path_service.h"
17 #include "base/win/registry.h" 17 #include "base/win/registry.h"
18 #include "base/win/windows_version.h" 18 #include "base/win/windows_version.h"
19 #include "chrome/common/env_vars.h" 19 #include "chrome/common/env_vars.h"
20 #include "chrome/installer/util/chrome_app_host_distribution.h" 20 #include "chrome/installer/util/chrome_app_host_distribution.h"
21 #include "chrome/installer/util/chrome_frame_distribution.h" 21 #include "chrome/installer/util/chrome_frame_distribution.h"
22 #include "chrome/installer/util/chromium_binaries_distribution.h" 22 #include "chrome/installer/util/chromium_binaries_distribution.h"
23 #include "chrome/installer/util/google_chrome_binaries_distribution.h" 23 #include "chrome/installer/util/google_chrome_binaries_distribution.h"
24 #include "chrome/installer/util/google_chrome_distribution.h" 24 #include "chrome/installer/util/google_chrome_distribution.h"
25 #include "chrome/installer/util/google_chrome_sxs_distribution.h" 25 #include "chrome/installer/util/google_chrome_sxs_distribution.h"
26 #include "chrome/installer/util/helper.h"
26 #include "chrome/installer/util/install_util.h" 27 #include "chrome/installer/util/install_util.h"
27 #include "chrome/installer/util/l10n_string_util.h" 28 #include "chrome/installer/util/l10n_string_util.h"
28 #include "chrome/installer/util/master_preferences.h" 29 #include "chrome/installer/util/master_preferences.h"
29 30
30 #include "installer_util_strings.h" // NOLINT 31 #include "installer_util_strings.h" // NOLINT
31 32
32 using installer::MasterPreferences; 33 using installer::MasterPreferences;
33 34
34 namespace { 35 namespace {
35 36
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 } 214 }
214 215
215 string16 BrowserDistribution::GetUninstallLinkName() { 216 string16 BrowserDistribution::GetUninstallLinkName() {
216 return L"Uninstall Chromium"; 217 return L"Uninstall Chromium";
217 } 218 }
218 219
219 string16 BrowserDistribution::GetUninstallRegPath() { 220 string16 BrowserDistribution::GetUninstallRegPath() {
220 return L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\Chromium"; 221 return L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\Chromium";
221 } 222 }
222 223
224 void BrowserDistribution::GetUserDataPaths(std::vector<base::FilePath>* paths) {
225 installer::GetChromeUserDataPaths(this, paths);
226 }
227
223 string16 BrowserDistribution::GetVersionKey() { 228 string16 BrowserDistribution::GetVersionKey() {
224 return L"Software\\Chromium"; 229 return L"Software\\Chromium";
225 } 230 }
226 231
227 bool BrowserDistribution::CanSetAsDefault() { 232 bool BrowserDistribution::CanSetAsDefault() {
228 return true; 233 return true;
229 } 234 }
230 235
231 bool BrowserDistribution::CanCreateDesktopShortcuts() { 236 bool BrowserDistribution::CanCreateDesktopShortcuts() {
232 return true; 237 return true;
(...skipping 22 matching lines...) Expand all
255 260
256 bool BrowserDistribution::AppHostIsSupported() { 261 bool BrowserDistribution::AppHostIsSupported() {
257 return false; 262 return false;
258 } 263 }
259 264
260 void BrowserDistribution::UpdateInstallStatus(bool system_install, 265 void BrowserDistribution::UpdateInstallStatus(bool system_install,
261 installer::ArchiveType archive_type, 266 installer::ArchiveType archive_type,
262 installer::InstallStatus install_status) { 267 installer::InstallStatus install_status) {
263 } 268 }
264 269
265 bool BrowserDistribution::GetExperimentDetails(
266 UserExperiment* experiment, int flavor) {
267 return false;
268 }
269
270 void BrowserDistribution::LaunchUserExperiment(
271 const base::FilePath& setup_path, installer::InstallStatus status,
272 const Version& version, const installer::Product& product,
273 bool system_level) {
274 }
275
276 bool BrowserDistribution::ShouldSetExperimentLabels() { 270 bool BrowserDistribution::ShouldSetExperimentLabels() {
277 return false; 271 return false;
278 } 272 }
279 273
280 void BrowserDistribution::InactiveUserToastExperiment(int flavor, 274 bool BrowserDistribution::HasUserExperiments() const {
281 const string16& experiment_group, 275 return false;
282 const installer::Product& installation,
283 const base::FilePath& application_path) {
284 } 276 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698