Open Bug 1951037 Opened 19 days ago Updated 8 days ago

Redesign `WhiteSpaceVisibilityKeeper` as stack only classes and nested classes of `HTMLEditor`

Categories

(Core :: DOM: Editor, task)

task

Tracking

()

People

(Reporter: masayuki, Unassigned)

References

(Depends on 2 open bugs, Blocks 1 open bug)

Details

Currently, WhiteSpaceVisibilityKeeper is a class having only static methods. Currently the new white-space normalizer (and the patched which will be landed in bug 1940377) is partially written in WhiteSpaceVisibilityKeepr, but the others are written in HTMLEditor itself. Additionally, there is no concrete reason of managing them in 2 places. Additionally, the methods have a lot of similar lambdas for the performance to skip checking the white-space style.

I'm currently thinking that after deleting the legacy white-space normalizer, we should make a base nested class of HTMLEditor which stores basic information of the target Text and each subclass should implement each final output of WhiteSpaceVisibilityKeeper does.

You need to log in before you can comment on or make changes to this bug.