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

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

Issue 1321143002: Make GetCommandExecuteImplClsid return the CLSID as a string directly. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@a1_no_delegate_execute_win10
Patch Set: review:grt Created 5 years, 3 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 specific implementation of BrowserDistribution class for 5 // This file defines specific implementation of BrowserDistribution class for
6 // Google Chrome. 6 // Google Chrome.
7 7
8 #include "chrome/installer/util/google_chrome_distribution.h" 8 #include "chrome/installer/util/google_chrome_distribution.h"
9 9
10 #include <windows.h> 10 #include <windows.h>
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 229
230 base::string16 GoogleChromeDistribution::GetUninstallRegPath() { 230 base::string16 GoogleChromeDistribution::GetUninstallRegPath() {
231 return L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\" 231 return L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\"
232 L"Google Chrome"; 232 L"Google Chrome";
233 } 233 }
234 234
235 base::string16 GoogleChromeDistribution::GetIconFilename() { 235 base::string16 GoogleChromeDistribution::GetIconFilename() {
236 return installer::kChromeExe; 236 return installer::kChromeExe;
237 } 237 }
238 238
239 bool GoogleChromeDistribution::GetCommandExecuteImplClsid( 239 base::string16 GoogleChromeDistribution::GetCommandExecuteImplClsid() {
240 base::string16* handler_class_uuid) { 240 return kCommandExecuteImplUuid;
241 if (handler_class_uuid)
242 *handler_class_uuid = kCommandExecuteImplUuid;
243 return true;
244 } 241 }
245 242
246 // This method checks if we need to change "ap" key in Google Update to try 243 // This method checks if we need to change "ap" key in Google Update to try
247 // full installer as fall back method in case incremental installer fails. 244 // full installer as fall back method in case incremental installer fails.
248 // - If incremental installer fails we append a magic string ("-full"), if 245 // - If incremental installer fails we append a magic string ("-full"), if
249 // it is not present already, so that Google Update server next time will send 246 // it is not present already, so that Google Update server next time will send
250 // full installer to update Chrome on the local machine 247 // full installer to update Chrome on the local machine
251 // - If we are currently running full installer, we remove this magic 248 // - If we are currently running full installer, we remove this magic
252 // string (if it is present) regardless of whether installer failed or not. 249 // string (if it is present) regardless of whether installer failed or not.
253 // There is no fall-back for full installer :) 250 // There is no fall-back for full installer :)
254 void GoogleChromeDistribution::UpdateInstallStatus(bool system_install, 251 void GoogleChromeDistribution::UpdateInstallStatus(bool system_install,
255 installer::ArchiveType archive_type, 252 installer::ArchiveType archive_type,
256 installer::InstallStatus install_status) { 253 installer::InstallStatus install_status) {
257 GoogleUpdateSettings::UpdateInstallStatus(system_install, 254 GoogleUpdateSettings::UpdateInstallStatus(system_install,
258 archive_type, InstallUtil::GetInstallReturnCode(install_status), 255 archive_type, InstallUtil::GetInstallReturnCode(install_status),
259 GetAppGuid()); 256 GetAppGuid());
260 } 257 }
261 258
262 bool GoogleChromeDistribution::ShouldSetExperimentLabels() { 259 bool GoogleChromeDistribution::ShouldSetExperimentLabels() {
263 return true; 260 return true;
264 } 261 }
265 262
266 bool GoogleChromeDistribution::HasUserExperiments() { 263 bool GoogleChromeDistribution::HasUserExperiments() {
267 return true; 264 return true;
268 } 265 }
OLDNEW
« no previous file with comments | « chrome/installer/util/google_chrome_distribution.h ('k') | chrome/installer/util/google_chrome_distribution_dummy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698