Closed Bug 845336 Opened 11 years ago Closed 6 years ago

No javascript error reporting when clicking on hook added to menuitem

Categories

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

defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: h4writer, Unassigned)

Details

Error reporting not working when error occurs during "onclick" on "menuitem". Testcase is reduced version from production code used by my brother at "uwsoftware.be". I owe him, because he reduced this from 30000+ js lines (took him a day or something).

<!DOCTYPE html>
<html>
 <head>
 </head
 <body contextmenu="editMenu">
  <menu type='context' id='editMenu'>
   <menuitem label='Edit' id='__editor_edit_menu' />
  </menu>

  test

  <script type="text/javascript">
    document.getElementById("__editor_edit_menu").addEventListener("click", function(e) {
      document.body.innerHTML = "1"; 
      console.log(prev); 
      document.body.innerHTML += "2";
    });
  </script>
 </body>
</html>

Howto reproduce: click right on "test", click on the contextitem "Edit", the document will show "1" and should report an error "ReferenceError: prev is not defined". The error isn't reported in FF/Aurora/Nightly!!!!

I'm not familiar with this code (IonMonkey developer), but like I said I owe it to him. So could anybody with the knowledge look into this or atleast give me some pointers to where I should look etc...
This is likely bug 503244, which will be solved by switching everything to WebIDL.
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.
Priority: -- → P5
(In reply to Josh Matthews [:jdm] from comment #1)
> This is likely bug 503244, which will be solved by switching everything to
> WebIDL.

This has been fixed a while, and indeed I can't reproduce this anymore - http://jsbin.com/zepajawifo/edit?html,js,output
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.