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

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

Issue 2764963002: Move ProgID methods from BrowserDistribution into install_static. (Closed)
Patch Set: huangs comments Created 3 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"
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 Subfolder subfolder_type) { 137 Subfolder subfolder_type) {
138 switch (subfolder_type) { 138 switch (subfolder_type) {
139 case SUBFOLDER_APPS: 139 case SUBFOLDER_APPS:
140 return installer::GetLocalizedString(IDS_APP_SHORTCUTS_SUBDIR_NAME_BASE); 140 return installer::GetLocalizedString(IDS_APP_SHORTCUTS_SUBDIR_NAME_BASE);
141 default: 141 default:
142 DCHECK_EQ(SUBFOLDER_CHROME, subfolder_type); 142 DCHECK_EQ(SUBFOLDER_CHROME, subfolder_type);
143 return GetShortcutName(); 143 return GetShortcutName();
144 } 144 }
145 } 145 }
146 146
147 base::string16 BrowserDistribution::GetBrowserProgIdPrefix() {
148 // This used to be "ChromiumHTML", but was forced to become "ChromiumHTM"
149 // because of http://crbug.com/153349. See the declaration of this function
150 // in the header file for more details.
151 return L"ChromiumHTM";
152 }
153
154 base::string16 BrowserDistribution::GetBrowserProgIdDesc() {
155 return L"Chromium HTML Document";
156 }
157
158
159 base::string16 BrowserDistribution::GetPublisherName() { 147 base::string16 BrowserDistribution::GetPublisherName() {
160 return L"Chromium"; 148 return L"Chromium";
161 } 149 }
162 150
163 base::string16 BrowserDistribution::GetAppDescription() { 151 base::string16 BrowserDistribution::GetAppDescription() {
164 return L"Browse the web"; 152 return L"Browse the web";
165 } 153 }
166 154
167 base::string16 BrowserDistribution::GetLongAppDescription() { 155 base::string16 BrowserDistribution::GetLongAppDescription() {
168 const base::string16& app_description = 156 const base::string16& app_description =
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 installer::InstallStatus install_status) { 188 installer::InstallStatus install_status) {
201 } 189 }
202 190
203 bool BrowserDistribution::ShouldSetExperimentLabels() { 191 bool BrowserDistribution::ShouldSetExperimentLabels() {
204 return false; 192 return false;
205 } 193 }
206 194
207 bool BrowserDistribution::HasUserExperiments() { 195 bool BrowserDistribution::HasUserExperiments() {
208 return false; 196 return false;
209 } 197 }
OLDNEW
« no previous file with comments | « chrome/installer/util/browser_distribution.h ('k') | chrome/installer/util/google_chrome_distribution.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698