superreview requested: [Bug 345521] "return 0" in handler cancels events : [Attachment 232080] Patch to fix and test event cancel behaviour

Mark Hammond <mhammond@skippinet.com.au> has asked Boris Zbarsky
<bzbarsky@mit.edu> for superreview:
Bug 345521: "return 0" in handler cancels events
https://bugzilla.mozilla.org/show_bug.cgi?id=345521

Attachment 232080: Patch to fix and test event cancel behaviour
https://bugzilla.mozilla.org/attachment.cgi?id=232080&action=edit

------- Additional Comments from Mark Hammond <mhammond@skippinet.com.au>
Here is a patch that fixes this behaviour, and a simple test for the existing
dom/tests/html/jshandlers.html file.

I can also confirm that IE behaves the same as Mozilla intends - ie, after
applying this patch, IE and Moz again behave the same.
0
bugzilla
8/4/2006 5:40:33 AM
mozilla.dev.super-review 29307 articles. 3 followers. Post Follow

0 Replies
692 Views

Similar Articles

[PageSpeed] 59

Reply:

Similar Artilces:

superreview granted: [Bug 345521] "return 0" in handler cancels events : [Attachment 232080] Patch to fix and test event cancel behaviour
Boris Zbarsky <bzbarsky@mit.edu> has granted Mark Hammond <mhammond@skippinet.com.au>'s request for superreview: Bug 345521: "return 0" in handler cancels events https://bugzilla.mozilla.org/show_bug.cgi?id=345521 Attachment 232080: Patch to fix and test event cancel behaviour https://bugzilla.mozilla.org/attachment.cgi?id=232080&action=edit ------- Additional Comments from Boris Zbarsky <bzbarsky@mit.edu> r+sr=bzbarsky. Does anyone ever run that test? ;) ...

superreview cancelled: [Bug 299221] [CLEANUP] assign "isChar" inside gecko event conversion : [Attachment 187762] fix v1.0
Josh Aas <joshmoz@gmail.com> has cancelled Josh Aas <joshmoz@gmail.com>'s request for superreview: Bug 299221: [CLEANUP] assign "isChar" inside gecko event conversion https://bugzilla.mozilla.org/show_bug.cgi?id=299221 Attachment 187762: fix v1.0 https://bugzilla.mozilla.org/attachment.cgi?id=187762&action=edit ...

superreview cancelled: [Bug 90918] RFE: Add url in "comments" field of the "get info" window when download file. : [Attachment 180129] Cleaned up patch, moving review requests
Mike Pinkerton <pinkerton@aol.net> has cancelled Torben <torben@despammed.com>'s request for superreview: Bug 90918: RFE: Add url in "comments" field of the "get info" window when download file. https://bugzilla.mozilla.org/show_bug.cgi?id=90918 Attachment 180129: Cleaned up patch, moving review requests https://bugzilla.mozilla.org/attachment.cgi?id=180129&action=edit ...

superreview requested: [Bug 291198] "overflow" and "underflow" events are missing detail attribute : [Attachment 181327] Fix for bug
Nate Nielsen <nielsen@memberwebs.com> has asked David Baron <dbaron@mozillafoundation.org> for superreview: Bug 291198: "overflow" and "underflow" events are missing detail attribute https://bugzilla.mozilla.org/show_bug.cgi?id=291198 Attachment 181327: Fix for bug https://bugzilla.mozilla.org/attachment.cgi?id=181327&action=edit ...

superreview requested: [Bug 257801] Pressing "Cancel" in the "Defer Account?" warning closes the "Advanced Account Settings" dialog : [Attachment 157725] Fix
Stefan Borggraefe <Stefan.Borggraefe@gmx.de> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 257801: Pressing "Cancel" in the "Defer Account?" warning closes the "Advanced Account Settings" dialog http://bugzilla.mozilla.org/show_bug.cgi?id=257801 Attachment 157725: Fix http://bugzilla.mozilla.org/attachment.cgi?id=157725&action=edit ...

superreview requested: [Bug 422009] [FIX]"Assertion failure: !cx->throwing" with XBL constructor that throws and syntax error in event handler : [Attachment 310421] Fix
Boris Zbarsky (reviews very slow until May) <bzbarsky@mit.edu> has asked Johnny Stenback (:jst) <jst@mozilla.org> for superreview: Bug 422009: [FIX]"Assertion failure: !cx->throwing" with XBL constructor that throws and syntax error in event handler https://bugzilla.mozilla.org/show_bug.cgi?id=422009 Attachment 310421: Fix https://bugzilla.mozilla.org/attachment.cgi?id=310421&action=edit ...

superreview requested: [Bug 216237] Stopping event propagation should not cancel system event group handlers : [Attachment 129837] patch
Brian Ryner <bryner@brianryner.com> has asked Johnny Stenback <jst@netscape.com> for superreview: Bug 216237: Stopping event propagation should not cancel system event group handlers http://bugzilla.mozilla.org/show_bug.cgi?id=216237 Attachment 129837: patch http://bugzilla.mozilla.org/attachment.cgi?id=129837&action=edit ...

