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

Unified Diff: components/filesystem/public/interfaces/directory.mojom

Issue 2539383002: Replace base::File wrapping with typemapping. (Closed)
Patch Set: Created 4 years 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 side-by-side diff with in-line comments
Download patch
Index: components/filesystem/public/interfaces/directory.mojom
diff --git a/components/filesystem/public/interfaces/directory.mojom b/components/filesystem/public/interfaces/directory.mojom
index 104a62f55283afb4c8380a91cf7767420fa5028f..a4d330a0c037db8a69fa4e3594b95bd60269e1aa 100644
--- a/components/filesystem/public/interfaces/directory.mojom
+++ b/components/filesystem/public/interfaces/directory.mojom
@@ -6,6 +6,7 @@ module filesystem.mojom;
import "components/filesystem/public/interfaces/file.mojom";
import "components/filesystem/public/interfaces/types.mojom";
+import "mojo/common/common_custom_types.mojom";
struct FileOpenDetails {
string path;
@@ -15,7 +16,7 @@ struct FileOpenDetails {
struct FileOpenResult {
string path;
FileError error;
- handle file_handle;
+ mojo.common.mojom.File? file_handle;
};
// This interface provides access to a directory in a "file system", providing
@@ -46,7 +47,7 @@ interface Directory {
// native file descriptor wrapped in a MojoHandle.
[Sync]
OpenFileHandle(string path, uint32 open_flags)
- => (FileError error, handle file_handle);
+ => (FileError error, mojo.common.mojom.File? file_handle);
// Like OpenFileHandle, but opens multiple files.
[Sync]

Powered by Google App Engine
This is Rietveld 408576698