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

Side by Side Diff: chrome/installer/util/move_tree_work_item.h

Issue 1109043003: Apply automated fixits for Chrome clang plugin to chrome. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #ifndef CHROME_INSTALLER_UTIL_MOVE_TREE_WORK_ITEM_H_ 5 #ifndef CHROME_INSTALLER_UTIL_MOVE_TREE_WORK_ITEM_H_
6 #define CHROME_INSTALLER_UTIL_MOVE_TREE_WORK_ITEM_H_ 6 #define CHROME_INSTALLER_UTIL_MOVE_TREE_WORK_ITEM_H_
7 7
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/files/scoped_temp_dir.h" 9 #include "base/files/scoped_temp_dir.h"
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
11 #include "chrome/installer/util/work_item.h" 11 #include "chrome/installer/util/work_item.h"
12 12
13 // A WorkItem subclass that recursively move a file system hierarchy from 13 // A WorkItem subclass that recursively move a file system hierarchy from
14 // source path to destination path. The file system hierarchy could be a 14 // source path to destination path. The file system hierarchy could be a
15 // single file, or a directory. 15 // single file, or a directory.
16 // 16 //
17 // Under the cover MoveTreeWorkItem moves the destination path, if existing, 17 // Under the cover MoveTreeWorkItem moves the destination path, if existing,
18 // to the temporary directory passed in, and then moves the source hierarchy 18 // to the temporary directory passed in, and then moves the source hierarchy
19 // to the destination location. During rollback the original destination 19 // to the destination location. During rollback the original destination
20 // hierarchy is moved back. 20 // hierarchy is moved back.
21 class MoveTreeWorkItem : public WorkItem { 21 class MoveTreeWorkItem : public WorkItem {
22 public: 22 public:
23 virtual ~MoveTreeWorkItem(); 23 ~MoveTreeWorkItem() override;
24 24
25 virtual bool Do(); 25 bool Do() override;
26 26
27 virtual void Rollback(); 27 void Rollback() override;
28 28
29 private: 29 private:
30 friend class WorkItem; 30 friend class WorkItem;
31 FRIEND_TEST_ALL_PREFIXES(MoveTreeWorkItemTest, 31 FRIEND_TEST_ALL_PREFIXES(MoveTreeWorkItemTest,
32 MoveDirectoryDestExistsCheckForDuplicatesFull); 32 MoveDirectoryDestExistsCheckForDuplicatesFull);
33 FRIEND_TEST_ALL_PREFIXES(MoveTreeWorkItemTest, 33 FRIEND_TEST_ALL_PREFIXES(MoveTreeWorkItemTest,
34 MoveDirectoryDestExistsCheckForDuplicatesPartial); 34 MoveDirectoryDestExistsCheckForDuplicatesPartial);
35 35
36 // |source_path| specifies file or directory that will be moved to location 36 // |source_path| specifies file or directory that will be moved to location
37 // specified by |dest_path|. To facilitate rollback, the caller needs to 37 // specified by |dest_path|. To facilitate rollback, the caller needs to
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 // Whether we moved the source files to the backup path instead just to 69 // Whether we moved the source files to the backup path instead just to
70 // preserve the behaviour of a Move. This can only become true if 70 // preserve the behaviour of a Move. This can only become true if
71 // duplicate_option_ is CHECK_DUPLICATES. 71 // duplicate_option_ is CHECK_DUPLICATES.
72 bool source_moved_to_backup_; 72 bool source_moved_to_backup_;
73 73
74 // Whether to check for duplicates before moving. 74 // Whether to check for duplicates before moving.
75 MoveTreeOption duplicate_option_; 75 MoveTreeOption duplicate_option_;
76 }; 76 };
77 77
78 #endif // CHROME_INSTALLER_UTIL_MOVE_TREE_WORK_ITEM_H_ 78 #endif // CHROME_INSTALLER_UTIL_MOVE_TREE_WORK_ITEM_H_
OLDNEW
« no previous file with comments | « chrome/installer/util/master_preferences_unittest.cc ('k') | chrome/installer/util/move_tree_work_item_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698