Now that we're going to implement HTML5 FileSystem API, it'd be natural to have WebFileSystem as a WebKit API for HTML5 FileSystem and to rename current WebFileSystem (that implements platform/FileSystem methods) to something like WebFileUtilities.
Created attachment 64528 [details] Patch
Comment on attachment 64528 [details] Patch Please add "[chromium]" prefix to the bug title and ChangeLog description. Please also mention why we need to rename WebFileSystem to WebFileUtilities (probably same thing as you put in the bug detail).
(In reply to comment #2) > (From update of attachment 64528 [details]) > Please add "[chromium]" prefix to the bug title and ChangeLog description. Please also mention why we need to rename WebFileSystem to WebFileUtilities (probably same thing as you put in the bug detail). Sorry for lack of description/discussion, there was a discussion about how the new API for the HTML5 FileSystem should be named/defined (you can see some of the discussion at https://bugs.webkit.org/show_bug.cgi?id=43151). Basically why this patch tries to rename WebFileSystem to WebFileUtilities is that we thought it would be less confusing (for external API users - which happen to be chromium only) to make "WebFileSystem" a corresponding API of HTML5 FileSystem. (Internally WebFileSystem implements platform/FileSystem so renaming it to another has some side effect too.) How do you think?
Created attachment 64535 [details] Patch
Comment on attachment 64535 [details] Patch Looks good except minor issues. Please fix them before you land the patch. WebKit/chromium/public/WebFileSystem.h:38 + // FIXME: Clean up this class once the renaming to WebFileUtilities has done. has done => is done Please also change all other occurrences.
Committed r65482: <http://trac.webkit.org/changeset/65482>
http://trac.webkit.org/changeset/65482 might have broken Chromium Win Release
(In reply to comment #7) > http://trac.webkit.org/changeset/65482 might have broken Chromium Win Release This has been fixed in http://trac.webkit.org/changeset/65485.