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

Side by Side Diff: chrome/installer/setup/uninstall.cc

Issue 13864015: Move app launcher and chrome apps shortcut strings into the installer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@migrate_app_id_fix
Patch Set: rebase, move app launcher and chrome app strings into the installer Created 7 years, 7 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 the methods useful for uninstalling Chrome. 5 // This file defines the methods useful for uninstalling Chrome.
6 6
7 #include "chrome/installer/setup/uninstall.h" 7 #include "chrome/installer/setup/uninstall.h"
8 8
9 #include <windows.h> 9 #include <windows.h>
10 10
(...skipping 335 matching lines...) Expand 10 before | Expand all | Expand 10 after
346 LOG(WARNING) << "Failed to delete Desktop shortcuts."; 346 LOG(WARNING) << "Failed to delete Desktop shortcuts.";
347 } 347 }
348 348
349 VLOG(1) << "Deleting Quick Launch shortcuts."; 349 VLOG(1) << "Deleting Quick Launch shortcuts.";
350 if (!ShellUtil::RemoveShortcuts(ShellUtil::SHORTCUT_LOCATION_QUICK_LAUNCH, 350 if (!ShellUtil::RemoveShortcuts(ShellUtil::SHORTCUT_LOCATION_QUICK_LAUNCH,
351 dist, install_level, target_exe)) { 351 dist, install_level, target_exe)) {
352 LOG(WARNING) << "Failed to delete Quick Launch shortcuts."; 352 LOG(WARNING) << "Failed to delete Quick Launch shortcuts.";
353 } 353 }
354 354
355 VLOG(1) << "Deleting Start Menu shortcuts."; 355 VLOG(1) << "Deleting Start Menu shortcuts.";
356 if (!ShellUtil::RemoveShortcuts(ShellUtil::SHORTCUT_LOCATION_START_MENU, dist, 356 if (!ShellUtil::RemoveShortcuts(
huangs 2013/05/15 02:33:46 How about SHORTCUT_LOCATION_START_MENU_CHROME_DIR
gab 2013/05/15 21:26:38 Yes, please add deletion of the SHORTCUT_LOCATION_
calamity 2013/08/27 07:59:35 Done.
357 install_level, target_exe)) { 357 ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_DIR, dist, install_level,
358 target_exe)) {
358 LOG(WARNING) << "Failed to delete Start Menu shortcuts."; 359 LOG(WARNING) << "Failed to delete Start Menu shortcuts.";
359 } 360 }
360 361
361 // Unpin all pinned-to-taskbar shortcuts that point to |chrome_exe|. 362 // Unpin all pinned-to-taskbar shortcuts that point to |chrome_exe|.
362 if (!ShellUtil::RemoveShortcuts(ShellUtil::SHORTCUT_LOCATION_TASKBAR_PINS, 363 if (!ShellUtil::RemoveShortcuts(ShellUtil::SHORTCUT_LOCATION_TASKBAR_PINS,
363 dist, ShellUtil::CURRENT_USER, target_exe)) { 364 dist, ShellUtil::CURRENT_USER, target_exe)) {
364 LOG(WARNING) << "Failed to unpin taskbar shortcuts at user-level."; 365 LOG(WARNING) << "Failed to unpin taskbar shortcuts at user-level.";
365 } 366 }
366 367
367 // Delete the folder of secondary tiles from the start screen for |dist|. 368 // Delete the folder of secondary tiles from the start screen for |dist|.
(...skipping 1062 matching lines...) Expand 10 before | Expand all | Expand 10 after
1430 // deletion unconditionally. If they are not empty, the session manager 1431 // deletion unconditionally. If they are not empty, the session manager
1431 // will not delete them on reboot. 1432 // will not delete them on reboot.
1432 ScheduleParentAndGrandparentForDeletion(target_path); 1433 ScheduleParentAndGrandparentForDeletion(target_path);
1433 } else if (DeleteApplicationProductAndVendorDirectories(target_path) == 1434 } else if (DeleteApplicationProductAndVendorDirectories(target_path) ==
1434 installer::DELETE_FAILED) { 1435 installer::DELETE_FAILED) {
1435 *uninstall_status = installer::UNINSTALL_FAILED; 1436 *uninstall_status = installer::UNINSTALL_FAILED;
1436 } 1437 }
1437 } 1438 }
1438 1439
1439 } // namespace installer 1440 } // namespace installer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698