[webkit-reviews] review denied: [Bug 62811] [filesystem/Chromium] Filesystem paths need proper URL escaping : [Attachment 97512] Make encodeWithURLEscapeSequences in KURLGoogle.cpp actually do what it says, then use it on filesystem paths.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 16 15:42:45 PDT 2011


Adam Barth <abarth at webkit.org> has denied Eric U. <ericu at chromium.org>'s
request for review:
Bug 62811: [filesystem/Chromium] Filesystem paths need proper URL escaping
https://bugs.webkit.org/show_bug.cgi?id=62811

Attachment 97512: Make encodeWithURLEscapeSequences in KURLGoogle.cpp actually
do what it says, then use it on filesystem paths.
https://bugs.webkit.org/attachment.cgi?id=97512&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
Looks great, but we'll need a ChangeLog.  Can you write a Chromium API test for
this change?


More information about the webkit-reviews mailing list