Merge pull request #127 from jsg2021/make-storage-private-mode-safe

Test the interface before just assuming its gtg.

Author: STRML

Diff for: lib/environment/LocalStorageKeyEnvironment.js

@@ -3,16 +3,28 @@
 var Environment = require('./Environment');
 
 /**
- * Routing environment which stores routing state in localStorage.
- */
+* Routing environment which stores routing state in localStorage.
+*/
 function LocalStorageKeyEnvironment(key) {
   this.key = key;
   var store = this.onStorage = this.onStorage.bind(this);
+  var storage;
 
-  this.storage = (typeof window !== 'undefined' && window.localStorage) || {
+  try {
+    storage = window.localStorage;
+    storage.setItem(key, storage.getItem(key));
+  } catch (e) {
+    storage = null;
+  }
+
+  this.storage = storage || {
     data: {},
     getItem: function(key) {return this.data[key];},
-    setItem: function(key, val) {this.data[key]=val;store();}
+    setItem: function(key, val) {
+      this.data[key]=val;
+      clearTimeout(this.storeEvent);
+      this.storeEvent = setTimeout(store,1);
+    }
   };
 
   Environment.call(this);
@@ -41,13 +53,12 @@ LocalStorageKeyEnvironment.prototype.start = function() {
 
 LocalStorageKeyEnvironment.prototype.stop = function() {
   if (typeof window !== 'undefined' && window.removeEventListener) {
-    window.removeEventListener('storage', this.onStorage);
+    window.removeEventListener('storage', this.onStorage, false);
   }
 };
 
 LocalStorageKeyEnvironment.prototype.onStorage = function() {
   var path = this.getPath();
-
   if (this.path !== path) {
     this.setPath(path, {});
   }