refactor: move browser names to a constant variable

Author: Donaldcwl

lib/config/browser-name.js

@@ -0,0 +1,8 @@
+export default {
+  CHROME: 'CHROME',
+  FIREFOX: 'FIREFOX',
+  DESKTOP_SAFARI: 'DESKTOP_SAFARI',
+  IE: 'IE',
+  MOBILE_SAFARI: 'MOBILE_SAFARI',
+  ETC: 'ETC',
+};

lib/config/max-canvas-size.js

@@ -1,10 +1,12 @@
+import BROWSER_NAME from './browser-name';
+
 // see: https://github.com/jhildenbiddle/canvas-size#test-results
 // see: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/canvas#maximum_canvas_size
 export default {
-  chrome: 16384,
-  firefox: 11180,
-  'desktop safari': 16384,
-  'internet explorer': 8192,
-  'mobile safari': 4096,
-  etc: 8192,
+  [BROWSER_NAME.CHROME]: 16384,
+  [BROWSER_NAME.FIREFOX]: 11180,
+  [BROWSER_NAME.DESKTOP_SAFARI]: 16384,
+  [BROWSER_NAME.IE]: 8192,
+  [BROWSER_NAME.MOBILE_SAFARI]: 4096,
+  [BROWSER_NAME.ETC]: 8192,
 };

lib/web-worker.js

@@ -4,7 +4,8 @@ import lib from './index';
 import compress from './image-compression';
 import { getNewCanvasAndCtx, isIOS } from './utils';
 import UPNG from './UPNG';
-import MaxCanvasSize from './config/max-canvas-size';
+import MAX_CANVAS_SIZE from './config/max-canvas-size';
+import BROWSER_NAME from './config/browser-name';
 
 let cnt = 0;
 let imageCompressionLibUrl;
@@ -89,7 +90,8 @@ function generateLib() {
     getNewCanvasAndCtx = ${getNewCanvasAndCtx}
     CustomFileReader = FileReader
     CustomFile = File
-    MaxCanvasSize = ${JSON.stringify(MaxCanvasSize)}
+    MAX_CANVAS_SIZE = ${JSON.stringify(MAX_CANVAS_SIZE)}
+    BROWSER_NAME = ${JSON.stringify(BROWSER_NAME)}
     function compress (){return (${compress}).apply(null, arguments)}
 
     // core-js