Move FileSystem related files from WebCore/html to WebCore/fileapi.
(In reply to comment #0) > Move FileSystem related files from WebCore/html to WebCore/fileapi. Correction; Move FileSystem related files from WebCore/storage to WebCore/fileapi.
Created attachment 66007 [details] Patch
Comment on attachment 66007 [details] Patch > WebCore/CMakeLists.txt:1593 > +if (ENABLE_FILE_SYSTEM) > + LIST(APPEND WebCore_SOURCES > + fileapi/DirectoryEntry.cpp > + fileapi/DirectoryReader.cpp > + fileapi/DOMFilePath.cpp > + fileapi/DOMFileSystem.cpp > + fileapi/Entry.cpp > + fileapi/EntryArray.cpp > + fileapi/FileEntry.cpp > + fileapi/FileSystemCallbacks.cpp > + fileapi/LocalFileSystem.cpp > + platform/AsyncFileSystem.cpp > + ) > +ENDIF () These weren't conditional before. Also, what about the existing files in fileapi? Should those be conditional too? > WebCore/GNUmakefile.am:3052 > + WebCore/fileapi/DirectoryEntry.cpp \ > + WebCore/fileapi/DirectoryEntry.h \ > + WebCore/fileapi/DirectoryReader.cpp \ > + WebCore/fileapi/DirectoryReader.h \ > + WebCore/fileapi/DOMFilePath.cpp \ > + WebCore/fileapi/DOMFilePath.h \ > + WebCore/fileapi/DOMFileSystem.cpp \ > + WebCore/fileapi/DOMFileSystem.h \ > + WebCore/fileapi/EntriesCallback.h \ > + WebCore/fileapi/Entry.cpp \ > + WebCore/fileapi/Entry.h \ > + WebCore/fileapi/EntryArray.cpp \ > + WebCore/fileapi/EntryArray.h \ > + WebCore/fileapi/EntryCallback.h \ > + WebCore/fileapi/ErrorCallback.h \ > + WebCore/fileapi/FileEntry.cpp \ > + WebCore/fileapi/FileEntry.h \ > + WebCore/fileapi/FileSystemCallback.h \ > + WebCore/fileapi/FileSystemCallbacks.cpp \ > + WebCore/fileapi/FileSystemCallbacks.h \ > + WebCore/fileapi/Flags.h \ > + WebCore/fileapi/LocalFileSystem.cpp \ > + WebCore/fileapi/LocalFileSystem.h \ > + WebCore/fileapi/Metadata.h \ > + WebCore/fileapi/MetadataCallback.h Should these be textually next to the other fileapi files?
Thanks for reviewing. (In reply to comment #3) > (From update of attachment 66007 [details]) > > WebCore/CMakeLists.txt:1593 > > +if (ENABLE_FILE_SYSTEM) > > + LIST(APPEND WebCore_SOURCES > > + fileapi/DirectoryEntry.cpp > > + fileapi/DirectoryReader.cpp > > + fileapi/DOMFilePath.cpp > > + fileapi/DOMFileSystem.cpp > > + fileapi/Entry.cpp > > + fileapi/EntryArray.cpp > > + fileapi/FileEntry.cpp > > + fileapi/FileSystemCallbacks.cpp > > + fileapi/LocalFileSystem.cpp > > + platform/AsyncFileSystem.cpp > > + ) > > +ENDIF () > These weren't conditional before. Also, what about the existing files in fileapi? Should those be conditional too? For now we don't need to build them if FILE_SYSTEM flag is not enabled, and there're no dependencies between existing files and files moved by this patch (yet). > > WebCore/GNUmakefile.am:3052 > > + WebCore/fileapi/DirectoryEntry.cpp \ > > + WebCore/fileapi/DirectoryEntry.h \ > > + WebCore/fileapi/DirectoryReader.cpp \ > > + WebCore/fileapi/DirectoryReader.h \ > > + WebCore/fileapi/DOMFilePath.cpp \ > > + WebCore/fileapi/DOMFilePath.h \ > > + WebCore/fileapi/DOMFileSystem.cpp \ > > + WebCore/fileapi/DOMFileSystem.h \ > > + WebCore/fileapi/EntriesCallback.h \ > > + WebCore/fileapi/Entry.cpp \ > > + WebCore/fileapi/Entry.h \ > > + WebCore/fileapi/EntryArray.cpp \ > > + WebCore/fileapi/EntryArray.h \ > > + WebCore/fileapi/EntryCallback.h \ > > + WebCore/fileapi/ErrorCallback.h \ > > + WebCore/fileapi/FileEntry.cpp \ > > + WebCore/fileapi/FileEntry.h \ > > + WebCore/fileapi/FileSystemCallback.h \ > > + WebCore/fileapi/FileSystemCallbacks.cpp \ > > + WebCore/fileapi/FileSystemCallbacks.h \ > > + WebCore/fileapi/Flags.h \ > > + WebCore/fileapi/LocalFileSystem.cpp \ > > + WebCore/fileapi/LocalFileSystem.h \ > > + WebCore/fileapi/Metadata.h \ > > + WebCore/fileapi/MetadataCallback.h > Should these be textually next to the other fileapi files? Same here, they are apart from others because they're in 'if ENABLE_FILE_SYSTEM' conditional. I'll move them next to the other fileapi files once we start to compile them by default.
Committed r66586: <http://trac.webkit.org/changeset/66586>
http://trac.webkit.org/changeset/66586 might have broken Qt Linux Release minimal, Qt Linux ARMv5 Release, and Qt Linux ARMv7 Release