add Lib.isSafari

Author: etpinard

src/lib/index.js

@@ -733,6 +733,11 @@ lib.isIE = function() {
     return typeof window.navigator.msSaveBlob !== 'undefined';
 };
 
+var IS_SAFARI_REGEX = /Version\/[\d\.]+.*Safari/;
+lib.isSafari = function() {
+    return IS_SAFARI_REGEX.test(window.navigator.userAgent);
+};
+
 /**
  * Duck typing to recognize a d3 selection, mostly for IE9's benefit
  * because it doesn't handle instanceof like modern browsers

src/snapshot/filesaver.js

@@ -24,18 +24,17 @@
 var fileSaver = function(url, name) {
     var saveLink = document.createElement('a');
     var canUseSaveLink = 'download' in saveLink;
-    var isSafari = /Version\/[\d\.]+.*Safari/.test(navigator.userAgent);
     var promise = new Promise(function(resolve, reject) {
         // IE <10 is explicitly unsupported
         if(typeof navigator !== 'undefined' && /MSIE [1-9]\./.test(navigator.userAgent)) {
             reject(new Error('IE < 10 unsupported'));
         }
 
         // First try a.download, then web filesystem, then object URLs
-        if(isSafari) {
             // Safari doesn't allow downloading of blob urls
             document.location.href = 'data:application/octet-stream' + url.slice(url.search(/[,;]/));
             resolve(name);
+        if(Lib.isSafari()) {
         }
 
         if(!name) {