Closed Bug 1419586 Opened 2 years ago Closed 2 years ago
Allow painted layers with ancestor mask layers to use OMTP
59 bytes, text/x-review-board-request
From talking with David, we don't think we need to restrict OMTP to layers without mask layers.  The mask layer rendering is just sending over transaction information , and the actual rasterization takes place in FrameLayerBuilder  which should happen after we have FlushedAsyncPaints() but before we have queued any new ones, so there should be no contention with Direct2D. I'm going to do a try run to verify there are no obvious issues.  https://searchfox.org/mozilla-central/rev/797c93d81fe446f78babf20894f0729f15f71ee6/gfx/layers/client/ClientPaintedLayer.cpp#71  https://searchfox.org/mozilla-central/rev/919dce54f43356c22d6ff6b81c07ef412b1bf933/layout/painting/FrameLayerBuilder.cpp#3930  https://searchfox.org/mozilla-central/rev/919dce54f43356c22d6ff6b81c07ef412b1bf933/gfx/layers/client/ClientImageLayer.cpp#134
Comment on attachment 8930951 [details] Allow painted layers with ancestor mask layers to use OMTP (bug 1419586, ) https://reviewboard.mozilla.org/r/202036/#review208382
Attachment #8930951 - Flags: review?(dvander) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/444fef22c201 Allow painted layers with ancestor mask layers to use OMTP (bug 1419586, r=dvander)
You need to log in before you can comment on or make changes to this bug.