Closed Bug 1516410 Opened 8 months ago Closed 7 months ago

Fix some display: contents code in ContentRemoved.

Categories

(Core :: Layout, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

No description provided.
There's no real behavior change, since:

 * aFlags right now is only used for whitespace suppression optimizations (which
   we don't do when display: contents is involved anyway).

 * The CouldBeDisplayContents() check is only an optimization: we're
   unconditionally recursing into ContentRemoved now, which is not incorrect,
   just inefficient.

I found this while reviewing https://phabricator.services.mozilla.com/D15319
(where this _would_ have an impact on behavior).
Priority: -- → P2
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b8346afda87b
Fix some display: contents code in ContentRemoved. r=mats
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.