[webkit-reviews] review granted: [Bug 85738] Cleanup: Move FileSystem API type definitions into a separate header file : [Attachment 140429] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun May 6 11:35:32 PDT 2012


David Levin <levin at chromium.org> has granted Kinuko Yasuda
<kinuko at chromium.org>'s request for review:
Bug 85738: Cleanup: Move FileSystem API type definitions into a separate header
file
https://bugs.webkit.org/show_bug.cgi?id=85738

Attachment 140429: Patch
https://bugs.webkit.org/attachment.cgi?id=140429&action=review

------- Additional Comments from David Levin <levin at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=140429&action=review


> Source/WebKit/chromium/src/AsyncFileSystemChromium.cpp:59
>  const size_t externalPathPrefixLength = sizeof(externalPathPrefix) - 1;

I think the one place in the code that used this went away. Feel free to delete
it.


More information about the webkit-reviews mailing list