Bug 75671

Summary: Date parsing is too restrictive.
Product: WebKit Reporter: Gavin Barraclough <barraclough>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fix oliver: review+

Gavin Barraclough
Reported 2012-01-05 16:40:25 PST
ES5 date parsing currently requires are fields to be present, which does not match the spec (ES5.1 15.9.1.15). The spec allow a date to be date only, or date + time. The date portion on the should match: (pseudocode!:) [(+|-)YY]YYYY[-MM[-DD]] The time portion should match: THH:mm[:ss[.sss]][Z|(+|-)HH:mm]
Attachments
Fix (21.34 KB, patch)
2012-01-05 16:57 PST, Gavin Barraclough 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2012-01-05 16:57:00 PST
Created attachment 121369 [details] Fix
Oliver Hunt
Comment 2 2012-01-05 17:00:38 PST
Comment on attachment 121369 [details] Fix View in context: https://bugs.webkit.org/attachment.cgi?id=121369&action=review run perf tests > Source/JavaScriptCore/ChangeLog:8 > + ES5 date parsing currently requires are fields to be present, which does not match the spec (ES5.1 15.9.1.15). all
Gavin Barraclough
Comment 3 2012-01-05 17:09:39 PST
Fixed in r104251
Note You need to log in before you can comment on or make changes to this bug.