RESOLVED FIXED 122053
filter: drop-shadow doesnot support viewport units 
https://bugs.webkit.org/show_bug.cgi?id=122053
Summary filter: drop-shadow doesnot support viewport units
gur.trio
Reported Saturday, September 28, 2013 8:50:05 AM UTC
filter: drop-shadow doesnot support viewport units
Attachments
Test case for filter: drop-shadow (323 bytes, text/html)
2013-09-28 00:50 PDT, gur.trio 		no flags
Details
Patch (11.83 KB, patch)
2013-09-28 01:22 PDT, gur.trio 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
gur.trio
Comment 1 Saturday, September 28, 2013 8:50:51 AM UTC
Created attachment 212883 [details] Test case for filter: drop-shadow
gur.trio
Comment 2 Saturday, September 28, 2013 9:22:39 AM UTC
Created attachment 212885 [details] Patch
Darin Adler
Comment 3 Saturday, September 28, 2013 4:48:23 PM UTC
Comment on attachment 212885 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=212885&action=review > Source/WebCore/css/StyleResolver.cpp:3877 > + if (item->x->isViewportPercentageLength()) > + x = viewportPercentageValue(*item->x, x); Maybe we should have a function that does this dance. I have noticed that we have to repeat this idiom at every call site. If the inline call to isViewportPercentageLength is important for performance, we can make the function do it inline. I’d want to go back and do this for as many of the places we use viewportPercentageValue as possible. I don’t like repetitive idioms like this one when they are unnecessary.
WebKit Commit Bot
Comment 4 Saturday, September 28, 2013 5:12:47 PM UTC
Comment on attachment 212885 [details] Patch Clearing flags on attachment: 212885 Committed r156606: <http://trac.webkit.org/changeset/156606>
WebKit Commit Bot
Comment 5 Saturday, September 28, 2013 5:12:49 PM UTC
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.