Changeset 4029

Timestamp:

12/05/07 00:26:13 (1 year ago)

Author:

davidserduke

Message:

Fixed #1264. If you read the bug there were many proposed changes. As it turned out most of them had already been implemented. The last ones necessary were in .domManip() with when a <table> was 'this' and for .text(). Adding these last changes seems to make dom and text manipulation in IE frames possible. Unit test cases were added as well.

In addition "submit.gif" was removed from the test suite index.html since it didn't exist.

Location:

trunk/jquery

Files:

• src/core.js (modified) (2 diffs)
• test/data/iframe.html (added)
• test/index.html (modified) (2 diffs)
• test/unit/core.js (modified) (2 diffs)

trunk/jquery/src/core.js

@@ -203 +203 @@
     text: function( text ) {
         if ( typeof text != "object" && text != null )
-            return this.empty().append( document.createTextNode( text ) );
+            return this.empty().append( (this[0] && this[0].ownerDocument || document).createTextNode( text ) );
 
         var ret = "";
@@ -469 +469 @@
 
             if ( table && jQuery.nodeName( this, "table" ) && jQuery.nodeName( elems[0], "tr" ) )
-                obj = this.getElementsByTagName("tbody")[0] || this.appendChild( document.createElement("tbody") );
+                obj = this.getElementsByTagName("tbody")[0] || this.appendChild( this.ownerDocument.createElement("tbody") );
 
             var scripts = jQuery( [] );

trunk/jquery/test/index.html

@@ -22 +22 @@
     <!-- Test HTML -->
     <div id="nothiddendiv" style="height:1px;background:white;"></div>
+    <!-- this iframe is outside the #main so it won't reload constantly wasting time, but it means the tests must be "safe" and clean up after themselves -->
+    <iframe id="loadediframe" name="loadediframe" style="display:none;" src="data/iframe.html"></iframe>
     <dl id="dl" style="display:none;">
     <div id="main" style="display: none;">
@@ -152 +154 @@
             <input type="submit" name="sub1" value="NO" />
             <input type="submit" name="sub2" value="NO" />
-            <input type="image" name="sub3" value="NO" src="submit.gif" />
+            <input type="image" name="sub3" value="NO" />
             <button name="sub4" type="submit" value="NO">NO</button>
             <input name="D1" type="text" value="NO" disabled="disabled" />

trunk/jquery/test/unit/core.js

@@ -552 +552 @@
     var pass = true;
     try {
-        $( $("iframe")[0].contentWindow.document.body ).append("<div>test</div>");
+        $( $("#iframe")[0].contentWindow.document.body ).append("<div>test</div>");
     } catch(e) {
         pass = false;
@@ -1189 +1189 @@
 
 test("contents()", function() {
-    expect(2);
+    expect(10);
     equals( $("#ap").contents().length, 9, "Check element contents" );
     ok( $("#iframe").contents()[0], "Check existance of IFrame document" );
-    // Disabled, randomly fails
-    //ok( $("#iframe").contents()[0].body, "Check existance of IFrame body" );
-});
+    var ibody = $("#loadediframe").contents()[0].body;
+    ok( ibody, "Check existance of IFrame body" );
+
+    equals( $("span", ibody).text(), "span text", "Find span in IFrame and check its text" );
+
+    $(ibody).append("<div>init text</div>");
+    equals( $("div", ibody).length, 2, "Check the original div and the new div are in IFrame" );
+
+    equals( $("div:last", ibody).text(), "init text", "Add text to div in IFrame" );
+
+    $("div:last", ibody).text("div text");
+    equals( $("div:last", ibody).text(), "div text", "Add text to div in IFrame" );
+
+    $("div:last", ibody).remove();
+    equals( $("div", ibody).length, 1, "Delete the div and check only one div left in IFrame" );
+
+    equals( $("div", ibody).text(), "span text", "Make sure the correct div is still left after deletion in IFrame" );
+
+    $("<table/>", ibody).append("<tr><td>cell</td></tr>").appendTo(ibody);
+    $("table", ibody).remove();
+    equals( $("div", ibody).length, 1, "Check for JS error on add and delete of a table in IFrame" );
+});