Working ServiceWorker

js/src/redux/providers/compilerActions.js

@@ -25,29 +25,15 @@ if ('serviceWorker' in navigator) {
     .register()
     .then(() => {
       console.log('registering service worker');
-
-      if (navigator.serviceWorker.controller) {
-        // already active and controlling this page
-        return navigator.serviceWorker;
-      }
-      // wait for a new service worker to control this page
-      return new Promise((resolve, reject) => {
-        try {
-          const onControllerChange = () => {
-            navigator.serviceWorker.removeEventListener('controllerchange', onControllerChange);
-            resolve(navigator.serviceWorker);
-          };
-
-          navigator.serviceWorker.addEventListener('controllerchange', onControllerChange);
-        } catch (error) {
-          reject(error);
-        }
-      });
+      return navigator.serviceWorker.ready;
     })
-    .then((_worker) => {
+    .then((registration) => {
+      console.log('registered service worker');
+
+      const _worker = registration.active;
+      _worker.controller = registration.active;
       const worker = new PromiseWorker(_worker);
 
-      console.log('registered service worker');
       return worker;
     });
 } else {

js/src/serviceWorker.js

@@ -25,10 +25,12 @@ registerPromiseWorker((msg) => {
 });
 
 self.addEventListener('install', (event) => {
+  console.warn('installing sw');
   event.waitUntil(self.skipWaiting());
 });
 
 self.addEventListener('activate', (event) => {
+  console.warn('activating sw');
   event.waitUntil(self.clients.claim());
 });
 
@@ -141,18 +143,19 @@ function fetchSolc (build) {
 
       return fetch(URL)
         .then((response) => {
-          if (!response || response.status !== 200 || response.type !== 'basic') {
+          if (!response || response.status !== 200) {
             return response;
           }
 
           const responseToCache = response.clone();
 
-          caches.open(CACHE_NAME)
+          return caches.open(CACHE_NAME)
             .then((cache) => {
-              cache.put(URL, responseToCache);
+              return cache.put(URL, responseToCache);
+            })
+            .then(() => {
+              return response;
             });
-
-          return response;
         });
     });
 }

js/src/views/WriteContract/writeContractStore.js

@@ -74,6 +74,7 @@ export default class WriteContractStore {
 
   @observable workerError = null;
 
+  loadingSolidity = false;
   lastCompilation = {};
   snippets = SNIPPETS;
   worker = null;
@@ -147,7 +148,11 @@ export default class WriteContractStore {
       return;
     }
 
-    return this.worker
+    if (this.loadingSolidity) {
+      return this.loadingSolidity;
+    }
+
+    this.loadingSolidity = this.worker
       .postMessage({
         action: 'load',
         data: build
@@ -161,8 +166,11 @@ export default class WriteContractStore {
         this.setWorkerError(error);
       })
       .then(() => {
+        this.loadingSolidity = false;
         this.loading = false;
       });
+
+    return this.loadingSolidity;
   }
 
   @action handleOpenDeployModal = () => {
@@ -243,18 +251,20 @@ export default class WriteContractStore {
         });
     }
 
-    return promise.then((data = {}) => {
-      const {
-        contract, contractIndex,
-        annotations, contracts, errors
-      } = data.result;
+    return promise.then((data = null) => {
+      if (data) {
+        const {
+          contract, contractIndex,
+          annotations, contracts, errors
+        } = data.result;
 
-      this.contract = contract;
-      this.contractIndex = contractIndex;
+        this.contract = contract;
+        this.contractIndex = contractIndex;
 
-      this.annotations = annotations;
-      this.contracts = contracts;
-      this.errors = errors;
+        this.annotations = annotations;
+        this.contracts = contracts;
+        this.errors = errors;
+      }
 
       this.compiled = true;
       this.compiling = false;

js/webpack/app.js

@@ -166,7 +166,7 @@ module.exports = {
       }),
 
       new ServiceWorkerWebpackPlugin({
-        entry: path.join(__dirname, '../src/serviceWorker.js'),
+        entry: path.join(__dirname, '../src/serviceWorker.js')
       }),
 
       DappsHTMLInjection