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

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

Issue 15255004: Refactor of BrowserDistribution. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rework Created 7 years, 5 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 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 18 matching lines...) Expand all
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::GetBaseAppName() { 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::GetShortcutName(ShortcutType shortcut_type) {
40 const string16& product_name = 40 switch (shortcut_type) {
41 installer::GetLocalizedString(IDS_PRODUCT_FRAME_NAME_BASE); 41 case SHORTCUT_CHROME:
42 return product_name; 42 case SHORTCUT_ALTERNATE_CHROME:
43 } 43 return installer::GetLocalizedString(IDS_PRODUCT_FRAME_NAME_BASE);
44 44 default:
45 string16 ChromeFrameDistribution::GetAlternateApplicationName() { 45 NOTREACHED();
46 const string16& product_name = 46 return string16();
47 installer::GetLocalizedString(IDS_PRODUCT_FRAME_NAME_BASE); 47 }
48 return product_name;
49 } 48 }
50 49
51 string16 ChromeFrameDistribution::GetInstallSubDir() { 50 string16 ChromeFrameDistribution::GetInstallSubDir() {
52 return L"Google\\Chrome Frame"; 51 return L"Google\\Chrome Frame";
53 } 52 }
54 53
55 string16 ChromeFrameDistribution::GetPublisherName() { 54 string16 ChromeFrameDistribution::GetPublisherName() {
56 const string16& publisher_name = 55 const string16& publisher_name =
57 installer::GetLocalizedString(IDS_ABOUT_VERSION_COMPANY_NAME_BASE); 56 installer::GetLocalizedString(IDS_ABOUT_VERSION_COMPANY_NAME_BASE);
58 return publisher_name; 57 return publisher_name;
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 string16 key(google_update::kRegPathClients); 104 string16 key(google_update::kRegPathClients);
106 key.append(L"\\"); 105 key.append(L"\\");
107 key.append(kChromeFrameGuid); 106 key.append(kChromeFrameGuid);
108 return key; 107 return key;
109 } 108 }
110 109
111 string16 ChromeFrameDistribution::GetIconFilename() { 110 string16 ChromeFrameDistribution::GetIconFilename() {
112 return installer::kChromeExe; 111 return installer::kChromeExe;
113 } 112 }
114 113
115 int ChromeFrameDistribution::GetIconIndex() { 114 int ChromeFrameDistribution::GetIconIndex(ShortcutType shortcut_type) {
116 return 0; 115 switch (shortcut_type) {
116 case SHORTCUT_CHROME:
117 return 0;
118 default:
119 NOTREACHED();
120 return 0;
121 }
117 } 122 }
118 123
119 bool ChromeFrameDistribution::CanSetAsDefault() { 124 bool ChromeFrameDistribution::CanSetAsDefault() {
120 return false; 125 return false;
121 } 126 }
122 127
123 bool ChromeFrameDistribution::CanCreateDesktopShortcuts() { 128 bool ChromeFrameDistribution::CanCreateDesktopShortcuts() {
124 return false; 129 return false;
125 } 130 }
126 131
127 bool ChromeFrameDistribution::GetCommandExecuteImplClsid( 132 bool ChromeFrameDistribution::GetCommandExecuteImplClsid(
128 string16* handler_class_uuid) { 133 string16* handler_class_uuid) {
129 return false; 134 return false;
130 } 135 }
131 136
132 void ChromeFrameDistribution::UpdateInstallStatus(bool system_install, 137 void ChromeFrameDistribution::UpdateInstallStatus(bool system_install,
133 installer::ArchiveType archive_type, 138 installer::ArchiveType archive_type,
134 installer::InstallStatus install_status) { 139 installer::InstallStatus install_status) {
135 #if defined(GOOGLE_CHROME_BUILD) 140 #if defined(GOOGLE_CHROME_BUILD)
136 GoogleUpdateSettings::UpdateInstallStatus(system_install, 141 GoogleUpdateSettings::UpdateInstallStatus(system_install,
137 archive_type, InstallUtil::GetInstallReturnCode(install_status), 142 archive_type, InstallUtil::GetInstallReturnCode(install_status),
138 kChromeFrameGuid); 143 kChromeFrameGuid);
139 #endif 144 #endif
140 } 145 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698