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

Side by Side Diff: storage/browser/fileapi/file_system_operation.h

Issue 2390323003: Document timestamp caveat in FileSystem API move operation. (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #ifndef STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_ 5 #ifndef STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_
6 #define STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_ 6 #define STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 279 matching lines...) Expand 10 before | Expand all | Expand 10 after
290 // comment for details. 290 // comment for details.
291 // 291 //
292 // For recursive case this internally creates new FileSystemOperations and 292 // For recursive case this internally creates new FileSystemOperations and
293 // calls: 293 // calls:
294 // - ReadDirectory, MoveFileLocal, CreateDirectory and Remove 294 // - ReadDirectory, MoveFileLocal, CreateDirectory and Remove
295 // for same-filesystem case, or 295 // for same-filesystem case, or
296 // - ReadDirectory, CreateSnapshotFile and Remove on source filesystem and 296 // - ReadDirectory, CreateSnapshotFile and Remove on source filesystem and
297 // CopyInForeignFile and CreateDirectory on dest filesystem 297 // CopyInForeignFile and CreateDirectory on dest filesystem
298 // for cross-filesystem case. 298 // for cross-filesystem case.
299 // 299 //
300 // TODO(crbug.com/171284): Restore directory timestamps after the Move
301 // operation.
300 virtual void Move(const FileSystemURL& src_path, 302 virtual void Move(const FileSystemURL& src_path,
301 const FileSystemURL& dest_path, 303 const FileSystemURL& dest_path,
302 CopyOrMoveOption option, 304 CopyOrMoveOption option,
303 const StatusCallback& callback) = 0; 305 const StatusCallback& callback) = 0;
304 306
305 // Checks if a directory is present at |path|. 307 // Checks if a directory is present at |path|.
306 virtual void DirectoryExists(const FileSystemURL& path, 308 virtual void DirectoryExists(const FileSystemURL& path,
307 const StatusCallback& callback) = 0; 309 const StatusCallback& callback) = 0;
308 310
309 // Checks if a file is present at |path|. 311 // Checks if a file is present at |path|.
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 kOperationOpenFile, 500 kOperationOpenFile,
499 kOperationCloseFile, 501 kOperationCloseFile,
500 kOperationGetLocalPath, 502 kOperationGetLocalPath,
501 kOperationCancel, 503 kOperationCancel,
502 }; 504 };
503 }; 505 };
504 506
505 } // namespace storage 507 } // namespace storage
506 508
507 #endif // STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_ 509 #endif // STORAGE_BROWSER_FILEAPI_FILE_SYSTEM_OPERATION_H_
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698