Fix: Uc browser worker #62

src/Console/stringify.js

@@ -1,6 +1,6 @@
 import stringify from '../lib/stringify';
 import StringifyWorker from '../lib/stringifyWorker';
-import {nextTick, uniqId} from '../lib/util';
+import {nextTick, uniqId, tryIt} from '../lib/util';
 
 let isWorkerSupported = !!window.Worker;
 
@@ -9,21 +9,27 @@ let callbacks = {},
 
 if (isWorkerSupported) 
 {
-    worker = new StringifyWorker();
-    worker.onmessage = function (e) 
+    tryIt(function () 
     {
-        let [id, result] = e.data;
-        if (callbacks[id]) 
+        /* Some browsers like uc mobile doesn't destroy worker properly after refreshing.
+         * After a few times of visiting, it reaches the maximum number of workers per site. 
+         */
+        worker = new StringifyWorker();
+        worker.onmessage = function (e) 
         {
-            callbacks[id](result);
-            delete callbacks[id];
-        }
-    };
+            let [id, result] = e.data;
+            if (callbacks[id]) 
+            {
+                callbacks[id](result);
+                delete callbacks[id];
+            }
+        };
+    });
 }
 
 function exports(obj, options, cb) 
 {
-    let useWorker = exports.useWorker && isWorkerSupported;
+    let useWorker = exports.useWorker && isWorkerSupported && worker;
 
     if (useWorker) 
     {

src/Elements/CssStore.js

@@ -1,4 +1,4 @@
-import {each} from '../lib/util';
+import {each, has} from '../lib/util';
 
 function formatStyle(style)
 {
@@ -47,7 +47,7 @@ export default class CssStore
         each(document.styleSheets, (styleSheet) =>
         {
             // Started with version 64, Chrome does not allow cross origin script to access this property.
-            if (!styleSheet.hasOwnProperty('cssRules')) return;
+            if (!has(styleSheet, 'cssRules')) return;
 
             each(styleSheet.cssRules, (cssRule) =>
             {

src/lib/stringifyUtil.js

@@ -46,11 +46,11 @@ export var idxOf = _.idxOf = (function ()
 {
     /* Get the index at which the first occurrence of value.
      *
-     * |Name       |Type  |Desc                |
-     * |-----------|------|--------------------|
-     * |arr        |array |Array to search     |
-     * |val        |*     |Value to search for |
-     * |[fromIdx=0]|number|Index to search from|
+     * |Name     |Type  |Desc                |
+     * |---------|------|--------------------|
+     * |arr      |array |Array to search     |
+     * |val      |*     |Value to search for |
+     * |fromIdx=0|number|Index to search from|
      *
      * ```javascript
      * idxOf([1, 2, 1, 2], 2, 2); // -> 3
@@ -133,7 +133,7 @@ export var optimizeCb = _.optimizeCb = (function ()
             case 4: return function (accumulator, val, idx, collection)
             {
                 return fn.call(ctx, accumulator, val, idx, collection);
-            }
+            };
         }
 
         return function ()
@@ -312,46 +312,6 @@ export var has = _.has = (function ()
     return exports;
 })();
 
-/* ------------------------------ keys ------------------------------ */
-
-export var keys = _.keys = (function (exports)
-{
-    /* Create an array of the own enumerable property names of object.
-     *
-     * |Name  |Type  |Desc                   |
-     * |------|------|-----------------------|
-     * |obj   |object|Object to query        |
-     * |return|array |Array of property names|
-     * 
-     * ```javascript
-     * keys({a: 1}); // -> ['a']
-     * ```
-     */
-
-    /* module
-     * env: all
-     * test: all
-     */
-
-    /* dependencies
-     * has 
-     */
-
-    exports = Object.keys || function (obj)
-    {
-        var ret = [], key;
-
-        for (key in obj)
-        {
-            if (has(obj, key)) ret.push(key);
-        }
-
-        return ret;
-    };
-
-    return exports;
-})({});
-
 /* ------------------------------ identity ------------------------------ */
 
 export var identity = _.identity = (function ()
@@ -597,6 +557,132 @@ export var isArrLike = _.isArrLike = (function ()
     return exports;
 })();
 
+/* ------------------------------ isBrowser ------------------------------ */
+
+export var isBrowser = _.isBrowser = (function (exports)
+{
+    /* Check if running in a browser.
+     *
+     * ```javascript
+     * console.log(isBrowser); // -> true if running in a browser
+     * ```
+     */
+
+    /* module
+     * env: all
+     * test: all
+     */
+
+    exports = typeof window === 'object' &&
+              typeof document === 'object' &&
+              document.nodeType === 9;
+
+    return exports;
+})({});
+
+/* ------------------------------ root ------------------------------ */
+
+export var root = _.root = (function (exports)
+{
+    /* Root object reference, `global` in nodeJs, `window` in browser. */
+
+    /* module
+     * env: all
+     * test: all
+     */
+
+    /* dependencies
+     * isBrowser 
+     */
+
+    exports = isBrowser ? window : global;
+
+    return exports;
+})({});
+
+/* ------------------------------ detectMocha ------------------------------ */
+
+export var detectMocha = _.detectMocha = (function ()
+{
+    /* Detect if mocha is running.
+     *
+     * ```javascript
+     * detectMocha(); // -> True if mocha is running.
+     * ```
+     */
+
+    /* module
+     * env: all
+     * test: all
+     */
+
+    /* dependencies
+     * root 
+     */ 
+
+    function exports() 
+    {
+        for (var i = 0, len = methods.length; i < len; i++) 
+        {
+            var method = methods[i];
+
+            if (typeof root[method] !== 'function') return false;
+        }
+
+        return true;
+    }
+
+    var methods = ['afterEach','after','beforeEach','before','describe','it'];
+
+    return exports;
+})();
+
+/* ------------------------------ keys ------------------------------ */
+
+export var keys = _.keys = (function (exports)
+{
+    /* Create an array of the own enumerable property names of object.
+     *
+     * |Name  |Type  |Desc                   |
+     * |------|------|-----------------------|
+     * |obj   |object|Object to query        |
+     * |return|array |Array of property names|
+     * 
+     * ```javascript
+     * keys({a: 1}); // -> ['a']
+     * ```
+     */
+
+    /* module
+     * env: all
+     * test: all
+     */
+
+    /* dependencies
+     * has detectMocha 
+     */
+
+    if (Object.keys && !detectMocha()) 
+    {
+        exports = Object.keys;
+    } else 
+    {
+        exports = function (obj)
+        {
+            var ret = [], key;
+
+            for (key in obj)
+            {
+                if (has(obj, key)) ret.push(key);
+            }
+
+            return ret;
+        };
+    }
+
+    return exports;
+})({});
+
 /* ------------------------------ each ------------------------------ */
 
 export var each = _.each = (function ()
@@ -723,37 +809,6 @@ export var extend = _.extend = (function (exports)
     return exports;
 })({});
 
-/* ------------------------------ extendOwn ------------------------------ */
-
-export var extendOwn = _.extendOwn = (function (exports)
-{
-    /* Like extend, but only copies own properties over to the destination object.
-     *
-     * |Name  |Type  |Desc              |
-     * |------|------|------------------|
-     * |obj   |object|Destination object|
-     * |*src  |object|Sources objects   |
-     * |return|object|Destination object|
-     *
-     * ```javascript
-     * extendOwn({name: 'RedHood'}, {age: 24}); // -> {name: 'RedHood', age: 24}
-     * ```
-     */
-
-    /* module
-     * env: all
-     * test: all
-     */
-
-    /* dependencies
-     * keys createAssigner 
-     */
-
-    exports = createAssigner(keys);
-
-    return exports;
-})({});
-
 /* ------------------------------ values ------------------------------ */
 
 export var values = _.values = (function ()
@@ -783,7 +838,7 @@ export var values = _.values = (function ()
     {
         var ret = [];
 
-        each(obj, function (val) { ret.push(val) });
+        each(obj, function (val) { ret.push(val); });
 
         return ret;
     }
@@ -828,6 +883,37 @@ export var contain = _.contain = (function ()
     return exports;
 })();
 
+/* ------------------------------ extendOwn ------------------------------ */
+
+export var extendOwn = _.extendOwn = (function (exports)
+{
+    /* Like extend, but only copies own properties over to the destination object.
+     *
+     * |Name  |Type  |Desc              |
+     * |------|------|------------------|
+     * |obj   |object|Destination object|
+     * |*src  |object|Sources objects   |
+     * |return|object|Destination object|
+     *
+     * ```javascript
+     * extendOwn({name: 'RedHood'}, {age: 24}); // -> {name: 'RedHood', age: 24}
+     * ```
+     */
+
+    /* module
+     * env: all
+     * test: all
+     */
+
+    /* dependencies
+     * keys createAssigner 
+     */
+
+    exports = createAssigner(keys);
+
+    return exports;
+})({});
+
 /* ------------------------------ isStr ------------------------------ */
 
 export var isStr = _.isStr = (function ()
@@ -840,7 +926,7 @@ export var isStr = _.isStr = (function ()
      * |return|boolean|True if value is a string primitive|
      *
      * ```javascript
-     * isStr('eris'); // -> true
+     * isStr('licia'); // -> true
      * ```
      */
 
@@ -1119,7 +1205,7 @@ export var safeCb = _.safeCb = (function (exports)
             return function (obj)
             {
                 return obj == null ? undefined : obj[key];
-            }
+            };
         };
     };