[webkit-reviews] review requested: [Bug 41510] Add ignoreWhitespace option at base64Decode() : [Attachment 60456] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Jul 3 14:05:49 PDT 2010


Patrick R. Gansterer <paroga at paroga.com> has asked  for review:
Bug 41510: Add ignoreWhitespace option at base64Decode()
https://bugs.webkit.org/show_bug.cgi?id=41510

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

------- Additional Comments from Patrick R. Gansterer <paroga at paroga.com>
> I am not sure we need a special case for whitespace in particular at all.
Perhaps the feature we need is “skip/tolerate non-base-64 characters”.
Other browsers ignore only whitespace.

RFC 2396:
> In some cases, extra whitespace (spaces, linebreaks, tabs, etc.) may
> need to be added to break long URI across lines. The whitespace
> should be ignored when extracting the URI.
There is no exact definition what whitespace is.


More information about the webkit-reviews mailing list