New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 808908 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[css-paint-api] [css-typed-om] Paint module on "body" element uses "document" node instead.

Project Member Reported by shend@chromium.org, Feb 5 2018

Issue description

In investigating https://github.com/w3c/css-houdini-drafts/issues/627 , I found that the test case breaks because, for some reason, we are passing a "document" node, instead of the "body" node, into FilteredComputedStylePropertyMap [1], which doesn't have a ComputedStyle. If we change the test case to "div", then everything works because the div has a ComputedStyle.

I suspect we should be passing in the "body" element into FiltedComputedStylePropertyMap, but not really sure how that's done. Assuming that's true, assigning to Blink>Paint because the bug would be in CSSPaintDefinition::Paint.

[1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/modules/csspaint/CSSPaintDefinition.cpp?q=CSSPaintDefinition::Paint&sq=package:chromium&l=100
 

Comment 1 by f...@opera.com, Feb 5 2018

Sounds like ViewPainter not using the "correct" image client (after/in combination with Document::PropagateStyleToViewport)

Sign in to add a comment