1657270 - Redesign `HTMLEditor::MayDeleteTopMostEmptyAncestor()`

Status: RESOLVED FIXED

(Core :: DOM: Editor, task, P2)

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1657270 - part 1: Move `HTMLEditor::MaybeDeleteTopMostEmptyAncestor()` into new stack only class r=m_kato!

Depends on D85965

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1657270 - part 2: Split `AutoEmptyBlockAncestorDeleter::Run()` to scanning the target part and touching the DOM tree part r=m_kato!

Depends on D85995

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1657270 - part 3: Split `<br>` element insertion part for empty list element off from `AutoEmptyBlockAncestorDeleter::Run()` r=m_kato!

Depends on D85996

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1657270 - part 4: Split caret computation part off from `AutoEmptyBlockAncestorDeleter::Run()` r=m_kato!

Depends on D85997

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1657270 - part 5: Guarantee that `AutoEmptyBlockAncestorDeleter` did nothing if it returns "not handled" or an error at the caller r=m_kato!

Depends on D85998

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d179ec4f3379
part 1: Move `HTMLEditor::MaybeDeleteTopMostEmptyAncestor()` into new stack only class r=m_kato
https://hg.mozilla.org/integration/autoland/rev/2fc29f6624da
part 2: Split `AutoEmptyBlockAncestorDeleter::Run()` to scanning the target part and touching the DOM tree part r=m_kato
https://hg.mozilla.org/integration/autoland/rev/6017922d5be0
part 3: Split `<br>` element insertion part for empty list element off from `AutoEmptyBlockAncestorDeleter::Run()` r=m_kato
https://hg.mozilla.org/integration/autoland/rev/cc5bca926704
part 4: Split caret computation part off from `AutoEmptyBlockAncestorDeleter::Run()` r=m_kato
https://hg.mozilla.org/integration/autoland/rev/ac2c3f906e9d
part 5: Guarantee that `AutoEmptyBlockAncestorDeleter` did nothing if it returns "not handled" or an error at the caller r=m_kato

Comment 7

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/d179ec4f3379
https://hg.mozilla.org/mozilla-central/rev/2fc29f6624da
https://hg.mozilla.org/mozilla-central/rev/6017922d5be0
https://hg.mozilla.org/mozilla-central/rev/cc5bca926704
https://hg.mozilla.org/mozilla-central/rev/ac2c3f906e9d