Bug 1493121 - Remove unnecessary use of `wrappedJSObject` in JsAccount's factory. r+a=jorgk
authorNeil Rashbrook <neil@parkwaycc.co.uk>
Fri, 21 Sep 2018 04:16:00 +0200
changeset 31854 1615e21f1be9bf032b68401d62f2e43f6bf64972
parent 31853 9976c780c36673c9f2bd3f0781b0331e0265dc82
child 31855 01d10b64821078746e600ea322d3f48b23f735f5
push id86
push usermozilla@jorgk.com
push dateSun, 14 Oct 2018 09:47:49 +0000
treeherdercomm-esr60@e524897f5bde [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1493121
Bug 1493121 - Remove unnecessary use of `wrappedJSObject` in JsAccount's factory. r+a=jorgk
mailnews/jsaccount/modules/JSAccountUtils.jsm
--- a/mailnews/jsaccount/modules/JSAccountUtils.jsm
+++ b/mailnews/jsaccount/modules/JSAccountUtils.jsm
@@ -114,17 +114,17 @@ JSAccountUtils.jaFactory = function (aPr
       delegator.methodsToDelegate = delegateList;
     } else {
       // Lazily create and populate the list of methods to delegate.
       log.info("creating delegate list for contractID " + aProperties.contractID);
       let delegateList = delegator.methodsToDelegate;
       Object.keys(delegator).forEach(name => {log.debug("delegator has key " + name);});
 
       // jsMethods contains the methods that may be targets of the C++ delegation to JS.
-      let jsMethods = Object.getPrototypeOf(delegator.jsDelegate.wrappedJSObject);
+      let jsMethods = Object.getPrototypeOf(jsDelegate);
       for (let name in jsMethods)
       {
         log.debug("processing jsDelegate method: " + name);
         if (name[0] == '_') { // don't bother with methods explicitly marked as internal.
           log.debug("skipping " + name);
           continue;
         }
         // Other methods to skip.