Summary: | [chromium] Prepare to rename WebFileSystem interface to WebFileUtilities | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kinuko Yasuda <kinuko> | ||||||
Component: | WebKit API | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, ericu, eric, fishd, jianli, kkanetkar, michaeln, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | OS X 10.5 | ||||||||
Attachments: |
|
Description
Kinuko Yasuda
2010-08-16 15:14:52 PDT
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. |