42553 - ender lite event processing needs to be at the frame level

Status: NEW

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

Description

[saari (gone)]

The current implementation of ender lite handles most of its events at the DOM 
level through normal DOM listeners. This needs to move to the frame level of 
event handling for standards compliance.

Nominating nsbeta2 because I bet this is causing other bugs.

Comment 1

[saari (gone)]

Okay, I'm going to change my mind again and say that this should be nsbeta2+ 
because:

The incorrect event sequencing potentially breaks all platform specific XBL key 
bindings, 
depending on when the keybinding is added to the DOM node, and this may also 
break 
XUL keybindings. I think this is why I cannot use any keybindings in the open 
location 
dialog on Win32. I think this is part of the problem for my nsbeta2+ bug 31233.

I'm pretty sure this event sequence problem is causing a whole raft of bugs.

Comment 2

[Phil Peterson]

[nsbeta2+]

Comment 3

[Peter Trudelle]

Doesn't fit latest 'would ya pull it off the wire' criteria, marking nsbeta2-,
nominating for nsbeta3

Comment 4

[Peter Trudelle]

nsbeta3+

Comment 5

[saari (gone)]

mjudge has a duplicate of this bug, assigning to him

Comment 6

[mjudge]

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

Comment 7

[mjudge]

bouncing back to saari. because he is the man. he is going to add another api to 
add domeventlisteners into another pile from the ones on the document.

Comment 8

[Peter Trudelle]

Chris, do you accept this bug? ->P3

Comment 9

[saari (gone)]

I'll take it

Comment 10

[saari (gone)]

If we're going to keep ender event processing in the DOM level, all the ender
event listeners need to start checking if PreventDefault has been called on an
event before they handle it.

Comment 11

[Peter Trudelle]

reassigning to mjudge, since he's working on it, and saari is leaving for 
vacation.  Just adding in the prevent default check to Enders event listeners 
should be enough for 6.0

Comment 12

[mjudge]

    This is NOT enough for 6.0.  If someone adds a listener after ender adds its 
listener, they will not be able to stop the editor from inserting keys, mouse 
clicks, focus ect.  if saari cant do this give it to hyatt.  He needs to do this 
for XBL keylisteners anyway.  
  I will make sure that we are checking for prevent default, i think we are 
allready, but this does not solve the problem of ordering. 

I am going to bounce this to hyatt myself to make sure he sees this. If 
hyatt wont fix it, just mark it WONTFIX.

Comment 13

[David Hyatt]

Removing nsbeta3+ for evaluation.

Comment 14

[Peter Trudelle]

nsbeta3-/future, will have to get to this in 6.0.1

Comment 15

[Peter "jag" Annema]

trudelle: time for a re-evaluation?

Comment 16

[saari (gone)]

This still needs to happen, but I don't think it fits the current goals. But it
needs to happen (or something to get the same effect ala event processing levels
or whatever). I'd say post 6.5 unfortunately.

Comment 17

[Peter Trudelle]

If anyone feels strongly about this, clear the future target and we will re-eval
as time allows.

Comment 18

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority.

If you have reason to believe this is wrong (especially for the severity), please write a comment and ni :jstutte.