Update a bunch of stuff for 2.0.0

.travis.yml

@@ -9,11 +9,11 @@ env:
   matrix:
   - COMMAND=test
   - COMMAND=test-browser
-  - COMMAND=test-phantom
   - COMMAND=report-coverage
   global:
   - secure: ug76MyYP5+/TL0HOJVCg7dbB2lyN8efBoJ9CjRQxlkZbiOOXVXMEi86wifAe+xeZVnCrB0S+2EX0jFtezmg9az3f1ktuN5TcjSWPJhtM8MpqX0RBSGvQtNY5GeDfsQcVWL4C4eC/zADBPLAvdznsE5OlFYrO9TjSXiPwekabU8L9Oxk7Yq0yZ/WhSGooNL9B/yaGO/inRDa65Ch7yZwoND6L8UmF3R12FXQM6Sn6OAhUgWRKfYPjDfCfv8pXGLNYkllv9pCEj9C65+sP7k6OA5L7pBLQj6S6Bz0n7XIpaGfSHgnTLOvd4f6ebrLDa71+b/bSagNMKTuQ3pGvEibKtwBQv+HF2qHhf4rnVrUFSYaXFSvjIfN/GCO+DI1B5o8T6y2Gyy2wSPoLxbRa0sZiHRuHBpsS3WrAwxBKYyDMof5nVkBBje6Ih5IiwKKbsvCXhocEwX1CKXsqXHd/hopCOQra2v5PVUTBmjV2LdV1CsVIKXm0JYr0VE3ab7PMHpgjtUHgDP1cclGZqTAi8PtfWd98apMUbXoKqXW9v9+f6ARtz+IfHLtPqrjTe2dB38SnVsZqrsopiqxTVKgTcz1vY3FA8Wg3HLOuhD529a9sed/gYCxA/u+QSAo4WuysqWqf6sUyq6uSBFM2ClzdBTgIxcltgWrB4y35kFav1+WNtsQ=
   - secure: IaeC5qFM1EarirH1n5gARuuGG4OQBEccpjmZz34PyhbJyEB9wOmwgbsMrggDiu7bUAJTx6WPIzVagYiJyK/ATbhsZ0KJUSjXqErVyIE9+gn5PDm74/Y8sK6g1gUlqNml6D7D6xyx0DGgAQqJcqBxOTID49byAYFNSvHN+H8Rjqn6lyPnEUl0v8ptPyi/am6clCTT49EVnaQ077o/aZdkau/1gzDSKHS+9iKku89GHC9iAdBbZIt9gJtbnBjJM7bV8kIie7yUL2o6kEfPkciglt2PdpTLwxAPWXuVizHVh/BzFVHN8Pn6JiDOu9cwtnzu7NxLrOebZ8ItVsSC1uJKAU5qwUmusZaaR1RJKiccIfCdZIekI9AQx3cu/cccWrPa9MA45Wwm2ZA5EFpMkQispjAdtkIWp3ZD7xR/Dg5yXnGal6e+ARx8h2fHYMONxRcAYNXiD/iJfTO42BTjgOAVGfS18y+NX7LkMujtxTBrwt7it/hDrewDKrmod0xo734BShMaysOOv0HSLKPH3ChOQglmzhvdAyFGtRwBtyS+rTzk8HmSYf0nS0MCvRGOm04ZppsV7F/ncpgfWudCa80zfLfPIWIRdIH4WKr4lWimA+bBXrojJbvjBqM80JofAvdWR+I/fnZI5GzTIXWoQ0+MBKCzhDBO4L4aOisixkLeEg0=
