Closed Bug 24645 Opened 25 years ago Closed 24 years ago

[XBL] Convert DOM window keybindings to XBL

Categories

(Core :: XBL, defect, P3)

x86
Windows 98
defect

Tracking

()

RESOLVED DUPLICATE of bug 52359
Future

People

(Reporter: hyatt, Assigned: dr)

References

(Blocks 1 open bug)

Details

(Keywords: arch, perf, Whiteboard: [nsbeta2-],[nsbeta3-])

Attachments

(1 file)

Convert the key binding files (browsers, textareas, enders, and inputs) over to XBL. This needs to happen sooner rather than later. We have broken basic functionality (like keyboard scrolling) in the embedded case.
Status: NEW → ASSIGNED
Target Milestone: M14
putting on beta1 radar
I have converted inputs and textareas over. Next come the browser and editor DOM windows.
Whiteboard: 50% complete
Cc'ing myself so that I can add the Unix-specific bindings (if you don't do it for me) as soon as the framework is in place. The XBL bindings in the text fields/areas are SO much faster than XUL -- it'll be great to have them in the editor as well!
*** Bug 26314 has been marked as a duplicate of this bug. ***
Does this include the mail/news window? Mail compose?
This should be PDT+.
Keywords: beta1
Putting on PDT- radar for beta1. Concerned about regression risk at this time.
Whiteboard: 50% complete → [PDT-]50% complete
No, this HAS to be PDT+. Other PDT+ bugs depend on this.
wiped out pdt- so they will reconsider, though you should probably list the pdt+ bugs that depend on this, since they will ask :-)
Whiteboard: [PDT-]50% complete → 50% complete
Please put an estimated completion date on this.
Whiteboard: 50% complete → [PDT+] 50% complete
Whiteboard: [PDT+] 50% complete → [PDT+] 2/16 Estimated Fix Date
Well, if we don't care about the embedding case, I guess we could go to beta without finishing this conversion. Your call. If it is acceptable for viewer to remain in the busted state it's in now regarding window key bindings, then I'm comfortable shoving this to M15.
Summary: [XBL] Convert Ender keybindings to XBL → [XBL] Convert DOM window keybindings to XBL
Also, composer performance is adversely impacted by leaving the bindings in XUL. If you notice, typing in text fields is much faster than typing in the composer window. If an XBL conversion gets done, then the composer window will be much faster. There would undoubtedly be some regressions that would occur as fallout from this conversion however.
Trying to be pragmatic here. I think we can get by without this being in for beta. If I start getting bugs that become PDT+ that would depend on this, we can reconsider it, but I think for now we're ok.
Whiteboard: [PDT+] 2/16 Estimated Fix Date
Target Milestone: M14 → M15
Marking - due to hyatts recommendation.
Whiteboard: [PDT-]
Blocks: 27771
Blocks: 27661
*** Bug 27739 has been marked as a duplicate of this bug. ***
Target Milestone: M15 → M16
Mass-moving all M16 non-feature bugs to M17, which we still consider to be part of beta2
Target Milestone: M16 → M17
spam, open xptoolkit qa contact moving over to jrgm
QA Contact: paulmac → jrgm
nominating for nsbeta2
Keywords: beta1nsbeta2
Whiteboard: [PDT-]
[nsbeta2-]
Whiteboard: [nsbeta2-]
Adding mlk keyword because of bug 27739 (which was marked as a duplicate of this one).
Keywords: mlk
Blocks: 39855
Mass-moving all nsbeta2- bugs to M20
Target Milestone: M17 → M20
Please clean up the leak caused by the fix for bug 27739 when fixing this bug. Thanks! (Re: bug 36639)
*** Bug 40291 has been marked as a duplicate of this bug. ***
*** Bug 41232 has been marked as a duplicate of this bug. ***
It's actually not too hard to fix the keybindings leak without fixing this bug (which is probably hard). I wrote code to do this a few weeks ago, but I couldn't test very easily because I ran into bug 42895. However, I realize now I could use the steps in bug 41232 to test my fix... I'll try doing this soon.
Keywords: mlkperf
Let bug 17390 cover the leak issues from this point on. Removing mlk keyword from this bug but adding perf keyword since perfomance issues were discussed for the real issue in this bug.
So hyatt, can we get somebody else to help you do this? or do this for you? The current keybinding stuff has significant problems with leaks and cleaning itself up. (Although dbaron has tracked down most of those issues...)
Keywords: nsbeta2arch, nsbeta3
nsbeta3-, not worth the risk for seamonkey.
Whiteboard: [nsbeta2-] → [nsbeta2-],[nsbeta3-]
Workaround for embedding is to use a XUL layer. Otherwise this is huge and risky.
Target Milestone: M20 → Future
Adding nsbeta2 keyword to bugs with nsbeta2 triage value in status field so the queries don't get screwed up
Keywords: nsbeta2
Reassign to dr, since he just filed bug 52359.
Assignee: hyatt → dr
Status: ASSIGNED → NEW
Hmm. Can I mark this as a dup of bug 52359, or vice-versa? The only thing that's left to do here is the browserBindings and editorBindings, right?
Status: NEW → ASSIGNED
this is a dup of bug 52359, which will be fixed rsn as part of 48758. *** This bug has been marked as a duplicate of 52359 ***
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
No longer blocks: 27661
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: