1243336 - Moz2d's new PushLayer/PopLayer API interacts badly with DrawTargetTiled

Status: RESOLVED FIXED

(Core :: Graphics, defect, P3)

Description

[Nicolas Silva [:nical]]

Attachment: screenshot of theverge.com with the glitches

It is probably expected since I think PushLayer isn't implemented for all backends yet. enabling DrawTargetTiled and PushLayer/PopLayer gives some weird semi-transaprent balck boxes over content. Some pages reproduce the issue a lot, like theverge.com.

Comment 1

[Bas Schouten (:bas.schouten)]

The cause here is that if we're not even pushing their clips to tiles that are completely clipped out, but we're also not pushing drawing commands, if we then proceed to push and pop opaque layers to them, they will be drawn back black with a certain opacity. I have a fix.

Comment 2

[Bas Schouten (:bas.schouten)]

Attachment: MozReview Request: Bug 1243336: Don't push layers to tiles that are completely clipped out. r=jrmuizel

Review commit: https://reviewboard.mozilla.org/r/34683/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/34683/

Comment 3

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8718668 [details]
MozReview Request: Bug 1243336: Don't push layers to tiles that are completely clipped out. r=jrmuizel

https://reviewboard.mozilla.org/r/34683/#review32745

Comment 4

[Jonathan Watt [:jwatt]]

Any plans to land this, Bas?

Comment 5

[Bas Schouten (:bas.schouten)]

(In reply to Jonathan Watt [:jwatt] from comment #4)
> Any plans to land this, Bas?

We probably should? Does it still pass tests and stuff? Do we still need it though? If we had this bug I'd expect more people to be complaining about it now that DTTiled and Push/PopLayer are all implemented.

Comment 6

[Jonathan Watt [:jwatt]]

Note Push/PopLayer are not turned on on Mac/Android (bug 1322729). As for your other questions, I don't know. I just noticed this hadn't landed in passing.