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

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

Issue 15255004: Refactor of BrowserDistribution. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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"
(...skipping 20 matching lines...) Expand all
31 } 31 }
32 32
33 string16 ChromeAppHostDistribution::GetAppGuid() { 33 string16 ChromeAppHostDistribution::GetAppGuid() {
34 return kChromeAppHostGuid; 34 return kChromeAppHostGuid;
35 } 35 }
36 36
37 string16 ChromeAppHostDistribution::GetBaseAppName() { 37 string16 ChromeAppHostDistribution::GetBaseAppName() {
38 return L"Google Chrome App Launcher"; 38 return L"Google Chrome App Launcher";
39 } 39 }
40 40
41 string16 ChromeAppHostDistribution::GetAppShortCutName() { 41 string16 ChromeAppHostDistribution::GetDisplayName() {
42 const string16& product_name = 42 return GetShortcutName(SHORTCUT_APP_LAUNCHER);
43 installer::GetLocalizedString(IDS_PRODUCT_APP_LAUNCHER_NAME_BASE);
44 return product_name;
45 } 43 }
46 44
47 string16 ChromeAppHostDistribution::GetAlternateApplicationName() { 45 string16 ChromeAppHostDistribution::GetShortcutName(
48 const string16& product_name = 46 ShortcutType shortcut_type) {
49 installer::GetLocalizedString(IDS_PRODUCT_APP_LAUNCHER_NAME_BASE); 47 DCHECK_EQ(shortcut_type, SHORTCUT_APP_LAUNCHER);
50 return product_name; 48 return installer::GetLocalizedString(IDS_PRODUCT_APP_LAUNCHER_NAME_BASE);
51 } 49 }
52 50
53 string16 ChromeAppHostDistribution::GetBaseAppId() { 51 string16 ChromeAppHostDistribution::GetBaseAppId() {
54 // Should be same as AppListController::GetAppModelId(). 52 // Should be same as AppListController::GetAppModelId().
55 return L"ChromeAppList"; 53 return L"ChromeAppList";
56 } 54 }
57 55
58 string16 ChromeAppHostDistribution::GetInstallSubDir() { 56 string16 ChromeAppHostDistribution::GetInstallSubDir() {
59 return BrowserDistribution::GetSpecificDistribution( 57 return BrowserDistribution::GetSpecificDistribution(
60 BrowserDistribution::CHROME_BINARIES)->GetInstallSubDir(); 58 BrowserDistribution::CHROME_BINARIES)->GetInstallSubDir();
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 139
142 void ChromeAppHostDistribution::UpdateInstallStatus(bool system_install, 140 void ChromeAppHostDistribution::UpdateInstallStatus(bool system_install,
143 installer::ArchiveType archive_type, 141 installer::ArchiveType archive_type,
144 installer::InstallStatus install_status) { 142 installer::InstallStatus install_status) {
145 #if defined(GOOGLE_CHROME_BUILD) 143 #if defined(GOOGLE_CHROME_BUILD)
146 GoogleUpdateSettings::UpdateInstallStatus(system_install, 144 GoogleUpdateSettings::UpdateInstallStatus(system_install,
147 archive_type, InstallUtil::GetInstallReturnCode(install_status), 145 archive_type, InstallUtil::GetInstallReturnCode(install_status),
148 kChromeAppHostGuid); 146 kChromeAppHostGuid);
149 #endif 147 #endif
150 } 148 }
OLDNEW
« no previous file with comments | « chrome/installer/util/chrome_app_host_distribution.h ('k') | chrome/installer/util/chrome_app_host_operations.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698