1427908 - Never reenter synchronously into frame construction, and remove needless check in IssueSingleInsertNotifications

Status: RESOLVED FIXED

(Core :: Layout, enhancement)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1427908: Never reenter synchronously into frame construction.

We remove async from the DOM all the time now since bug 1389743.

We could, before this patch, recurse into frame construction in a sync way, due
to the way we handle the weird insertion cases for <fieldset>, <details>, and
<mathml>.

This patch makes those also async, making the IssueSingleInsertNotifications
condition unnecessary.

Review commit: https://reviewboard.mozilla.org/r/209998/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/209998/

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8939695 [details]
Bug 1427908: Never reenter synchronously into frame construction.

https://reviewboard.mozilla.org/r/209998/#review216356

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 8939695 [details]
Bug 1427908: Never reenter synchronously into frame construction.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/209998/diff/1-2/

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/97d9e1823553
Never reenter synchronously into frame construction. r=bz

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/97d9e1823553