RESOLVED FIXED 79437
Move FileSystem-related APIs from DOMWindow.idl to DOMWindowFileSystem.idl 
https://bugs.webkit.org/show_bug.cgi?id=79437
Summary Move FileSystem-related APIs from DOMWindow.idl to DOMWindowFileSystem.idl
Kentaro Hara
Reported 2012-02-23 21:18:56 PST
For WebKit modularization, we can move FileSystem-related APIs from DOMWindow.idl to DOMWindowFileSystem.idl.
Attachments
Patch (3.16 KB, patch)
2012-02-23 21:22 PST, Kentaro Hara 		abarth: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-02-23 21:22:47 PST
Created attachment 128645 [details] Patch
Adam Barth
Comment 2 2012-02-23 23:34:13 PST
Comment on attachment 128645 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=128645&action=review > Source/WebCore/fileapi/DOMWindowFileSystem.idl:35 > + attribute FileConstructor File; > + attribute FileListConstructor FileList; > + attribute BlobConstructor Blob; > + attribute [Conditional=BLOB] FileErrorConstructor FileError; > + attribute [Conditional=BLOB] FileReaderConstructor FileReader; > + attribute [Conditional=BLOB] WebKitBlobBuilderConstructor WebKitBlobBuilder; > + attribute [Conditional=BLOB] DOMURLConstructor webkitURL; I'm not sure this is quite right. I'm slightly unclear which of these APIs are part of FileSystem versus Blob or FileReader / FileWriter: http://dev.w3.org/2006/webapi/FileAPI/ http://www.w3.org/TR/file-system-api/ We might need to ask EricU for some guidance.
Eric U.
Comment 3 2012-02-27 09:42:24 PST
> > + attribute FileConstructor File; > > + attribute FileListConstructor FileList; > > + attribute BlobConstructor Blob; > > + attribute [Conditional=BLOB] FileErrorConstructor FileError; > > + attribute [Conditional=BLOB] FileReaderConstructor FileReader; > > + attribute [Conditional=BLOB] WebKitBlobBuilderConstructor WebKitBlobBuilder; > > + attribute [Conditional=BLOB] DOMURLConstructor webkitURL; > > I'm not sure this is quite right. I'm slightly unclear which of these APIs are part of FileSystem versus Blob or FileReader / FileWriter: > > http://dev.w3.org/2006/webapi/FileAPI/ > http://www.w3.org/TR/file-system-api/ > > We might need to ask EricU for some guidance. None of those are from FileSystem. BlobBuilder is from FileWriter [http://www.w3.org/TR/file-writer-api/], and the rest are from the File API. They can all be moved out of DOMWIndow, but shouldn't go into anything with the name FileSystem on it. As for BlobBuilder, I'm not sure; it's going to go away at some point soon anyway, as it's to be replaced by a constructable Blob. So it can go wherever's convenient, and don't stress about it too much.
Adam Barth
Comment 4 2012-02-27 11:12:41 PST
Comment on attachment 128645 [details] Patch The file-related APIs are spread out over a couple of different specs. We should figure out which APIs go with which specs and see if the code can be organized along those lines too.
Adam Barth
Comment 5 2012-03-23 17:29:36 PDT
Mark took care of these, I believe.
Note You need to log in before you can comment on or make changes to this bug.