Summary: | Move FileSystem related files from WebCore/storage to WebCore/fileapi. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kinuko Yasuda <kinuko> | ||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, eric, fishd, jianli, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
Kinuko Yasuda
2010-08-30 19:43:56 PDT
(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 |