string bundle needs to be moved to editor app shell

VERIFIED FIXED in M10

Status

()

P1
major
VERIFIED FIXED
19 years ago
17 years ago

People

(Reporter: buster, Assigned: cmanske)

Tracking

Trunk
All
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

19 years ago
from the newsgroup:

In article <37C2F725.9177F83E@netscape.com>, Steve Clark
<buster@netscape.com> 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
(Reporter)

Updated

19 years ago
Priority: P3 → P2
(Reporter)

Updated

19 years ago
Priority: P2 → P1
Target Milestone: M10
(Reporter)

Comment 1

19 years ago
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.

Updated

19 years ago
Depends on: 12158

Updated

19 years ago
Assignee: sfraser → cmanske

Comment 2

19 years ago
charlie will handle this
(Assignee)

Updated

19 years ago
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

19 years ago
Checked in 8/28

Comment 4

19 years ago
Buster/Charley, can you mark this verified-fixed...thanks!
(Reporter)

Updated

19 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 5

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