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

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

Issue 10855002: Change the type of file_type parameter to int, as the parameter actually takes or-ed bitmasks, (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 4 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 451 matching lines...) Expand 10 before | Expand all | Expand 10 after
462 } 462 }
463 463
464 DeleteInstallTempDir(target_path); 464 DeleteInstallTempDir(target_path);
465 465
466 DeleteResult result = DELETE_SUCCEEDED; 466 DeleteResult result = DELETE_SUCCEEDED;
467 467
468 using file_util::FileEnumerator; 468 using file_util::FileEnumerator;
469 FileEnumerator file_enumerator( 469 FileEnumerator file_enumerator(
470 target_path, 470 target_path,
471 false, 471 false,
472 static_cast<FileEnumerator::FileType>(FileEnumerator::FILES | 472 (FileEnumerator::FILES | FileEnumerator::DIRECTORIES));
jar (doing other things) 2012/08/06 18:27:02 nit: remove parens around arg 3; Consider pushing
Haruki Sato 2012/08/06 23:22:18 Done. Thanks.
473 FileEnumerator::DIRECTORIES));
474 while (true) { 473 while (true) {
475 FilePath to_delete(file_enumerator.Next()); 474 FilePath to_delete(file_enumerator.Next());
476 if (to_delete.empty()) 475 if (to_delete.empty())
477 break; 476 break;
478 if (to_delete.BaseName().value() == installer::kChromeAppHostExe) 477 if (to_delete.BaseName().value() == installer::kChromeAppHostExe)
479 continue; 478 continue;
480 479
481 VLOG(1) << "Deleting install path " << target_path.value(); 480 VLOG(1) << "Deleting install path " << target_path.value();
482 if (!file_util::Delete(to_delete, true)) { 481 if (!file_util::Delete(to_delete, true)) {
483 LOG(ERROR) << "Failed to delete path (1st try): " << to_delete.value(); 482 LOG(ERROR) << "Failed to delete path (1st try): " << to_delete.value();
(...skipping 663 matching lines...) Expand 10 before | Expand all | Expand 10 after
1147 1146
1148 // Try and delete the preserved local state once the post-install 1147 // Try and delete the preserved local state once the post-install
1149 // operations are complete. 1148 // operations are complete.
1150 if (!backup_state_file.empty()) 1149 if (!backup_state_file.empty())
1151 file_util::Delete(backup_state_file, false); 1150 file_util::Delete(backup_state_file, false);
1152 1151
1153 return ret; 1152 return ret;
1154 } 1153 }
1155 1154
1156 } // namespace installer 1155 } // namespace installer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698