From 9ec429cf6270e455aba4eba85f4db80e633806b6 Mon Sep 17 00:00:00 2001
From: John Hoven <hovenj@gmail.com>
Date: Thu, 6 Mar 2014 13:56:09 -0600
Subject: [PATCH] Attributes: Trim whitespace from option text when returned as
 a value

Fixes #14858
Ref #14686
Closes gh-1531
---
 AUTHORS.txt             |  1 +
 src/attributes/val.js   |  4 +++-
 test/unit/attributes.js | 13 +++++++++++++
 3 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/AUTHORS.txt b/AUTHORS.txt
index 70a97e8c3..0856478f2 100644
--- a/AUTHORS.txt
+++ b/AUTHORS.txt
@@ -210,4 +210,5 @@ S. Andrew Sheppard <andrew@wq.io>
 Roman Reiß <me@silverwind.io>
 Benjy Cui <benjytrys@gmail.com>
 Rodrigo Rosenfeld Rosas <rr.rosas@gmail.com>
+John Hoven <hovenj@gmail.com>
 
diff --git a/src/attributes/val.js b/src/attributes/val.js
index b13641b28..4a3b0e5c1 100644
--- a/src/attributes/val.js
+++ b/src/attributes/val.js
@@ -74,7 +74,9 @@ jQuery.extend({
 				var val = jQuery.find.attr( elem, "value" );
 				return val != null ?
 					val :
-					jQuery.text( elem );
+					// Support: IE10-11+
+					// option.text throws exceptions (#14686, #14858)
+					jQuery.trim( jQuery.text( elem ) );
 			}
 		},
 		select: {
diff --git a/test/unit/attributes.js b/test/unit/attributes.js
index 32fd7b958..2ab583caa 100644
--- a/test/unit/attributes.js
+++ b/test/unit/attributes.js
@@ -1462,3 +1462,16 @@ test( "should not throw at $(option).val() (#14686)", 1, function() {
 		ok( false );
 	}
 });
+
+test( "Insignificant white space returned for $(option).val() (#14858)", function() {
+	expect ( 3 );
+
+	var val = jQuery( "<option></option>" ).val();
+	equal( val.length, 0, "Empty option should have no value" );
+
+	val = jQuery( "<option>  </option>" ).val();
+	equal( val.length, 0, "insignificant white-space returned for value" );
+
+	val = jQuery( "<option>  test  </option>" ).val();
+	equal( val.length, 4, "insignificant white-space returned for value" );
+});