aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMichał Gołębiowski <m.goleb@gmail.com>2016-02-25 01:18:55 +0100
committerMichał Gołębiowski <m.goleb@gmail.com>2016-03-08 23:26:46 +0100
commit9b086888b803db872e8906cbdbf0de458231f225 (patch)
tree084539db8297eae7f97f82ffe0da2d318f81251f /src
parentb43a3685b60b307d61f41f0c94412380ed46ab22 (diff)
downloadjquery-9b086888b803db872e8906cbdbf0de458231f225.tar.gz
jquery-9b086888b803db872e8906cbdbf0de458231f225.zip
Docs:Tests: Remove obsolete code from tests, update support comments
Support comments that were lacking the final IE/Edge version that exhibits the bug were checked & updated. Links to the Chromium bug tracker were updated. Code in tests related to unsupported browsers (like Android 2.3 in non-basic tests) has been removed. Fixes gh-2868 Closes gh-2949
Diffstat (limited to 'src')
-rw-r--r--src/ajax/parseXML.js3
-rw-r--r--src/ajax/xhr.js6
-rw-r--r--src/core/support.js2
-rw-r--r--src/css/curCSS.js4
-rw-r--r--src/css/showHide.js2
-rw-r--r--src/css/support.js6
-rw-r--r--src/css/var/getStyles.js2
-rw-r--r--src/data/Data.js2
-rw-r--r--src/deferred/exceptionHook.js2
-rw-r--r--src/effects/Tween.js2
-rw-r--r--src/event.js4
-rw-r--r--src/event/focusin.js6
-rw-r--r--src/manipulation/getAll.js2
-rw-r--r--src/manipulation/support.js4
-rw-r--r--src/offset.js4
15 files changed, 26 insertions, 25 deletions
diff --git a/src/ajax/parseXML.js b/src/ajax/parseXML.js
index 6599aaf5b..eb17dbd26 100644
--- a/src/ajax/parseXML.js
+++ b/src/ajax/parseXML.js
@@ -9,7 +9,8 @@ jQuery.parseXML = function( data ) {
return null;
}
- // Support: IE9
+ // Support: IE 9-11 only
+ // IE throws on parseFromString with invalid input.
try {
xml = ( new window.DOMParser() ).parseFromString( data, "text/xml" );
} catch ( e ) {
diff --git a/src/ajax/xhr.js b/src/ajax/xhr.js
index db670ff15..9b8f47310 100644
--- a/src/ajax/xhr.js
+++ b/src/ajax/xhr.js
@@ -15,7 +15,7 @@ var xhrSuccessStatus = {
// File protocol always yields status code 0, assume 200
0: 200,
- // Support: IE9
+ // Support: IE9 only
// #1450: sometimes IE returns 1223 when it should be 204
1223: 204
},
@@ -79,7 +79,7 @@ jQuery.ajaxTransport( function( options ) {
xhr.abort();
} else if ( type === "error" ) {
- // Support: IE9
+ // Support: IE9 only
// On a manual native abort, IE9 throws
// errors on any property access that is not readyState
if ( typeof xhr.status !== "number" ) {
@@ -115,7 +115,7 @@ jQuery.ajaxTransport( function( options ) {
xhr.onload = callback();
errorCallback = xhr.onerror = callback( "error" );
- // Support: IE9
+ // Support: IE9 only
// Use onreadystatechange to replace onabort
// to handle uncaught aborts
if ( xhr.onabort !== undefined ) {
diff --git a/src/core/support.js b/src/core/support.js
index 0609a700b..3857a252e 100644
--- a/src/core/support.js
+++ b/src/core/support.js
@@ -3,7 +3,7 @@ define( [
"../var/support"
], function( document, support ) {
-// Support: Safari 8+
+// Support: Safari 8 only
// In Safari 8 documents created via document.implementation.createHTMLDocument
// collapse sibling forms: the second one becomes a child of the first one.
// Because of that, this security measure has to be disabled in Safari 8.
diff --git a/src/css/curCSS.js b/src/css/curCSS.js
index 17ea2a9d4..9054d4cf9 100644
--- a/src/css/curCSS.js
+++ b/src/css/curCSS.js
@@ -13,7 +13,7 @@ function curCSS( elem, name, computed ) {
computed = computed || getStyles( elem );
- // Support: IE9
+ // Support: IE9 only
// getPropertyValue is only needed for .css('filter') (#12537)
if ( computed ) {
ret = computed.getPropertyValue( name ) || computed[ name ];
@@ -47,7 +47,7 @@ function curCSS( elem, name, computed ) {
return ret !== undefined ?
- // Support: IE9-11+
+ // Support: IE9-11 only
// IE returns zIndex value as an integer.
ret + "" :
ret;
diff --git a/src/css/showHide.js b/src/css/showHide.js
index 8c9b0da18..45f8f0696 100644
--- a/src/css/showHide.js
+++ b/src/css/showHide.js
@@ -56,7 +56,7 @@ function showHide( elements, show ) {
}
if ( elem.style.display === "" && jQuery.css( elem, "display" ) === "none" &&
- // Support: Firefox <=42 - 43
+ // Support: Firefox 43+
// Don't set inline display on disconnected elements with computed display: none
jQuery.contains( elem.ownerDocument, elem ) ) {
diff --git a/src/css/support.js b/src/css/support.js
index f8e02d048..1150485bf 100644
--- a/src/css/support.js
+++ b/src/css/support.js
@@ -15,7 +15,7 @@ define( [
return;
}
- // Support: IE9-11+
+ // Support: IE9-11 only
// Style of cloned element affects source element cloned (#8908)
div.style.backgroundClip = "content-box";
div.cloneNode( true ).style.backgroundClip = "";
@@ -66,7 +66,7 @@ define( [
},
pixelMarginRight: function() {
- // Support: Android 4.0-4.3
+ // Support: Android 4.0 - 4.3 only
// We're checking for boxSizingReliableVal here instead of pixelMarginRightVal
// since that compresses better and they're computed together anyway.
if ( boxSizingReliableVal == null ) {
@@ -76,7 +76,7 @@ define( [
},
reliableMarginLeft: function() {
- // Support: IE <=8 only, Android 4.0 - 4.3 only, Firefox <=3 - 37
+ // Support: Android 4.0 - 4.3 only, Firefox <=3 - 44+
if ( boxSizingReliableVal == null ) {
computeStyleTests();
}
diff --git a/src/css/var/getStyles.js b/src/css/var/getStyles.js
index 02a4b8194..f9dc5cb24 100644
--- a/src/css/var/getStyles.js
+++ b/src/css/var/getStyles.js
@@ -1,7 +1,7 @@
define( function() {
return function( elem ) {
- // Support: IE<=11+, Firefox<=30+ (#15098, #14150)
+ // Support: IE<=11 only, Firefox<=30+ (#15098, #14150)
// IE throws on elements created in popups
// FF meanwhile throws on frame elements through "defaultView.getComputedStyle"
var view = elem.ownerDocument.defaultView;
diff --git a/src/data/Data.js b/src/data/Data.js
index e214282b3..779b291a5 100644
--- a/src/data/Data.js
+++ b/src/data/Data.js
@@ -141,7 +141,7 @@ Data.prototype = {
// Support: Chrome <= 35-45+
// Webkit & Blink performance suffers when deleting properties
// from DOM nodes, so set to undefined instead
- // https://code.google.com/p/chromium/issues/detail?id=378607
+ // https://bugs.chromium.org/p/chromium/issues/detail?id=378607 (bug restricted)
if ( owner.nodeType ) {
owner[ this.expando ] = undefined;
} else {
diff --git a/src/deferred/exceptionHook.js b/src/deferred/exceptionHook.js
index b9955063a..652cc7d64 100644
--- a/src/deferred/exceptionHook.js
+++ b/src/deferred/exceptionHook.js
@@ -9,7 +9,7 @@ var rerrorNames = /^(Eval|Internal|Range|Reference|Syntax|Type|URI)Error$/;
jQuery.Deferred.exceptionHook = function( error, stack ) {
- // Support: IE9
+ // Support: IE9 only
// Console exists when dev tools are open, which can happen at any time
if ( window.console && window.console.warn && error && rerrorNames.test( error.name ) ) {
window.console.warn( "jQuery.Deferred exception: " + error.message, stack );
diff --git a/src/effects/Tween.js b/src/effects/Tween.js
index 47f15a947..8379ded7a 100644
--- a/src/effects/Tween.js
+++ b/src/effects/Tween.js
@@ -88,7 +88,7 @@ Tween.propHooks = {
}
};
-// Support: IE9
+// Support: IE9 only
// Panic based approach to setting things on disconnected nodes
Tween.propHooks.scrollTop = Tween.propHooks.scrollLeft = {
set: function( tween ) {
diff --git a/src/event.js b/src/event.js
index 3cf0c3d7c..8cd8e3faa 100644
--- a/src/event.js
+++ b/src/event.js
@@ -22,7 +22,7 @@ function returnFalse() {
return false;
}
-// Support: IE9
+// Support: IE9 only
// See #13393 for more info
function safeActiveElement() {
try {
@@ -625,7 +625,7 @@ jQuery.Event.prototype = {
//
// Support: Safari 7 only
// Safari sends mouseenter too often; see:
-// https://code.google.com/p/chromium/issues/detail?id=470258
+// https://bugs.chromium.org/p/chromium/issues/detail?id=470258
// for the description of the bug (it existed in older Chrome versions as well).
jQuery.each( {
mouseenter: "mouseover",
diff --git a/src/event/focusin.js b/src/event/focusin.js
index ae7f831a8..6c6267be3 100644
--- a/src/event/focusin.js
+++ b/src/event/focusin.js
@@ -7,14 +7,14 @@ define( [
"./trigger"
], function( jQuery, dataPriv, support ) {
-// Support: Firefox
+// Support: Firefox 44+
// Firefox doesn't have focus(in | out) events
// Related ticket - https://bugzilla.mozilla.org/show_bug.cgi?id=687787
//
-// Support: Chrome, Safari
+// Support: Chrome 48+, Safari 9.0+
// focus(in | out) events fire after focus & blur events,
// which is spec violation - http://www.w3.org/TR/DOM-Level-3-Events/#events-focusevent-event-order
-// Related ticket - https://code.google.com/p/chromium/issues/detail?id=449857
+// Related ticket - https://bugs.chromium.org/p/chromium/issues/detail?id=449857
if ( !support.focusin ) {
jQuery.each( { focus: "focusin", blur: "focusout" }, function( orig, fix ) {
diff --git a/src/manipulation/getAll.js b/src/manipulation/getAll.js
index cc913f29e..a5a0ccaf2 100644
--- a/src/manipulation/getAll.js
+++ b/src/manipulation/getAll.js
@@ -4,7 +4,7 @@ define( [
function getAll( context, tag ) {
- // Support: IE9-11+
+ // Support: IE9-11 only
// Use typeof to avoid zero-argument method invocation on host objects (#15151)
var ret = typeof context.getElementsByTagName !== "undefined" ?
context.getElementsByTagName( tag || "*" ) :
diff --git a/src/manipulation/support.js b/src/manipulation/support.js
index 4f6b9de87..da474e0a3 100644
--- a/src/manipulation/support.js
+++ b/src/manipulation/support.js
@@ -8,7 +8,7 @@ define( [
div = fragment.appendChild( document.createElement( "div" ) ),
input = document.createElement( "input" );
- // Support: Android 4.0-4.3
+ // Support: Android 4.0 - 4.3 only
// Check state lost if the name is set (#11217)
// Support: Windows Web Apps (WWA)
// `name` and `type` must use .setAttribute for WWA (#14901)
@@ -22,7 +22,7 @@ define( [
// Older WebKit doesn't clone checked state correctly in fragments
support.checkClone = div.cloneNode( true ).cloneNode( true ).lastChild.checked;
- // Support: IE<=11+
+ // Support: IE<=11 only
// Make sure textarea (and checkbox) defaultValue is properly cloned
div.innerHTML = "<textarea>x</textarea>";
support.noCloneChecked = !!div.cloneNode( true ).lastChild.defaultValue;
diff --git a/src/offset.js b/src/offset.js
index 365041391..c378e4d37 100644
--- a/src/offset.js
+++ b/src/offset.js
@@ -203,10 +203,10 @@ jQuery.each( { scrollLeft: "pageXOffset", scrollTop: "pageYOffset" }, function(
};
} );
-// Support: Safari<7-8+, Chrome<37-44+
+// Support: Safari<7-9.0+, Chrome<37-48+
// Add the top/left cssHooks using jQuery.fn.position
// Webkit bug: https://bugs.webkit.org/show_bug.cgi?id=29084
-// Blink bug: https://code.google.com/p/chromium/issues/detail?id=229280
+// Blink bug: https://bugs.chromium.org/p/chromium/issues/detail?id=589347
// getComputedStyle returns percent when specified for top/left/bottom/right;
// rather than make the css module depend on the offset module, just check for it here
jQuery.each( [ "top", "left" ], function( i, prop ) {