Attrs: Remove outdated valHook for option element

This hook was relevant for BlackBerry 4.7 which is no longer supported.

But this code path now raises error in IE10-11 (#14686) in 1.x-master branch.
So to just to be safe, also add test for IE issue to this branch too.

diff --git a/src/attributes/val.js b/src/attributes/val.js
index 6fc4a84bb2212fb09306a84a46685720215dd91e..ca5fa08380b6f53ddaf2d0a97cffb0ecf2490f92 100644 (file)
--- a/src/attributes/val.js
+++ b/src/attributes/val.js
@@ -71,14 +71,6 @@ jQuery.fn.extend({
 
 jQuery.extend({
        valHooks: {
-               option: {
-                       get: function( elem ) {
-                               // attributes.value is undefined in Blackberry 4.7 but
-                               // uses .value. See #6932
-                               var val = elem.attributes.value;
-                               return !val || val.specified ? elem.value : elem.text;
-                       }
-               },
                select: {
                        get: function( elem ) {
                                var value, option,

diff --git a/test/unit/attributes.js b/test/unit/attributes.js
index 548c6f3180a659b10529e8687fafff95290e97d6..7fc604470cb27da13fb13219175ecbfe22d62018 100644 (file)
--- a/test/unit/attributes.js
+++ b/test/unit/attributes.js
@@ -1445,3 +1445,12 @@ test( "coords returns correct values in IE6/IE7, see #10828", function() {
        area = map.html("<area shape='rect' coords='0,0,0,0' href='#' alt='a' />").find("area");
        equal( area.attr("coords"), "0,0,0,0", "did not retrieve coords correctly" );
 });
+
+test( "should not throw at $(option).val() (#14686)", 1, function() {
+       try {
+               jQuery( "<option/>" ).val();
+               ok( true );
+       } catch ( _ ) {
+               ok( false );
+       }
+});