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

Side by Side Diff: components/filesystem/public/interfaces/file.mojom

Issue 1839823002: mojo leveldb: Remove the created file thread. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: And cut out manual Signal()ing most places. Created 4 years, 8 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 | « components/filesystem/public/interfaces/directory.mojom ('k') | components/leveldb/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 // TODO(vtl): notes to self: 5 // TODO(vtl): notes to self:
6 // - file offsets, file positions, and file sizes are int64 (though positions 6 // - file offsets, file positions, and file sizes are int64 (though positions
7 // and sizes must always be non-negative) 7 // and sizes must always be non-negative)
8 // - buffer size parameters (for read/write) are uint32 8 // - buffer size parameters (for read/write) are uint32
9 9
10 module filesystem; 10 module filesystem;
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 // I.e., the access mode, etc. (as specified to |Directory::OpenFile()| by the 58 // I.e., the access mode, etc. (as specified to |Directory::OpenFile()| by the
59 // |open_flags| argument) as well as file position. 59 // |open_flags| argument) as well as file position.
60 Dup(File& file) => (FileError error); 60 Dup(File& file) => (FileError error);
61 61
62 // Syncs data to disk. 62 // Syncs data to disk.
63 Flush() => (FileError error); 63 Flush() => (FileError error);
64 64
65 // Attempts to take an exclusive write lock on the file. This both takes a 65 // Attempts to take an exclusive write lock on the file. This both takes a
66 // native OS file lock (so that non-chrome, non-mojo processes can't write to 66 // native OS file lock (so that non-chrome, non-mojo processes can't write to
67 // this file). 67 // this file).
68 [Sync]
68 Lock() => (FileError error); 69 Lock() => (FileError error);
69 70
70 // Unlocks a previously locked file. 71 // Unlocks a previously locked file.
72 [Sync]
71 Unlock() => (FileError error); 73 Unlock() => (FileError error);
72 74
73 // Returns a handle to the file for raw access. 75 // Returns a handle to the file for raw access.
74 AsHandle() => (FileError error, handle file_handle); 76 AsHandle() => (FileError error, handle file_handle);
75 }; 77 };
OLDNEW
« no previous file with comments | « components/filesystem/public/interfaces/directory.mojom ('k') | components/leveldb/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698