Bug 119187 - Inset box-shadows fail to round around corners when border-radius is set in vh/vw units.
Summary: Inset box-shadows fail to round around corners when border-radius is set in v...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.8
: P2 Normal
Assignee: gur.trio
URL: http://jsfiddle.net/plapier/cwEAL/
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-27 20:22 PDT by Phil LaPier
Modified: 2013-09-26 09:56 PDT (History)
8 users (show)

See Also:


Attachments
Screenshot of the bug (88.05 KB, image/png)
2013-07-27 20:22 PDT, Phil LaPier
no flags Details
Patch (8.01 KB, patch)
2013-09-23 23:46 PDT, gur.trio
no flags Details | Formatted Diff | Diff
Patch (14.45 KB, patch)
2013-09-24 22:25 PDT, gur.trio
no flags Details | Formatted Diff | Diff
Patch (14.45 KB, patch)
2013-09-25 21:26 PDT, gur.trio
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Phil LaPier 2013-07-27 20:22:35 PDT
Created attachment 207595 [details]
Screenshot of the bug

The vh and vw units produce different bugs.

They both fail to round the inset shadow along the radius of the corner.

Non-inset shadows work as expected. 

View the JSFiddle: http://jsfiddle.net/plapier/cwEAL/
Comment 1 gur.trio 2013-09-23 23:46:17 PDT
Created attachment 212431 [details]
Patch
Comment 2 gur.trio 2013-09-24 00:37:37 PDT
(In reply to comment #1)
> Created an attachment (id=212431) [details]
> Patch

Hi Darin. Can you please review these changes?
Have moved the function viewportPercentageValue of StyleResolver from private to public so that other properties can use it while calculating the viewport lengths.
Comment 3 Darin Adler 2013-09-24 09:08:05 PDT
Comment on attachment 212431 [details]
Patch

Change looks OK, but I'm not sure there is enough test coverage.
Comment 4 gur.trio 2013-09-24 22:25:43 PDT
Created attachment 212539 [details]
Patch
Comment 5 Darin Adler 2013-09-25 09:38:14 PDT
Comment on attachment 212539 [details]
Patch

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

> Source/WebCore/css/StyleResolver.h:314
> +    int viewportPercentageValue(CSSPrimitiveValue& unit, int percentage);
>      struct RuleRange {

Why is this grouped together with RuleRange? There should at least be a blank line separating them.
Comment 6 gur.trio 2013-09-25 21:26:29 PDT
Created attachment 212662 [details]
Patch
Comment 7 gur.trio 2013-09-25 21:27:26 PDT
(In reply to comment #5)
> (From update of attachment 212539 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=212539&action=review
> 
> > Source/WebCore/css/StyleResolver.h:314
> > +    int viewportPercentageValue(CSSPrimitiveValue& unit, int percentage);
> >      struct RuleRange {
> 
> Why is this grouped together with RuleRange? There should at least be a blank line separating them.

Actually while merging from one workspace to another this issue came. Sorry about that.
Comment 8 WebKit Commit Bot 2013-09-26 09:56:28 PDT
Comment on attachment 212662 [details]
Patch

Clearing flags on attachment: 212662

Committed r156466: <http://trac.webkit.org/changeset/156466>
Comment 9 WebKit Commit Bot 2013-09-26 09:56:30 PDT
All reviewed patches have been landed.  Closing bug.