Open Bug 1785007 (stop-editor-declaring-friends) Opened 2 years ago Updated 2 years ago

Split editor classes for outer and inner one from point of view of the editor module


(Core :: DOM: Editor, enhancement, P4)





(Reporter: masayuki, Unassigned)


(Blocks 1 open bug)


Currently, EditorBase, TextEditor and HTMLEditor declares public methods only for outer users of the editor module, e.g., DOM and layout. For helper classes in the editor module, they register a lot of friend classes like:

Therefore, the member scope of editor classes do not work actually. This is too bad especially for new contributors. Currently I'm thinking that we should keep EditorBase, TextEditor and HTMLEditor classes as public methods in the tree. Then, we should make them has new classes which are copies of current those classes, perhaps, new name should be EditorBaseCore, TextEditorCore and HTMLEditorCore or something. Then, they can set their member scopes properly, and we can stop exposing editor local headers such as EditorUtils.h, HTMLEditHelpers.h, etc which are required to declare the editor classes.

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