For WebKit modularization, we can move FileSystem-related APIs from DOMWindow.idl to DOMWindowFileSystem.idl.
Created attachment 128645 [details] Patch
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.
> > + 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.
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.
Mark took care of these, I believe.