Bug 117500

Summary: Add CSS parsing recovery to functions
Product: WebKit Reporter: Sergio Villar Senin <svillar>
Component: CSSAssignee: Sergio Villar Senin <svillar>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, darin, esprehn+autocc, glenn, gyuyoung.kim, kling, macpherson, menard, pfeldman, rniwa, svillar, tkent
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 116071    
Attachments:
Description Flags
Patch none

Sergio Villar Senin
Reported 2013-06-11 01:18:38 PDT
We should consider merging: CSS parsing recovery in function. prop: expt(}); color: red; } and prop: expt(10px }); color: red; } are parsed differently. This change fixes that. Not completely though: difference occures if wrong part contains parentheses. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=150205
Attachments
Patch (6.48 KB, patch)
2013-06-11 01:44 PDT, Sergio Villar Senin 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sergio Villar Senin
Comment 1 2013-06-11 01:44:11 PDT
Created attachment 204319 [details] Patch
Sergio Villar Senin
Comment 2 2013-06-12 00:20:43 PDT
Comment on attachment 204319 [details] Patch Clearing flags on attachment: 204319 Committed r151488: <http://trac.webkit.org/changeset/151488>
Sergio Villar Senin
Comment 3 2013-06-12 00:20:52 PDT
All reviewed patches have been landed. Closing bug.
Gyuyoung Kim
Comment 4 2013-06-12 02:30:37 PDT
*** Bug 117068 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.