Closed Bug 1516410 Opened 8 months ago Closed 7 months ago
Fix some display: contents code in Content
47 bytes, text/x-phabricator-request
|Details | Review|
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).
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/b8346afda87b Fix some display: contents code in ContentRemoved. r=mats
You need to log in before you can comment on or make changes to this bug.