Closed Bug 871786 Opened 6 years ago Closed 6 years ago
No description provided.
This is needed because some existing event handlers expect that |instanceof KeyEvent| will return true for keyboard events.
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Attachment #749027 - Flags: review?(bugs)
Comment on attachment 749027 [details] [diff] [review] Separate D2E KeyEvent constants and a method from KeyboardEvent I don't know how that Bindings.conf part works.
Comment on attachment 749029 [details] [diff] [review] Remove KeyboardEvent classinfo Could you remove key event from dom_quickstubs.qsconf
Attachment #749029 - Flags: review?(bugs) → review+
Comment on attachment 749027 [details] [diff] [review] Separate D2E KeyEvent constants and a method from KeyboardEvent This looks fine as long as there are no other implementations of KeyEvent.
Attachment #749027 - Flags: review?(bzbarsky) → review+
This is more conformant to the latest spec because KeyEvent will not be inserted into KeyboardEvent's prototype chain.
patch for check-in
Attachment #749029 - Attachment is obsolete: true
Comment on attachment 749487 [details] [diff] [review] Separate D2E KeyEvent constants and a method from KeyboardEvent, v2 >+ 'nativeType': 'nsDOMKeyboardEvent', You don't need that is 'concrete':False and nothing concrete inherits from the interface. r=me with that.
Attachment #749487 - Flags: review?(bzbarsky) → review+
Some .wedidl files are added despite of bug 872856. Trying to land. https://hg.mozilla.org/integration/mozilla-inbound/rev/88a1b42e82a1 https://hg.mozilla.org/integration/mozilla-inbound/rev/fb4696685200
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.