On Mon, Aug 30, 2010 at 9:11 AM, Maciej Stachowiak <[email protected]> wrote:
> > On Aug 30, 2010, at 8:36 AM, Darin Fisher wrote: > > On Mon, Aug 30, 2010 at 12:18 AM, Adam Barth <[email protected]> wrote: > >> On Fri, Aug 27, 2010 at 8:12 PM, Maciej Stachowiak <[email protected]> wrote: >> > Yes. The file-related stuff should all be in one directory, I think. >> >> Ok. I moved the files from WebCore/html to WebCore/fileapi. >> >> On Aug 27, 2010, at 6:19 PM, Kinuko Yasuda wrote: >> > We have bunch of FileSystem (which is a part of File API) related files >> in >> > WebCore/storage/. >> > Maybe we should move them to the new directory too? >> >> Are these the files you're talking about? >> >> WebCore/storage/DOMFilePath.cpp >> WebCore/storage/DOMFilePath.h >> WebCore/storage/DOMFileSystem.cpp >> WebCore/storage/DOMFileSystem.h >> WebCore/storage/DOMFileSystem.idl >> WebCore/storage/FileEntry.cpp >> WebCore/storage/FileEntry.h >> WebCore/storage/FileEntry.idl >> WebCore/storage/FileSystemCallback.h >> WebCore/storage/FileSystemCallback.idl >> WebCore/storage/FileSystemCallbacks.cpp >> WebCore/storage/FileSystemCallbacks.h >> WebCore/storage/LocalFileSystem.cpp >> WebCore/storage/LocalFileSystem.h >> >> I'm happy to move them to WebCore/fileapi, but I'm also happy for you to >> do it. >> >> Thanks, >> Adam >> >> > > How about just moving everything into WebCore/storage? This is all > storage-related stuff. > > > I think the File API is large enough to deserve its own directory. In fact, > it might be worth splitting up the remaining contents of the storage > directory too. It is confusing to have large but almost entirely separate > APIs all piled into one directory. It is true they are all > "storage-related", but that is a pretty broad theme, and LocalStorage, SQL > Storage, Indexed DB and File API have little or no interaction with each > other. > > Regards, > Maciej > > > > That's fair. Plus, there are a lot of files in there already. -Darin
_______________________________________________ webkit-dev mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
