[webkit-reviews] review granted: [Bug 6452]
KURL::appendEscapingBadChars() doesn't know about
%u-escaping. : [Attachment 6486] don't escape % characters
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Tue Feb 14 18:05:24 PST 2006
Maciej Stachowiak <mjs at apple.com> has granted Alexey Proskuryakov
<ap at nypop.com>'s request for review:
Bug 6452: KURL::appendEscapingBadChars() doesn't know about %u-escaping.
http://bugzilla.opendarwin.org/show_bug.cgi?id=6452
Attachment 6486: don't escape % characters
http://bugzilla.opendarwin.org/attachment.cgi?id=6486&action=edit
------- Additional Comments from Maciej Stachowiak <mjs at apple.com>
Based on your test case I think never escaping % is the right thing. I think I
am the one who did the rewrite and I didn't have a specific reason for escaping
%.
r=me
More information about the webkit-reviews
mailing list