34896 - text widgets should use anonymous content

Status: VERIFIED FIXED

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

Description

[mjudge]

Reducing ender down to the minimum of instantiations.  Rather than lazy 
instantiations, we place the editor on top of anonymous content. 

1. Make a new widget with a view and an anonymous DIV.
2. Make an editor attached to that widget and initialize with a given selection 
object as well as a content node(aka the DIV)
3. Make selection PER one of these widgets. make an interface to allow querying 
of display selection ON/OFF.
4. Implement all the stuff that the old widget did.
5. fix all bugs before checkin in.

Comment 1

[Kathleen :Brade]

cc akkana

Comment 2

[gabriel]

Adding self to cc, as this relates to bug 31485 which I reported.

Comment 3

[mjudge]

*** Bug 34982 has been marked as a duplicate of this bug. ***

Comment 4

[Kathleen :Brade]

new summary per team meeting

Comment 5

[rubydoo123]

this is the rework that Mike is doing for "ender lite" this is a must fix for 
beta2

Comment 6

[leger]

Putting on [nsbeta2+] radar for beta2 fix.

Comment 7

[Dawn Endico]

moving from architecture to browser product

Comment 8

[mjudge]

this is almost ready to go just putting finishing touches on it.

Comment 9

[mjudge]

once again into the breach. i desperately want to get this to wrap i will not
even set a break point because i a SOOO sure this is going to work . fore score
and seven years ago.

Comment 10

[leger]

Giving [nsbeta2+]Exception Feature status.

Comment 11

[mjudge]

one more thing. ender-lite turns out does NOT fix this problem. sorry i mislead 
people i didnt realize the problem well enough before.

Comment 12

[mjudge]

turned it on now. any problems lets solve them 1 at a time with more bugs. 
thanks

Comment 13

[sujay]

Mike can you help me verifdy this one?

Comment 14

[mjudge]

this is definately fixed. the only fear now is of other problems it may cause...

Comment 15

[Masaki Katakai]

Hi Michael and Akkana,

nsGfxTextControlFrame::SetFocus() calls nsIWidget->SetFocus()
but nsGfxTextControlFrame2::SetFocus() is just null and
doesn't call anything. Is there any reason?

It seems that the changes causes 42263, widget/src/gtk/nsWidget could not
get the SetFocus() call for proper nsIWidget.
UNIX IME can not get the exact Widget by SetFocus() call
so can not manage the input context properly. Could you
implement SetFocus() in nsGfxTextControlFrame2::SetFocus()
to call nsIWidget->SetFocus()?