+  - secure: qA/t6j6P+Y/m0U1B5Ht9LPl/KMTU3L4v13tZIUIDP8ULplzr8A7nzAF9y1lIsThi/3941m+vAYU+AEcnGn3BSlDXh3XoTwOfUZJtgwL2SJyNZaekOHtK/CLNDKPhjL1jSSPwEl4K9rUUjMNJvF+3RzuJHpNHsjzYWS/PWcC0W9fZR2mapYSUmRzpgcYRM3821kpCd2eN++rhByqr/By2Z/2hdHD9iVTHo5DMHkGQmTCAqw1uabMhvmsDHod2r85dEFbDFaDeJWrtJNxhKhU2Jbr0TSFsoi+3oF2e2A0MP82waRvDKlB2UKmJu86m1mcPjVcidXFss1kIWsfIX+sCrLcd6FCzlh4veMGxvYnQ049aUnJa1R85p04Xv4m0yrR18Ay1IPAruyX4ojTFbRjdjUZOi/KZKJoN0Om7N+h8F7sf4BXWWWHuugNFoUtqPw3Qqxys+GeGPGXlXL99sLk9C1+Xo/O7XSZG+ZtNk5woEuq7stYZ8E4wddfWDvfokIszCHeknGCpVLJiKC3zDjV8slQbn6gL/qv/NXEvQPRMzXLwOAd8AOOCa73nT9h6T8IQr/0QV8IUD5Ec0epxIZRg1Kp0kMd5P8fWOBviXdFAoCbdi6pDAAZtpZD8hCgJ3guktabXXT25C+xSf2s1w4oAwrXpyHHY9QkHNnlGj1Meh74=
 branches:
   only:
   - master

.zuul.yml

@@ -1,21 +1,19 @@
 ui: mocha-bdd
+concurrency: 1
 tunnel:
-  type: localtunnel
-  https: true
+  type: ngrok
 browsers:
   - name: chrome
-    version: 42..latest
+    version: latest
   - name: firefox
-    version: 45..latest
+    version: latest
   - name: safari
-    version: [8, 9..latest]
+    version: latest
   - name: ie
     version: 10..latest
   - name: microsoftedge
-    version: 13..latest
+    version: latest
   - name: iphone
-    version: [8.4, 9.2]
-  - name: ipad
-    version: [8.4, 9.2]
+    version: latest
   - name: android
-    version: 4.4..latest
+    version: 4.4

CHANGELOG.md

