Make sure that expando properties aren't set on embed, applet, or object elements...

diff --git a/src/data.js b/src/data.js
index 1d993fe..44d77a9 100644 (file)
--- a/src/data.js
+++ b/src/data.js
@@ -6,7 +6,19 @@ jQuery.extend({
        
        expando:expando,
 
+       // The following elements throw uncatchable exceptions if you
+       // attempt to add expando properties to them.
+       noData: {
+               "embed": true,
+               "object": true,
+               "applet": true
+       },
+
        data: function( elem, name, data ) {
+               if ( jQuery.noData[elem.nodeNode.toLowerCase()] ) {
+                       return;
+               }
+
                elem = elem == window ?
                        windowData :
                        elem;
@@ -43,6 +55,10 @@ jQuery.extend({
        },
 
        removeData: function( elem, name ) {
+               if ( jQuery.noData[elem.nodeNode.toLowerCase()] ) {
+                       return;
+               }
+
                elem = elem == window ?
                        windowData :
                        elem;

diff --git a/src/manipulation.js b/src/manipulation.js
index a8cb484..e1657a8 100644 (file)
--- a/src/manipulation.js
+++ b/src/manipulation.js
@@ -463,7 +463,7 @@ jQuery.extend({
 
 function cleanData( elems ) {
        for ( var i = 0, elem, id; (elem = elems[i]) != null; i++ ) {
-               if ( (id = elem[expando]) ) {
+               if ( !jQuery.noData[elem.nodeNode.toLowerCase()] && (id = elem[expando]) ) {
                        delete jQuery.cache[ id ];
                }
        }