Bug 1604821 Comment 8 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Jamie Nicol [:jnicol] from comment #7)
> I can reproduce on Linux (with webrender disabled) too. Mozregression gives this range: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ce1a86d3b4db161c95d1147676bbed839d7a4732&tochange=9056f2ee492fa481aa86146aba236c074628e9fd

+1, I just got the same regression range :)

> Matt's retained display list stuff jumps out the most, but I tried setting `layout.display-list.retain=false` and it did not make a difference.

I note that the bug in question, bug 1404181, is about landing retained-DL prefed off.

My guess would be, the patch series in that bug did make _some_ changes which were unconditional (rather than being conditioned on the retained-DL pref), and one of those changes caused the regression.
(In reply to Jamie Nicol [:jnicol] from comment #7)
> I can reproduce on Linux (with webrender disabled) too. Mozregression gives this range: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ce1a86d3b4db161c95d1147676bbed839d7a4732&tochange=9056f2ee492fa481aa86146aba236c074628e9fd

+1, I just got the same regression range :)

> Matt's retained display list stuff jumps out the most, but I tried setting `layout.display-list.retain=false` and it did not make a difference.

I note that the bug in question, bug 1404181, is about landing retained-DL prefed off (such that, if this were caused by enabling the retained-DL pref, the bug would not repro in the state where the retained-DL patches have landed but have not been prefed on yet).

My guess would be, the patch series in that bug did make _some_ changes which were unconditional (rather than being conditioned on the retained-DL pref), and one of those changes caused the regression.

Back to Bug 1604821 Comment 8