RESOLVED FIXED 111481
Do not add CSSPropertyBorderImage shorthand part of the property list when parsing CSS border property 
https://bugs.webkit.org/show_bug.cgi?id=111481
Summary Do not add CSSPropertyBorderImage shorthand part of the property list when pa...
Alexis Menard (darktears)
Reported 2013-03-05 14:12:33 PST
Do not add CSSPropertyBorderImage shorthand part of the property list when parsing CSS border property
Attachments
Patch (2.08 KB, patch)
2013-03-05 14:21 PST, Alexis Menard (darktears) 		no flags
DetailsFormatted DiffDiff
Patch for landing (5.55 KB, patch)
2013-03-06 03:53 PST, Alexis Menard (darktears) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alexis Menard (darktears)
Comment 1 2013-03-05 14:21:32 PST
Created attachment 191565 [details] Patch
WebKit Review Bot
Comment 2 2013-03-05 15:29:05 PST
Comment on attachment 191565 [details] Patch Attachment 191565 [details] did not pass chromium-ews (chromium-xvfb): Output: http://webkit-commit-queue.appspot.com/results/17074038 New failing tests: inspector/styles/styles-new-API.html
Julien Chaffraix
Comment 3 2013-03-05 15:41:59 PST
Comment on attachment 191565 [details] Patch r=me, the inspector/styles/styles-new-API.html diff is real: it's caused by a couple of border-image now being expanded. Here is the diff: https://gist.github.com/anonymous/5095388
Build Bot
Comment 4 2013-03-05 16:53:40 PST
Comment on attachment 191565 [details] Patch Attachment 191565 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-commit-queue.appspot.com/results/17004049 New failing tests: inspector/styles/styles-new-API.html
Alexis Menard (darktears)
Comment 5 2013-03-06 03:53:53 PST
Created attachment 191709 [details] Patch for landing
WebKit Review Bot
Comment 6 2013-03-06 04:17:00 PST
Comment on attachment 191709 [details] Patch for landing Clearing flags on attachment: 191709 Committed r144908: <http://trac.webkit.org/changeset/144908>
WebKit Review Bot
Comment 7 2013-03-06 04:17:04 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.