fix for...of return issue

src/gmail.js

@@ -749,7 +749,7 @@
                 "false": !1,
                 "null": null
             };
         each(params.replace(/\+/g, " ").split("&"), function (v, j) {
             var param = v.split("="),
                 key = decodeURIComponent(param[0]),
                 val,
@@ -800,7 +800,7 @@
         }
     };
 
     api.tools.parse_actions = function(params, xhr) {
 
         // upload_attachment event - if found, don"t check other observers. See issue #22
         if(params.url.act === "fup" || params.url.act === "fuv" || params.body_is_object) {
@@ -1918,7 +1918,7 @@
         }
 
         let parsedResponse = [];
         let originalResponse = response;
         try {
             // gmail post response structure
             // )}]"\n<datalength><rawData>\n<dataLength><rawData>...
@@ -2097,7 +2097,7 @@
         const win = api.helper.get_xhr_window();
 
         api.tools.patch(win.XMLHttpRequest.prototype.open, (orig) => {
             win.XMLHttpRequest.prototype.open = function (method, url, async, user, password) {
                 var out = orig.apply(this, arguments);
                 this.xhrParams = {
                     method: method.toString(),
@@ -2389,7 +2389,7 @@
         // loop through applicable types
         var types = type ? [ type ] : [ "before", "on", "after", "dom" ];
         for (let type of types) {
-            if(typeof api.tracker.watchdog[type] !== "object") return true; // no callbacks for this type
+            if(typeof api.tracker.watchdog[type] !== "object") continue; // no callbacks for this type
 
             // if action specified, remove any callbacks for this action, otherwise remove all callbacks for all actions
             if(action) {
@@ -2756,7 +2756,7 @@
 
     // observes every element inserted into the DOM by Gmail and looks at the classes on those elements,
     // checking for any configured observers related to those classes
     api.tools.insertion_observer = function(target, dom_observers, dom_observer_map, sub) {
         //console.log("insertion", target, target.className);
         if(!dom_observer_map) return;
 
@@ -2767,7 +2767,7 @@
         for (let className of classes) {
             var observers = dom_observer_map[className];
             if (!observers) {
-                return;
+                continue;
             }
 
             for (var observer of observers) {
@@ -2779,7 +2779,7 @@
 
                     // if a config id specified for this observer, ensure it matches for this element
                     if(config.selector && !element.is(config.selector)) {
-                        return;
+                        break;
                     }
 
                     // check for any defined sub_selector match - if not found, then this is not a match for this observer