eDontReparentFrames is unnecessary.

RESOLVED FIXED in Firefox 62

Status

()

enhancement
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: emilio, Assigned: emilio)

Tracking

(Blocks 1 bug)

unspecified
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(1 attachment)

I was trying to look into ways of simplifying our first-line stuff, since it's really bug-ridden (at least on the style side of it).

I noticed that one part of the "lazily set parent frame" optimization isn't really used at all: We only set lazilySetParentPointer on the first reflow, then only call DrainSelfOverflowListInternal with eDontReparentFrames on non-first reflows. Here's a try run that asserts that we don't pass the flag on our test-suite:

  https://treeherder.mozilla.org/#/jobs?repo=try&revision=583161894ad4e6518699d6fedd3f119c3c8c0597

So I think we can safely remove it.
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8981139 [details]
Bug 1464813: Remove eDontReparentFrames, since it's not used.

https://reviewboard.mozilla.org/r/247236/#review253340

IIRC, we preserved this option since we were planning to extend lazy reparenting to more cases at that time. But that never happened (probably for good reason).
Thanks for cleaning this up!
Attachment #8981139 - Flags: review?(mats) → review+

Comment 3

a year ago
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a1057c5c2542
Remove eDontReparentFrames, since it's not used. r=mats

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a1057c5c2542
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.