]> source.dussan.org Git - jquery.git/commitdiff
Selector: Remove the "a:enabled" workaround for Chrome <=77
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Mon, 16 Dec 2019 18:43:38 +0000 (19:43 +0100)
committerGitHub <noreply@github.com>
Mon, 16 Dec 2019 18:43:38 +0000 (19:43 +0100)
Remove the workaround for a broken `:enabled` pseudo-class on anchor elements
in Chrome <=77. These versions of Chrome considers anchor elements with the
`href` attribute as matching `:enabled`.

Closes gh-4569

src/selector/rbuggyQSA.js
test/unit/selector.js

index 2c853a0f1e8985f45541a2e756adcde3adec3c69..e9ebbb34eb2e781e8a47052a569f403eb95f188f 100644 (file)
@@ -6,15 +6,6 @@ var rbuggyQSA = [],
        testEl = document.createElement( "div" ),
        input = document.createElement( "input" );
 
-testEl.innerHTML = "<a href=''></a>";
-
-// Support: Chrome 38 - 77 only
-// Chrome considers anchor elements with href to match ":enabled"
-// See https://bugs.chromium.org/p/chromium/issues/detail?id=993387
-if ( testEl.querySelectorAll( ":enabled" ).length ) {
-       rbuggyQSA.push( ":enabled" );
-}
-
 // Support: IE 9 - 11+
 // IE's :disabled selector does not pick up the children of disabled fieldsets
 if ( isIE ) {
index a8204242a8ad42cacc87b54ae3991e92d2fd1dff..1b657fe77326028508035c7faf903940a317f552 100644 (file)
@@ -2,8 +2,6 @@ QUnit.module( "selector", {
        beforeEach: function() {
                this.safari = /\bsafari\b/i.test( navigator.userAgent ) &&
                        !/\bchrome\b/i.test( navigator.userAgent );
-               this.chrome = /\bchrome\b/i.test( navigator.userAgent ) &&
-                       !/\bedge\b/i.test( navigator.userAgent );
        },
        afterEach: moduleTeardown
 } );
@@ -1379,9 +1377,7 @@ QUnit.test( "pseudo - :(dis|en)abled, explicitly disabled", function( assert ) {
                        "disabled-select", "disabled-optgroup", "disabled-option" ]
        );
 
-       if ( QUnit.jQuerySelectors || !this.chrome ) {
-               // Support: Chrome 75+
-               // Chrome recognizes anchor elements as enabled.
+       if ( QUnit.jQuerySelectors ) {
                assert.t(
                        "Enabled elements",
                        "#enabled-fieldset :enabled",