refactor(*): remove workarounds for IE <9, log all parameters in IE 9

mgol · mgol · commit 8ec4ae5b6357 · 2017-04-12T15:17:52.000+02:00
1. Remove remanining workarounds for IE &lt;9
2. Log all parameters in IE 9, not just the first two.
3. Update IE/Edge-related support comments.
4. ES6 classes now require Edge 14 or newer to work.
diff --git a/.github/ISSUE_TEMPLATE.md b/.github/ISSUE_TEMPLATE.md
@@ -29,8 +29,8 @@ https://plnkr.co or similar (you can use this template as a starting point: http
 **Angular version:** 1.x.y
 <!-- Check whether this is still an issue in the most recent stable or in the snapshot AngularJS version (https://code.angularjs.org/snapshot/) -->
 
-**Browser:** [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
+**Browser:** [all | Chrome XX | Firefox XX | Edge XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
 <!-- All browsers where this could be reproduced (and Operating System if relevant) -->
 
 **Anything else:**
-<!-- e.g. stacktraces, related issues, suggestions how to fix -->
+<!-- e.g. stacktraces, related issues, suggestions how to fix -->
diff --git a/docs/config/templates/ngdoc/api/directive.template.html b/docs/config/templates/ngdoc/api/directive.template.html
@@ -18,9 +18,6 @@ <h2 id="usage">Usage</h2>
     <ul>
     {% if doc.restrict.element %}
       <li>as element:
-      {% if doc.name.indexOf('ng') == 0 -%}
-      (This directive can be used as custom element, but be aware of <a href="guide/ie">IE restrictions</a>).
-      {%- endif %}
       {% code %}
       <{$ doc.name | dashCase $}
         {%- for param in doc.params %}
diff --git a/docs/content/guide/bootstrap.ngdoc b/docs/content/guide/bootstrap.ngdoc
@@ -40,8 +40,8 @@ initialization.
         <html ng-app>
 
   3. If you choose to use the old style directive syntax `ng:` then include xml-namespace in `html`
-    to make IE happy. (This is here for historical reasons, and we no longer recommend use of
-    `ng:`.)
+    when running the page in the XHTML mode. (This is here for historical reasons, and we no longer
+    recommend use of `ng:`.)
 
         <html xmlns:ng="http://angularjs.org">
 
diff --git a/docs/content/guide/ie.ngdoc b/docs/content/guide/ie.ngdoc
@@ -7,7 +7,7 @@
 
 <div class="alert alert-warning">
 **Note:** AngularJS 1.3 has dropped support for IE8. Read more about it on
-[our blog](http://blog.angularjs.org/2013/12/angularjs-13-new-release-approaches.html).
+[our blog](https://blog.angularjs.org/2013/12/angularjs-13-new-release-approaches.html).
 AngularJS 1.2 will continue to support IE8, but the core team does not plan to spend time
 addressing issues specific to IE8 or earlier.
 </div>
@@ -19,16 +19,16 @@ on IE.
 The project currently supports and will attempt to fix bugs for IE9 and above. The continuous
 integration server runs all the tests against IE9, IE10, and IE11. See
 [Travis CI](https://travis-ci.org/angular/angular.js) and
-[ci.angularjs.org](http://ci.angularjs.org).
+[ci.angularjs.org](https://ci.angularjs.org).
 
 We do not run tests on IE8 and below. A subset of the AngularJS functionality may work on these
 browsers, but it is up to you to test and decide whether it works for your particular app.
 
 
 To ensure your AngularJS application works on IE please consider:
 
-1. Use `ng-style` tags instead of `style="{{ someCss }}"`. The latter works in Chrome and Firefox
-   but does not work in Internet Explorer <= 11 (the most recent version at time of writing).
+1. Use `ng-style` tags instead of `style="{{ someCss }}"`. The latter works in Chrome, Firefox,
+   Safari and Edge but does not work in Internet Explorer (even 11).
 2. For the `type` attribute of buttons, use `ng-attr-type` tags instead of
    `type="{{ someExpression }}"`. If using the latter, Internet Explorer overwrites the expression
    with `type="submit"` before AngularJS has a chance to interpolate it.
diff --git a/docs/content/misc/faq.ngdoc b/docs/content/misc/faq.ngdoc
@@ -142,10 +142,8 @@ We run our extensive test suite against the following browsers: the latest versi
 Firefox, Safari, and Safari for iOS, as well as Internet Explorer versions 9-11. See
 {@link guide/ie Internet Explorer Compatibility} for more details on supporting legacy IE browsers.
 
-If a browser is untested, it doesn't mean it won't work; for example, older Android (2.3.x)
-is supported in the sense that we avoid the dot notation for reserved words as property names,
-but we don't actively test changes against it. You can also expect browsers to work that share
-a large part of their codebase with a browser we test, such as Opera > version 12
+If a browser is untested, it doesn't mean it won't work. You can also expect browsers to work that
+share a large part of their codebase with a browser we test, such as Opera 15 or newer
 (uses the Blink engine), or the various Firefox derivatives.
 
 
diff --git a/src/Angular.js b/src/Angular.js
@@ -1353,7 +1353,7 @@ function fromJson(json) {
 
 var ALL_COLONS = /:/g;
 function timezoneToOffset(timezone, fallback) {
-  // Support: IE 9-11 only, Edge 13-14+
+  // Support: IE 9-11 only, Edge 13-15+
   // IE/Edge do not "understand" colon (`:`) in timezone
   timezone = timezone.replace(ALL_COLONS, '');
   var requestedTimezoneOffset = Date.parse('Jan 01, 1970 00:00:00 ' + timezone) / 60000;
@@ -1380,12 +1380,7 @@ function convertTimezoneToLocal(date, timezone, reverse) {
  * @returns {string} Returns the string representation of the element.
  */
 function startingTag(element) {
-  element = jqLite(element).clone();
-  try {
-    // turns out IE does not let you set .html() on elements which
-    // are not allowed to have children. So we just ignore it.
-    element.empty();
-  } catch (e) { /* empty */ }
+  element = jqLite(element).clone().empty();
   var elemHtml = jqLite('<div>').append(element).html();
   try {
     return element[0].nodeType === NODE_TYPE_TEXT ? lowercase(elemHtml) :
@@ -1523,6 +1518,7 @@ function allowAutoBootstrap(document) {
   var script = document.currentScript;
 
   if (!script) {
+    // Support: IE 9-11 only
     // IE does not have `document.currentScript`
     return true;
   }
diff --git a/src/auto/injector.js b/src/auto/injector.js
@@ -876,9 +876,7 @@ function createInjector(modulesToLoad, strictDi) {
       }
       var result = func.$$ngIsClass;
       if (!isBoolean(result)) {
-        // Support: Edge 12-13 only
-        // See: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/6156135/
-        result = func.$$ngIsClass = /^(?:class\b|constructor\()/.test(stringifyFn(func));
+        result = func.$$ngIsClass = /^class\b/.test(stringifyFn(func));
       }
       return result;
     }
diff --git a/src/ng/directive/ngOptions.js b/src/ng/directive/ngOptions.js
@@ -407,7 +407,8 @@ var ngOptionsDirective = ['$compile', '$document', '$parse', function($compile,
   }
 
 
-  // we can't just jqLite('<option>') since jqLite is not smart enough
+  // Support: IE 9 only
+  // We can't just jqLite('<option>') since jqLite is not smart enough
   // to create it in <select> and IE barfs otherwise.
   var optionTemplate = window.document.createElement('option'),
       optGroupTemplate = window.document.createElement('optgroup');
@@ -619,7 +620,8 @@ var ngOptionsDirective = ['$compile', '$document', '$parse', function($compile,
       function updateOptionElement(option, element) {
         option.element = element;
         element.disabled = option.disabled;
-        // NOTE: The label must be set before the value, otherwise IE10/11/EDGE create unresponsive
+        // Support: IE 11 only, Edge 12-13 only
+        // NOTE: The label must be set before the value, otherwise IE 11 & Edge create unresponsive
         // selects in certain circumstances when multiple selects are next to each other and display
         // the option list in listbox style, i.e. the select is [multiple], or specifies a [size].
         // See https://github.com/angular/angular.js/issues/11314 for more info.
diff --git a/src/ng/directive/select.js b/src/ng/directive/select.js
@@ -5,7 +5,7 @@
 var noopNgModelController = { $setViewValue: noop, $render: noop };
 
 function setOptionSelectedStatus(optionEl, value) {
-  optionEl.prop('selected', value); // needed for IE
+  optionEl.prop('selected', value);
   /**
    * When unselecting an option, setting the property to null / false should be enough
    * However, screenreaders might react to the selected attribute instead, see
@@ -40,6 +40,7 @@ var SelectController =
   // does not match any of the options. When it is rendered the value of the unknown
   // option is '? XXX ?' where XXX is the hashKey of the value that is not known.
   //
+  // Support: IE 9 only
   // We can't just jqLite('<option>') since jqLite is not smart enough
   // to create it in <select> and IE barfs otherwise.
   self.unknownOption = jqLite(window.document.createElement('option'));
@@ -620,10 +621,11 @@ var selectDirective = function() {
                                                includes(value, selectCtrl.selectValueMap[option.value]));
             var currentlySelected = option.selected;
 
-            // IE and Edge, adding options to the selection via shift+click/UP/DOWN,
+            // Support: IE 9-11 only, Edge 12-15+
+            // In IE and Edge adding options to the selection via shift+click/UP/DOWN
             // will de-select already selected options if "selected" on those options was set
             // more than once (i.e. when the options were already selected)
-            // So we only modify the selected property if neccessary.
+            // So we only modify the selected property if necessary.
             // Note: this behavior cannot be replicated via unit tests because it only shows in the
             // actual user interface.
             if (shouldBeSelected !== currentlySelected) {
diff --git a/src/ng/log.js b/src/ng/log.js
@@ -146,29 +146,17 @@ function $LogProvider() {
 
     function consoleLog(type) {
       var console = $window.console || {},
-          logFn = console[type] || console.log || noop,
-          hasApply = false;
+          logFn = console[type] || console.log || noop;
 
-      // Note: reading logFn.apply throws an error in IE11 in IE8 document mode.
-      // The reason behind this is that console.log has type "object" in IE8...
-      try {
-        hasApply = !!logFn.apply;
-      } catch (e) { /* empty */ }
-
-      if (hasApply) {
-        return function() {
-          var args = [];
-          forEach(arguments, function(arg) {
-            args.push(formatError(arg));
-          });
-          return logFn.apply(console, args);
-        };
-      }
-
-      // we are IE which either doesn't have window.console => this is noop and we do nothing,
-      // or we are IE where console.log doesn't have apply so we log at least first 2 args
-      return function(arg1, arg2) {
-        logFn(arg1, arg2 == null ? '' : arg2);
+      return function() {
+        var args = [];
+        forEach(arguments, function(arg) {
+          args.push(formatError(arg));
+        });
+        // Support: IE 9 only
+        // console methods don't inherit from Function.prototype in IE 9 so we can't
+        // call `logFn.apply(console, args)` directly.
+        return Function.prototype.apply.call(logFn, console, args);
       };
     }
   }];
diff --git a/src/ngSanitize/sanitize.js b/src/ngSanitize/sanitize.js
@@ -313,26 +313,15 @@ function $SanitizeProvider() {
     return obj;
   }
 
-  var inertBodyElement;
-  (function(window) {
+  var inertBodyElement = (function(window) {
     var doc;
     if (window.document && window.document.implementation) {
       doc = window.document.implementation.createHTMLDocument('inert');
     } else {
       throw $sanitizeMinErr('noinert', 'Can\'t create an inert html document');
     }
     var docElement = doc.documentElement || doc.getDocumentElement();
-    var bodyElements = docElement.getElementsByTagName('body');
-
-    // usually there should be only one body element in the document, but IE doesn't have any, so we need to create one
-    if (bodyElements.length === 1) {
-      inertBodyElement = bodyElements[0];
-    } else {
-      var html = doc.createElement('html');
-      inertBodyElement = doc.createElement('body');
-      html.appendChild(inertBodyElement);
-      doc.appendChild(html);
-    }
+    return docElement.getElementsByTagName('body')[0];
   })(window);
 
   /**
@@ -363,8 +352,9 @@ function $SanitizeProvider() {
       }
       mXSSAttempts--;
 
+      // Support: IE 9-11 only
       // strip custom-namespaced attributes on IE<=11
-      if (window.document.documentMode) {
+      if (msie) {
         stripCustomNsAttrs(inertBodyElement);
       }
       html = inertBodyElement.innerHTML; //trigger mXSS
diff --git a/test/AngularSpec.js b/test/AngularSpec.js
@@ -1733,6 +1733,7 @@ describe('angular', function() {
       dealoc(appElement);
     });
 
+    // Support: IE 9-11 only
     // IE does not support `document.currentScript` (nor extensions with protocol), so skip tests.
     if (!msie) {
       describe('auto bootstrap restrictions', function() {
diff --git a/test/helpers/privateMocks.js b/test/helpers/privateMocks.js
@@ -31,9 +31,9 @@ function xthey(msg, vals, spec) {
 }
 
 function browserSupportsCssAnimations() {
-  // Support: IE < 10
-  // Only IE10+ support keyframes / transitions
-  return !(window.document.documentMode < 10);
+  // Support: IE 9 only
+  // Only IE 10+ support keyframes / transitions
+  return !(msie < 10);
 }
 
 function createMockStyleSheet(doc) {
diff --git a/test/helpers/testabilityPatch.js b/test/helpers/testabilityPatch.js
@@ -30,13 +30,6 @@ beforeEach(function() {
   if (window.publishExternalAPI) {
     publishExternalAPI(angular);
 
-    // workaround for IE bug https://plus.google.com/104744871076396904202/posts/Kqjuj6RSbbT
-    // IE overwrite window.jQuery with undefined because of empty jQuery var statement, so we have to
-    // correct this, but only if we are not running in jqLite mode
-    if (!_jqLiteMode && _jQuery !== jQuery) {
-      jQuery = _jQuery;
-    }
-
     // This resets global id counter;
     uid = 0;
 
@@ -184,6 +177,7 @@ function sortedHtml(element, showNgClass) {
 
         var attr = attributes[i];
         if (attr.name.match(/^ng[:-]/) ||
+            !/^ng\d+/.test(attr.name) &&
             (attr.value || attr.value === '') &&
             attr.value !== 'null' &&
             attr.value !== 'auto' &&
@@ -199,19 +193,6 @@ function sortedHtml(element, showNgClass) {
             attr.name !== 'tabIndex' &&
             attr.name !== 'style' &&
             attr.name.substr(0, 6) !== 'jQuery') {
-          // in IE we need to check for all of these.
-          if (/ng\d+/.exec(attr.name) ||
-              attr.name === 'getElementById' ||
-              // IE7 has `selected` in attributes
-              attr.name === 'selected' ||
-              // IE7 adds `value` attribute to all LI tags
-              (node.nodeName === 'LI' && attr.name === 'value') ||
-              // IE8 adds bogus rowspan=1 and colspan=1 to TD elements
-              (node.nodeName === 'TD' && attr.name === 'rowSpan' && attr.value === '1') ||
-              (node.nodeName === 'TD' && attr.name === 'colSpan' && attr.value === '1')) {
-            continue;
-          }
-
           attrs.push(' ' + attr.name + '="' + attr.value + '"');
         }
       }
diff --git a/test/jqLiteSpec.js b/test/jqLiteSpec.js
@@ -1368,7 +1368,7 @@ describe('jqLite', function() {
       expect(callback).toHaveBeenCalledTimes(1);
     });
 
-    it('should set event.target on IE', function() {
+    it('should set event.target', function() {
       var elm = jqLite(a);
       elm.on('click', function(event) {
         expect(event.target).toBe(a);
diff --git a/test/minErrSpec.js b/test/minErrSpec.js
@@ -53,6 +53,7 @@ describe('minErr', function() {
                                arr,      obj,      anonFn,      namedFn);
 
     expect(myError.message).toContain('[test:26] arr: [1,2,3]; obj: {"a":123,"b":"baar"};');
+    // Support: IE 9-11 only
     // IE does not add space after "function"
     expect(myError.message).toMatch(/anonFn: function\s?\(something\);/);
     expect(myError.message).toContain('namedFn: function foo(something)');
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
diff --git a/test/ng/directive/booleanAttrsSpec.js b/test/ng/directive/booleanAttrsSpec.js
diff --git a/test/ng/locationSpec.js b/test/ng/locationSpec.js
diff --git a/test/ng/logSpec.js b/test/ng/logSpec.js
diff --git a/test/ng/urlUtilsSpec.js b/test/ng/urlUtilsSpec.js

Original file line number	Diff line number	Diff line change
`@@ -876,9 +876,7 @@ function createInjector(modulesToLoad, strictDi) {`
`876`	`876`	`}`
`877`	`877`	`var result = func.$$ngIsClass;`
`878`	`878`	`if (!isBoolean(result)) {`
`879`		`- // Support: Edge 12-13 only`
`880`		`- // See: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/6156135/`
`881`		`- result = func.$$ngIsClass = /^(?:class\b\|constructor\()/.test(stringifyFn(func));`
	`879`	`+ result = func.$$ngIsClass = /^class\b/.test(stringifyFn(func));`
`882`	`880`	`}`
`883`	`881`	`return result;`
`884`	`882`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,9 +31,9 @@ function xthey(msg, vals, spec) {`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`function browserSupportsCssAnimations() {`
`34`		`- // Support: IE < 10`
`35`		`- // Only IE10+ support keyframes / transitions`
`36`		`- return !(window.document.documentMode < 10);`
	`34`	`+ // Support: IE 9 only`
	`35`	`+ // Only IE 10+ support keyframes / transitions`
	`36`	`+ return !(msie < 10);`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`function createMockStyleSheet(doc) {`