Bug 145606 - Use borderBoxRect instead of contentBoxRect for backdrop filter.
Summary: Use borderBoxRect instead of contentBoxRect for backdrop filter.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-06-03 09:02 PDT by zalan
Modified: 2015-06-04 08:09 PDT (History)
7 users (show)

See Also:


Attachments
Patch (2.94 KB, patch)
2015-06-03 09:12 PDT, zalan
no flags Details | Formatted Diff | Diff
Patch (7.83 KB, patch)
2015-06-03 22:09 PDT, zalan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2015-06-03 09:02:36 PDT
SSIA
Comment 1 zalan 2015-06-03 09:12:52 PDT
Created attachment 254180 [details]
Patch
Comment 2 Simon Fraser (smfr) 2015-06-03 09:15:47 PDT
Comment on attachment 254180 [details]
Patch

I don't think the box should be affected by backgroundClip().
Comment 3 zalan 2015-06-03 22:09:49 PDT
Created attachment 254245 [details]
Patch
Comment 4 WebKit Commit Bot 2015-06-03 23:24:13 PDT
Comment on attachment 254245 [details]
Patch

Clearing flags on attachment: 254245

Committed r185192: <http://trac.webkit.org/changeset/185192>
Comment 5 WebKit Commit Bot 2015-06-03 23:24:16 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 zalan 2015-06-04 07:21:45 PDT
(In reply to comment #6)
> The new test fails on multiple queues:
> https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.
> html#showAllRuns=true&tests=css3%2Ffilters%2Fbackdrop%2Fbackdrop-filter-with-
> mask.html
Oh, this is because Simon added the mask layers to the layer tree output. It needs rebaselining. updating...
Comment 8 zalan 2015-06-04 08:09:03 PDT
(In reply to comment #6)
> The new test fails on multiple queues:
> https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.
> html#showAllRuns=true&tests=css3%2Ffilters%2Fbackdrop%2Fbackdrop-filter-with-
> mask.html

https://trac.webkit.org/changeset/185199