Descriptionchromeos: Replace ModifyCacheState with a number of small functions to simplify DriveCache
ModifyCacheState was doing a number of things at the same time:
- Move or copy a file. (and copy was only used by Store())
- If the symlink_path is not empty, create or delete symlink
That says, ModifyCacheState is unnecessarily complicated so that it can be dividend into four small functions, MoveFile, CopyFile, CreateSymlink and DeleteSymlink.
BUG=None
TEST=unit_tests --gtest_filter="Drive*"
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166890
Patch Set 1 : _ #
Total comments: 6
Patch Set 2 : Address comments #
Total comments: 2
Patch Set 3 : Remove unnecessary 'else' #Messages
Total messages: 7 (0 generated)
|