add comments and drop unused logic for IE9 and IE10

devtools/test_dashboard/index.html

@@ -19,10 +19,6 @@
   </div>
   <div id="snapshot"></div>
 
-  <!-- uncomment below for IE9/10 support -->
-  <!-- <script>if(typeof window.Int16Array !== 'function')document.write("<scri"+"pt src='../../dist/extras/typedarray.min.js'></scr"+"ipt>");</script>
-  <script>document.write("<scri"+"pt src='../../dist/extras/request_animation_frame.js'></scr"+"ipt>");</script> -->
-
   <script src="../../dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
   <script id="source" src="../../build/plotly.js" charset="utf-8"></script>
   <script src="../../build/test_dashboard-bundle.js" charset="utf-8"></script>

dist/README.md

@@ -14,15 +14,6 @@ or the un-minified version as:
 <script src="plotly.js" charset="utf-8"></script>
 ```
 
-### To support IE9
-
-*Before* the plotly.js script tag, add:
-
-```html
-<script>if(typeof window.Int16Array !== 'function')document.write("<scri"+"pt src='extras/typedarray.min.js'></scr"+"ipt>");</script>
-<script>document.write("<scri"+"pt src='extras/request_animation_frame.js'></scr"+"ipt>");</script>
-```
-
 ### To support MathJax
 
 *Before* the plotly.js script tag, add:

src/lib/array.js

@@ -10,15 +10,8 @@
 
 var isArray = Array.isArray;
 
-// IE9 fallbacks
-
-var ab = (typeof ArrayBuffer === 'undefined' || !ArrayBuffer.isView) ?
-    {isView: function() { return false; }} :
-    ArrayBuffer;
-
-var dv = (typeof DataView === 'undefined') ?
-    function() {} :
-    DataView;
+var ab = ArrayBuffer;
+var dv = DataView;
 
 function isTypedArray(a) {
     return ab.isView(a) && !(a instanceof dv);

src/lib/index.js

@@ -712,11 +712,6 @@ lib.isIE = function() {
     return typeof window.navigator.msSaveBlob !== 'undefined';
 };
 
-var IS_IE9_OR_BELOW_REGEX = /MSIE [1-9]\./;
-lib.isIE9orBelow = function() {
-    return lib.isIE() && IS_IE9_OR_BELOW_REGEX.test(window.navigator.userAgent);
-};
-
 var IS_SAFARI_REGEX = /Version\/[\d\.]+.*Safari/;
 lib.isSafari = function() {
     return IS_SAFARI_REGEX.test(window.navigator.userAgent);
@@ -727,6 +722,8 @@ lib.isIOS = function() {
     return IS_IOS_REGEX.test(window.navigator.userAgent);
 };
 
+// Do we need this process now that IE9 and IE10 are not supported?
+
 /**
  * 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

@@ -31,10 +31,6 @@ function fileSaver(url, name, format) {
         var blob;
         var objectUrl;
 
-        if(Lib.isIE9orBelow()) {
-            reject(new Error('IE < 10 unsupported'));
-        }
-
         // Safari doesn't allow downloading of blob urls
         if(Lib.isSafari()) {
             var prefix = format === 'svg' ? ',' : ';base64,';

src/snapshot/svgtoimg.js

@@ -45,7 +45,7 @@ function svgToImg(opts) {
         var img = new Image();
         var svgBlob, url;
 
-        if(format === 'svg' || Lib.isIE9orBelow() || Lib.isSafari()) {
+        if(format === 'svg' || Lib.isSafari()) {
             url = helpers.encodeSVG(svg);
         } else {
             svgBlob = helpers.createBlob(svg, 'svg');

src/snapshot/tosvg.js

@@ -164,6 +164,8 @@ module.exports = function toSVG(gd, format, scale) {
     // Fix quotations around font strings and gradient URLs
     s = s.replace(DUMMY_REGEX, '\'');
 
+    // Do we need this process now that IE9 and IE10 are not supported?
+
     // IE is very strict, so we will need to clean
     //  svg with the following regex
     //  yes this is messy, but do not know a better way

tasks/stats.js

@@ -46,15 +46,6 @@ function getInfoContent() {
         '<script src="plotly.js" charset="utf-8"></script>',
         '```',
         '',
-        '### To support IE9',
-        '',
-        '*Before* the plotly.js script tag, add:',
-        '',
-        '```html',
-        '<script>if(typeof window.Int16Array !== \'function\')document.write("<scri"+"pt src=\'extras/typedarray.min.js\'></scr"+"ipt>");</script>',
-        '<script>document.write("<scri"+"pt src=\'extras/request_animation_frame.js\'></scr"+"ipt>");</script>',
-        '```',
-        '',
         '### To support MathJax',
         '',
         '*Before* the plotly.js script tag, add:',

test/image/strict-d3.js

@@ -2,6 +2,9 @@
  * strict-d3: wrap selection.style to prohibit specific incorrect style values
  * that are known to cause problems in IE (at least IE9)
  */
+
+// Do we need this process now that IE9 and IE10 are not supported?
+
 'use strict';
 
 var d3 = require('@plotly/d3');

test/jasmine/karma.conf.js

@@ -48,9 +48,6 @@ if(argv.info) {
         'Run all tests with the `noCI` tag on Firefox in a 1500px wide window:',
         '  $ npm run test-jasmine -- --tags=noCI --FF --width=1500',
         '',
-        'Run the `ie9_test.js` bundle test with the verbose reporter:',
-        '  $ npm run test-jasmine -- --bundleTest=ie9 --verbose',
-        '',
         'Arguments:',
         '  - All non-flagged arguments corresponds to the test suites in `test/jasmine/tests/` to be run.',
         '    No need to add the `_test.js` suffix, we expand them correctly here.',
@@ -122,7 +119,6 @@ if(isFullSuite) {
 var pathToShortcutPath = path.join(__dirname, '..', '..', 'tasks', 'util', 'shortcut_paths.js');
 var pathToStrictD3 = path.join(__dirname, '..', '..', 'tasks', 'util', 'strict_d3.js');
 var pathToJQuery = path.join(__dirname, 'assets', 'jquery-1.8.3.min.js');
-var pathToIE9mock = path.join(__dirname, 'assets', 'ie9_mock.js');
 var pathToCustomMatchers = path.join(__dirname, 'assets', 'custom_matchers.js');
 var pathToUnpolyfill = path.join(__dirname, 'assets', 'unpolyfill.js');
 var pathToMathJax = path.join(constants.pathToDist, 'extras', 'mathjax');
@@ -323,13 +319,6 @@ if(isBundleTest) {
             func.defaultConfig.files.push(constants.pathToPlotlyDistMin);
             func.defaultConfig.preprocessors[testFileGlob] = ['browserify'];
             break;
-        case 'ie9':
-            // load ie9_mock.js before plotly.js+test bundle
-            // to catch reference errors that could occur
-            // when plotly.js is first loaded.
-            func.defaultConfig.files.push(pathToIE9mock);
-            func.defaultConfig.preprocessors[testFileGlob] = ['browserify'];
-            break;
         case 'plotschema':
             func.defaultConfig.browserify.ignoreTransform = './tasks/compress_attributes.js';
             func.defaultConfig.preprocessors[testFileGlob] = ['browserify'];