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_frame_distribution.cc

Issue 10446111: Abstract suffixing logic away from GetApplicationName (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase on suffix@r142211 Created 8 years, 6 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 | Annotate | Revision Log
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 Frame. It overrides the bare minimum of methods necessary to get a 6 // Chrome Frame. It overrides the bare minimum of methods necessary to get a
7 // Chrome Frame installer that does not interact with Google Chrome or 7 // Chrome Frame installer that does not interact with Google Chrome or
8 // Chromium installations. 8 // Chromium installations.
9 9
10 #include "chrome/installer/util/chrome_frame_distribution.h" 10 #include "chrome/installer/util/chrome_frame_distribution.h"
(...skipping 14 matching lines...) Expand all
25 } 25 }
26 26
27 ChromeFrameDistribution::ChromeFrameDistribution() 27 ChromeFrameDistribution::ChromeFrameDistribution()
28 : BrowserDistribution(CHROME_FRAME) { 28 : BrowserDistribution(CHROME_FRAME) {
29 } 29 }
30 30
31 string16 ChromeFrameDistribution::GetAppGuid() { 31 string16 ChromeFrameDistribution::GetAppGuid() {
32 return kChromeFrameGuid; 32 return kChromeFrameGuid;
33 } 33 }
34 34
35 string16 ChromeFrameDistribution::GetApplicationName() { 35 string16 ChromeFrameDistribution::GetBaseAppName() {
36 return L"Google Chrome Frame"; 36 return L"Google Chrome Frame";
37 } 37 }
38 38
39 string16 ChromeFrameDistribution::GetAppShortCutName() { 39 string16 ChromeFrameDistribution::GetAppShortCutName() {
40 const string16& product_name = 40 const string16& product_name =
41 installer::GetLocalizedString(IDS_PRODUCT_FRAME_NAME_BASE); 41 installer::GetLocalizedString(IDS_PRODUCT_FRAME_NAME_BASE);
42 return product_name; 42 return product_name;
43 } 43 }
44 44
45 string16 ChromeFrameDistribution::GetAlternateApplicationName() { 45 string16 ChromeFrameDistribution::GetAlternateApplicationName() {
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 130
131 void ChromeFrameDistribution::UpdateInstallStatus(bool system_install, 131 void ChromeFrameDistribution::UpdateInstallStatus(bool system_install,
132 installer::ArchiveType archive_type, 132 installer::ArchiveType archive_type,
133 installer::InstallStatus install_status) { 133 installer::InstallStatus install_status) {
134 #if defined(GOOGLE_CHROME_BUILD) 134 #if defined(GOOGLE_CHROME_BUILD)
135 GoogleUpdateSettings::UpdateInstallStatus(system_install, 135 GoogleUpdateSettings::UpdateInstallStatus(system_install,
136 archive_type, InstallUtil::GetInstallReturnCode(install_status), 136 archive_type, InstallUtil::GetInstallReturnCode(install_status),
137 kChromeFrameGuid); 137 kChromeFrameGuid);
138 #endif 138 #endif
139 } 139 }
OLDNEW
« no previous file with comments | « chrome/installer/util/chrome_frame_distribution.h ('k') | chrome/installer/util/chromium_binaries_distribution.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698