1673790 - Split `HTMLEditSubActionHandler.cpp` to 3 files

Status: RESOLVED FIXED

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

Description

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

Although still bug 1659402 is open, we should split HTMLEditSubActionHandler.cpp ASAP due to too big (15k lines).

One should have ListElementSelectionState, ListItemElementSelectionState, AlignStateAtSelection, ParagraphStateAtSelection. Perhaps, the file name should be HTMLEditorState.cpp.

Next should have HTMLEditor::HandleDeleteSelection(), HTMLEditor::HTMLEditor::ComputeTargetRanges() and their helper stack only classes. Perhaps, the file name should be HTMLEditorDeleteHandler.cpp.

The others should remain in current file.

Comment 1

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

Attachment: Bug 1673790 - Split `editor/libeditor/HTMLEditSubActionHandler.cpp` r=m_kato!

It's too big (+15k lines) and there are 2 groups which does not depend on the
others. One is getting current state for Document.queryCommandState etc.
This is copied to HTMLEditorState.cpp. And the other is delete handlers.
This is copied to HTMLEditorDeleteHandler.cpp.

Comment 2

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

FYI: HTMLEditorDeleteHandler.cpp has some HTMLEditor methods which are used only when handling the deletion in the file (may be called via WSRunObject.cpp though).

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dcb0a318d44d
Split `editor/libeditor/HTMLEditSubActionHandler.cpp` r=m_kato

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/dcb0a318d44d