12421 - string bundle needs to be moved to editor app shell

Status: VERIFIED FIXED

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

Description

[buster]

from the newsgroup:

In article <[e-mail]>, Steve Clark
<[e-mail]> wrote:

> 75% of our initialization time is creating the string bundle.  I assume
> the string bundle is only needed if the editor is going to be used in a
> mode where it provides it's own UI (dialogs and such), and it isn't
> needed otherwise (as with the simple HTML text control.)  I'd like to
> add a boolean param to nsIEditor::Init() to control this.  I can't just
> make it a flag, because the flags are defined in higher level editors.
> Maybe we should reserve a few bits of the flags for nsIEditor itself?
> Then I could just use the flags param.

Charlie and I have talked about the string bundle before, and
agreed that it should be moved into the editorShell anyway.
Recall that we're trying to keep all editor UI-related stuff
in the editor shell.

Charlie or I could do that work at any time.

Simon

Comment 1

[buster]

simon: the priority on this has been escalated.  It's a fairly substantial part
of the performance problem with text controls, and text controls will be turned
on by default very, very soon.  Please get this done as soon as reasonably
possible.  Thanks.

Comment 2

[Simon Fraser [no longer active]]

charlie will handle this

Comment 3

[Charles Manske]

Checked in 8/28

Comment 4

[sujay]

Buster/Charley, can you mark this verified-fixed...thanks!

Comment 5

[buster]

verified.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/87572c92f6cf
[fenix] Remove the duplicated/outdated 'activation' ping docs (https://github.com/mozilla-mobile/fenix/pull/12421)