Landing pull request 503. 1.7 data: set a flag in the private data cache to avoid having to scan attributes multiple times - Fixes #8909.

More Details:
 - https://github.com/jquery/jquery/pull/503
 - http://bugs.jquery.com/ticket/8909

1 files changed, 21 insertions, 0 deletions

diff --git a/test/unit/data.js b/test/unit/data.js
index ebeb1b4ee..4feba32c0 100644
--- a/test/unit/data.js
+++ b/test/unit/data.js
@@ -599,3 +599,24 @@ test("Triggering the removeData should not throw exceptions. (#10080)", function
 	// change the url to trigger unload
 	frame.attr("src", "data/iframe.html?param=true");
 });
+
+test( "Only check element attributes once when calling .data() - #8909", function() {
+	expect( 2 );
+	var testing = {
+			test: "testing",
+			test2: "testing"
+		},
+		element = jQuery( "<div data-test='testing'>" ),
+		node = element[ 0 ];
+
+	// set an attribute using attr to ensure it
+	node.setAttribute( "data-test2", "testing" );
+	deepEqual( element.data(), testing, "Sanity Check" );
+
+	node.setAttribute( "data-test3", "testing" );
+	deepEqual( element.data(), testing, "The data didn't change even though the data-* attrs did" );
+
+	// clean up data cache
+	element.remove();
+
+});