@@ -0,0 +1,13 @@
+Changelog
+====
+
+v2.0.0
+---
+
+- Stopped stringifying worker messages ([#14](https://github.com/nolanlawson/promise-worker/pull/14))
+- Removed built-in Promise polyfill
+
+v1.0.0
+---
+
+- Initial release

README.md

@@ -5,9 +5,8 @@ A small and performant library for communicating with Web Workers or Service Wor
 
 **Goals:**
 
- * Tiny footprint (~2.5kB min+gz)
+ * Tiny footprint (~1.4kB min+gz)
  * Assumes you have a separate `worker.js` file (easier to debug, better browser support)
- * `JSON.stringify`s messages [for performance](http://nolanlawson.com/2016/02/29/high-performance-web-worker-messages/)
 
 **Live examples:**
 
@@ -202,8 +201,12 @@ self.addEventListener('activate', function(event) {
 Browser support
 ----
 
+Note that as of v2.0.0, `promise-worker` does not contain a built-in Promise 
+polyfill. Use something like [es6-promise](https://github.com/stefanpenner/es6-promise) 
+if you need to support [browsers that don't support Promises](https://caniuse.com/#feat=promises).
+
 See [.zuul.yml](https://github.com/nolanlawson/promise-worker/blob/master/.zuul.yml) for the full list
-of tested browsers, but basically:
+of tested browsers. Assuming you have a Promise polyfill, the supported browsers should be:
 
 * Chrome
 * Firefox
@@ -269,10 +272,6 @@ Or to test in a browser using SauceLabs:
 
     npm run test-browser
 
-Or to test in PhantomJS:
-
-    npm run test-phantom
-
 Or to test with coverage reports:
 
     npm run coverage

bin/build-worker-bundles.js

@@ -1,34 +1,34 @@
-'use strict';
+'use strict'
 
-var Promise = require('lie');
-var denodeify = require('denodeify');
-var rimraf = denodeify(require('rimraf'));
-var mkdirp = denodeify(require('mkdirp'));
-var glob = require('glob-promise');
-var streamToPromise = require('stream-to-promise');
-var browserify = require('browserify');
-var fs = require('fs');
-var writeFile = denodeify(fs.writeFile);
-var path = require('path');
+var Promise = require('lie')
+var denodeify = require('denodeify')
+var rimraf = denodeify(require('rimraf'))
+var mkdirp = denodeify(require('mkdirp'))
+var glob = require('glob-promise')
+var streamToPromise = require('stream-to-promise')
+var browserify = require('browserify')
+var fs = require('fs')
+var writeFile = denodeify(fs.writeFile)
+var path = require('path')
 
 Promise.resolve().then(function () {
-  return rimraf('test/bundle');
+  return rimraf('test/bundle')
 }).then(function () {
-  return mkdirp('test/bundle');
+  return mkdirp('test/bundle')
 }).then(function () {
-  return glob('test/worker*js');
+  return glob('test/worker*js')
 }).then(function (files) {
   return Promise.all(files.map(function (file) {
-    var b = browserify(file, {debug: true});
+    var b = browserify(file, { debug: true })
     if (process.env.COVERAGE === '1') {
-      b = b.transform('istanbulify');
+      b = b.transform('istanbulify')
     }
-    b = b.bundle();
+    b = b.bundle()
     return streamToPromise(b).then(function (buff) {
-      var outputFile = 'bundle-' + path.basename(file);
-      return writeFile(outputFile, buff, 'utf-8');
-    });
-  }));
+      var outputFile = 'bundle-' + path.basename(file)
+      return writeFile(outputFile, buff, 'utf-8')
+    })
+  }))
 }).catch(function (err) {
-  console.log(err.stack);
-});
+  console.log(err.stack)
+})

index.js

@@ -1,70 +1,67 @@
-'use strict';
+'use strict'
 
-/* istanbul ignore next */
-var MyPromise = typeof Promise !== 'undefined' ? Promise : require('lie');
+var messageIds = 0
 
-var messageIds = 0;
-
-function onMessage(self, e) {
-  var message = e.data;
+function onMessage (self, e) {
+  var message = e.data
   if (!Array.isArray(message) || message.length < 2) {
     // Ignore - this message is not for us.
-    return;
+    return
   }
-  var messageId = message[0];
-  var error = message[1];
-  var result = message[2];
+  var messageId = message[0]
+  var error = message[1]
+  var result = message[2]
 
-  var callback = self._callbacks[messageId];
+  var callback = self._callbacks[messageId]
 
   if (!callback) {
     // Ignore - user might have created multiple PromiseWorkers.
     // This message is not for us.
-    return;
+    return
   }
 
-  delete self._callbacks[messageId];
-  callback(error, result);
+  delete self._callbacks[messageId]
+  callback(error, result)
 }
 
-function PromiseWorker(worker) {
-  var self = this;
-  self._worker = worker;
-  self._callbacks = {};
+function PromiseWorker (worker) {
+  var self = this
+  self._worker = worker
+  self._callbacks = {}
 
   worker.addEventListener('message', function (e) {
-    onMessage(self, e);
-  });
+    onMessage(self, e)
+  })
 }
 
 PromiseWorker.prototype.postMessage = function (userMessage) {
-  var self = this;
-  var messageId = messageIds++;
+  var self = this
+  var messageId = messageIds++
 
-  var messageToSend = [messageId, userMessage];
+  var messageToSend = [messageId, userMessage]
 
-  return new MyPromise(function (resolve, reject) {
+  return new Promise(function (resolve, reject) {
     self._callbacks[messageId] = function (error, result) {
       if (error) {
-        return reject(new Error(error.message));
+        return reject(new Error(error.message))
       }
-      resolve(result);
-    };
+      resolve(result)
+    }
 
     /* istanbul ignore if */
     if (typeof self._worker.controller !== 'undefined') {
       // service worker, use MessageChannels because e.source is broken in Chrome < 51:
       // https://bugs.chromium.org/p/chromium/issues/detail?id=543198
-      var channel = new MessageChannel();
+      var channel = new MessageChannel()
       channel.port1.onmessage = function (e) {
-        onMessage(self, e);
-      };
-      self._worker.controller.postMessage(messageToSend, [channel.port2]);
+        onMessage(self, e)
+      }
+      self._worker.controller.postMessage(messageToSend, [channel.port2])
     } else {
       // web worker
-      self._worker.postMessage(messageToSend);
+      self._worker.postMessage(messageToSend)
     }
-  });
-};
+  })
+}
 
-module.exports = PromiseWorker;
+module.exports = PromiseWorker