Bug 908954 - Remove try…catch comment inside wrapWidgetEventHandler once bug 503244 is fixed r=MattN
authoraarushivij <aarushivij@gmail.com>
Fri, 17 Apr 2020 05:50:56 +0000
changeset 524550 a427b90947bf6ae2fd8a7ccd8068791ca4c34018
parent 524549 f24466a7ae6a1e9fee588957576fdb57c8bce414
child 524551 5ff2af14358ee98d57e9e06f38724b31c25500a3
push id37323
push userdluca@mozilla.com
push dateFri, 17 Apr 2020 16:25:55 +0000
treeherdermozilla-central@b4b1d6f91ef0 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersMattN
bugs908954, 503244
milestone77.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 908954 - Remove try…catch comment inside wrapWidgetEventHandler once bug 503244 is fixed r=MattN Differential Revision: https://phabricator.services.mozilla.com/D71268
browser/components/customizableui/CustomizableUI.jsm
--- a/browser/components/customizableui/CustomizableUI.jsm
+++ b/browser/components/customizableui/CustomizableUI.jsm
@@ -2849,18 +2849,16 @@ var CustomizableUIInternal = {
   },
 
   wrapWidgetEventHandler(aEventName, aWidget) {
     if (typeof aWidget.implementation[aEventName] != "function") {
       aWidget[aEventName] = null;
       return;
     }
     aWidget[aEventName] = function(...aArgs) {
-      // Wrap inside a try...catch to properly log errors, until bug 862627 is
-      // fixed, which in turn might help bug 503244.
       try {
         // Don't copy the function to the normalized widget object, instead
         // keep it on the original object provided to the API so that
         // additional methods can be implemented and used by the event
         // handlers.
         return aWidget.implementation[aEventName].apply(
           aWidget.implementation,
           aArgs