Closed Bug 1761159 Opened 2 years ago Closed 2 years ago

Make SplitNodeResult constructors logical

Categories

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

task

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

Attachments

(1 file)

Currently, they take content nodes as physical position from a split point. However, for making the creators simpler, they should take the content nodes as logical, i.e., a new node and a original node.

Like JoinNodesResult, SplitNodeResult should take 2 content nodes as
"new one" and "original one" instead of "previous one" and "next one".
Then, the creators is easier to read.

Additionally, this patch creates some static methods and converter method
to create a result with explaining the result meaning.

Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/30b727170d71
Make constructors of `SplitNodeResult` take content nodes with logical meaning rather than physical meaning r=m_kato
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: