[webkit-reviews] review granted: [Bug 235468] [JSC] Relax Date.parse requirement : [Attachment 449724] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Yusuke Suzuki <ysuzuki at apple.com>'s
request for review:
Bug 235468: [JSC] Relax Date.parse requirement
https://bugs.webkit.org/show_bug.cgi?id=235468

Attachment 449724: Patch

https://bugs.webkit.org/attachment.cgi?id=449724&action=review




--- Comment #3 from Darin Adler <darin at apple.com> ---
Comment on attachment 449724
  --> https://bugs.webkit.org/attachment.cgi?id=449724
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=449724&action=review

> Source/WTF/ChangeLog:12
> +	   While the spec does not require accepting 't' / ' ' separator, ISO
8601 accepts it.
> +	   This is because ECMA262's Date format is *not* ISO 8601 (it is
called simplification
> +	   of ISO 8601[1]).
> +	   This patch relaxes this strictness to accept more formats, which can
be accepted in
> +	   the other engines too.

Is there some interoperability test suite we can add this to, in addition to
having it in a WebKit-specific test?