Summary: | Cleanup: Change boolean synchronous flag argument into enum in FileSystem API code | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kinuko Yasuda <kinuko> | ||||||
Component: | WebCore Misc. | Assignee: | Kinuko Yasuda <kinuko> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | ||||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Kinuko Yasuda
2012-05-06 11:02:17 PDT
Created attachment 140426 [details]
Patch
Created attachment 140427 [details]
Patch
Comment on attachment 140427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=140427&action=review > Source/WebCore/platform/FileSystemType.h:1 > +/* I'm surprised that there aren't some build file changes to go along with this. It would be nice to include it in build files. As long as this not being the those files doesn't break anything and it doesn't seem to, I would be fine with that as a follow up patch -- In fact I made this comment in the other patch so I'd land both and do one follow up patch for both of them :) (but no rush on that follow up patch). Committed r116283: <http://trac.webkit.org/changeset/116283> Comment on attachment 140427 [details]
Patch
Will create a follow-up patch for adding the header file to build/make files.
|