superreview cancelled: [Bug 234847] "overflow:hidden" doesn't hide events : [Attachment 141791] patch
Bernd <bernd_mozilla@gmx.de> has cancelled Bernd <bernd_mozilla@gmx.de>'s request for superreview: Bug 234847: "overflow:hidden" doesn't hide events http://bugzilla.mozilla.org/show_bug.cgi?id=234847 Attachment 141791: patch http://bugzilla.mozilla.org/attachment.cgi?id=141791&action=edit ...

superreview requested: [Bug 410820] <svg> elements respond to mouse events when they should not : [Attachment 304607] patch for pointer-events="none" case
Jonathan Watt <jwatt@jwatt.org> has asked Robert O'Callahan (:roc) <roc@ocallahan.org> for superreview: Bug 410820: <svg> elements respond to mouse events when they should not https://bugzilla.mozilla.org/show_bug.cgi?id=410820 Attachment 304607: patch for pointer-events="none" case https://bugzilla.mozilla.org/attachment.cgi?id=304607&action=edit ------- Additional Comments from Jonathan Watt <jwatt@jwatt.org> I'd hate to see Firefox 3 go out without at least supporting the pointer-events="none" case. ...

superreview requested: [Bug 299221] [CLEANUP] assign "isChar" inside gecko event conversion : [Attachment 187762] fix v1.0
Josh Aas <joshmoz@gmail.com> has asked Simon Fraser <sfraser_bugs@smfr.org> for superreview: Bug 299221: [CLEANUP] assign "isChar" inside gecko event conversion https://bugzilla.mozilla.org/show_bug.cgi?id=299221 Attachment 187762: fix v1.0 https://bugzilla.mozilla.org/attachment.cgi?id=187762&action=edit ...

superreview canceled: [Bug 406909] check for widget destruction after sending events in scrollwheel and flagsChanged event handlers : [Attachment 291598] fix v1.0
Robert O'Callahan (:roc) <roc@ocallahan.org> has canceled Josh Aas <joshmoz@gmail.com>'s request for superreview: Bug 406909: check for widget destruction after sending events in scrollwheel and flagsChanged event handlers https://bugzilla.mozilla.org/show_bug.cgi?id=406909 Attachment 291598: fix v1.0 https://bugzilla.mozilla.org/attachment.cgi?id=291598&action=edit ...

superreview cancelled: [Bug 299221] [CLEANUP] assign "isChar" inside gecko event conversion : [Attachment 187852] fix v1.1
Mike Pinkerton <pinkerton@aol.net> has cancelled Josh Aas <joshmoz@gmail.com>'s request for superreview: Bug 299221: [CLEANUP] assign "isChar" inside gecko event conversion https://bugzilla.mozilla.org/show_bug.cgi?id=299221 Attachment 187852: fix v1.1 https://bugzilla.mozilla.org/attachment.cgi?id=187852&action=edit ------- Additional Comments from Mike Pinkerton <pinkerton@aol.net> well then... ...

superreview cancelled: [Bug 342715] Need an API to allow extensions to hook "document load started" events : [Attachment 227032] patch
Robert O'Callahan (Novell) <roc@ocallahan.org> has cancelled Robert O'Callahan (Novell) <roc@ocallahan.org>'s request for superreview: Bug 342715: Need an API to allow extensions to hook "document load started" events https://bugzilla.mozilla.org/show_bug.cgi?id=342715 Attachment 227032: patch https://bugzilla.mozilla.org/attachment.cgi?id=227032&action=edit ------- Additional Comments from Robert O'Callahan (Novell) <roc@ocallahan.org> Updated comment. Also, I changed the flags combination to STATE_TRANSFERRING|STATE_IS_REQUEST|S...

superreview cancelled: [Bug 199262] Swap "Mark" and "Junk" Buttons (so that "Next" and "Mark" buttons are next to each other) : [Attachment 118601] patch that swaps the "Mark" and "Junk" buttons
Adam Becevello <bugs@abecevello.imapmail.org> has cancelled Adam Becevello <bugs@abecevello.imapmail.org>'s request for superreview: Bug 199262: Swap "Mark" and "Junk" Buttons (so that "Next" and "Mark" buttons are next to each other) https://bugzilla.mozilla.org/show_bug.cgi?id=199262 Attachment 118601: patch that swaps the "Mark" and "Junk" buttons https://bugzilla.mozilla.org/attachment.cgi?id=118601&action=edit ------- Additional Comments from Adam Becevello <bugs@abecevello.imapmail.org> Remo...

Web resources about - superreview requested: [Bug 345521] "return 0" in handler cancels events : [Attachment 232080] Patch to fix and test event cancel behaviour - mozilla.dev.super-review

Resources last updated: 1/2/2016 9:10:11 AM