1409215 - don't change scroll thumbs into AGRs during display list building

Status: RESOLVED FIXED

(Core :: Web Painting, enhancement)

Comment 1

[Timothy Nikkel (:tnikkel)]

Attachment: scrollthumb

Comment 2

[Markus Stange [:mstange]]

Comment on attachment 8919094 [details] [diff] [review]
scrollthumb

Review of attachment 8919094 [details] [diff] [review]:
-----------------------------------------------------------------

The comment seems to be the only difference compared to part 15 in bug 1404181 which I just reviewed earlier today, so this was an easy re-review.

::: layout/painting/nsDisplayList.cpp
@@ +1580,5 @@
> +    nsIScrollableFrame* sf = static_cast<nsSliderFrame*>(parent)->GetScrollFrame();
> +    // The word "Maybe" in IsMaybeScrollingActive might be confusing but we do
> +    // indeed need to always consider scroll thumbs as AGRs if
> +    // IsMaybeScrollingActive is true because that is the same condition we use
> +    // in ScrollFrameHelper::AppendScrollPartsTo to layerizer scroll thumbs.

layerizer -> layerize

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d3d17327436
Don't change scroll thumbs into AGRs during display list building. r=mstange

Comment 4

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/7d3d17327436