19759 2008-06-24 18:14:20 -0700 Needs test: ChildNodeList for an attribute node is not properly invalidated after appending a child. 2010-06-18 17:16:55 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Mac OS X 10.4 RESOLVED DUPLICATE 33850 P4 Normal --- 1 chrisb webkit-unassigned ap oldest_to_newest 84326 0 chrisb 2008-06-24 18:14:20 -0700 The length property of a ChildNodeList for an attribute node does not change after append a new text node to the attribute node. 84327 1 21919 chrisb 2008-06-24 18:15:17 -0700 Created attachment 21919 Test case 84328 2 21920 chrisb 2008-06-24 18:19:06 -0700 Created attachment 21920 Improved test case 84329 3 chrisb 2008-06-24 18:21:37 -0700 FireFox ( v2 and v3) throws an exception when appendChild on the attribute node in the attached test case. 84335 4 21921 chrisb 2008-06-24 20:41:43 -0700 Created attachment 21921 Another version of the test case This test case demonstrates that accessing the length property of the ChildNodeList before the call to appendChild changes the value of the length property after the call to appendChild. 84368 5 ap 2008-06-25 02:51:02 -0700 Confirming, because this is formally a bug, but it seems to be quite low priority, given that there is now known content affected, that Firefox doesn't support this, and that there are talks of deprecating attribute nodes in HTML DOM altogether. 84412 6 chrisb 2008-06-25 09:28:25 -0700 (In reply to comment #5) > Confirming, because this is formally a bug, but it seems to be quite low > priority, given that there is now known content affected, that Firefox doesn't > support this, and that there are talks of deprecating attribute nodes in HTML > DOM altogether. > Fair enough. I entered this bug because one our internal unit tests was failing. 239506 7 ap 2010-06-17 11:47:18 -0700 This was fixed in bug 33850, but it would be nice to land this version of test case. 240225 8 ap 2010-06-18 17:16:55 -0700 Landed the test in <http://trac.webkit.org/changeset/61457>. *** This bug has been marked as a duplicate of bug 33850 *** 21919 2008-06-24 18:15:17 -0700 2008-06-24 18:19:06 -0700 Test case foo.html text/html 520 chrisb PGh0bWw+CjxoZWFkPgo8c2NyaXB0PgpmdW5jdGlvbiBvbkxvYWQoKQp7CiAgICB2YXIgdGhlRGl2 ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRoZURpdiIpOwogICAgdmFyIHRoZUF0dHJzID0g dGhlRGl2LmF0dHJpYnV0ZXM7CiAgICB2YXIgbmFtZUF0dHIgPSB0aGVBdHRycy5nZXROYW1lZEl0 ZW0oIm5hbWUiKTsKICAgIHZhciBuYW1lQXR0ckNoaWxkTm9kZXMgPSBuYW1lQXR0ci5jaGlsZE5v ZGVzOwogICAgdmFyIHRleHROb2RlID0gZG9jdW1lbnQuY3JlYXRlVGV4dE5vZGUoInRlcmRheSIp OwogICAgd2luZG93LmFsZXJ0KG5hbWVBdHRyQ2hpbGROb2Rlcy5sZW5ndGgpOwogICAgbmFtZUF0 dHIuYXBwZW5kQ2hpbGQodGV4dE5vZGUpOwogICAgd2luZG93LmFsZXJ0KG5hbWVBdHRyQ2hpbGRO b2Rlcy5sZW5ndGgpOwogICAgCn0KPC9zY3JpcHQ+CjwvaGVhZD4KPGJvZHkgb25sb2FkPSJvbkxv YWQoKTsiPgo8ZGl2IGlkPSJ0aGVEaXYiIG5hbWU9IlllcyI+SGVsbG88L2Rpdj4KPC9ib2R5Pgo8 L2h0bWw+Cg== 21920 2008-06-24 18:19:06 -0700 2008-06-24 18:19:06 -0700 Improved test case foo.html text/html 638 chrisb PGh0bWw+CjxoZWFkPgo8c2NyaXB0PgpmdW5jdGlvbiBvbkxvYWQoKQp7CiAgICB2YXIgdGhlRGl2 ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRoZURpdiIpOwogICAgdmFyIHRoZUF0dHJzID0g dGhlRGl2LmF0dHJpYnV0ZXM7CiAgICB2YXIgbmFtZUF0dHIgPSB0aGVBdHRycy5nZXROYW1lZEl0 ZW0oIm5hbWUiKTsKICAgIHZhciBuYW1lQXR0ckNoaWxkTm9kZXMgPSBuYW1lQXR0ci5jaGlsZE5v ZGVzOwogICAgdmFyIHRleHROb2RlID0gZG9jdW1lbnQuY3JlYXRlVGV4dE5vZGUoInRlcmRheSIp OwogICAgd2luZG93LmFsZXJ0KCJiZWZvcmUgYXBwZW5kOiBsZW5ndGg9IiArIG5hbWVBdHRyQ2hp bGROb2Rlcy5sZW5ndGggKyAiLCB2YWx1ZT0iICsgbmFtZUF0dHIubm9kZVZhbHVlICk7CiAgICBu YW1lQXR0ci5hcHBlbmRDaGlsZCh0ZXh0Tm9kZSk7CiAgICB3aW5kb3cuYWxlcnQoImFmdGVyIGFw cGVuZDogbGVuZ3RoPSIgKyBuYW1lQXR0ckNoaWxkTm9kZXMubGVuZ3RoICsgIiwgdmFsdWU9IiAr IG5hbWVBdHRyLm5vZGVWYWx1ZSApOwp9Cjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5IG9ubG9hZD0i b25Mb2FkKCk7Ij4KPGRpdiBpZD0idGhlRGl2IiBuYW1lPSJZZXMiPkhlbGxvPC9kaXY+CjwvYm9k eT4KPC9odG1sPgo= 21921 2008-06-24 20:41:43 -0700 2008-06-24 20:41:43 -0700 Another version of the test case foo.html text/html 927 chrisb PGh0bWw+CjxoZWFkPgo8c2NyaXB0PgpmdW5jdGlvbiBvbkxvYWQoKQp7CiAgICB2YXIgdGhlRGl2 ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRoZURpdiIpOwogICAgdmFyIHRoZUF0dHJzID0g dGhlRGl2LmF0dHJpYnV0ZXM7CgogICAgdmFyIG5hbWVBdHRyID0gdGhlQXR0cnMuZ2V0TmFtZWRJ dGVtKCJuYW1lIik7CiAgICB2YXIgbmFtZUF0dHJDaGlsZE5vZGVzID0gbmFtZUF0dHIuY2hpbGRO b2RlczsKICAgIHZhciB0ZXh0Tm9kZSA9IGRvY3VtZW50LmNyZWF0ZVRleHROb2RlKCJ0ZXJkYXki KTsKICAgIHZhciB1bnVzZWQgPSBuYW1lQXR0ckNoaWxkTm9kZXMubGVuZ3RoOwogICAgbmFtZUF0 dHIuYXBwZW5kQ2hpbGQodGV4dE5vZGUpOwoKICAgIHZhciBuYW1lMkF0dHIgPSB0aGVBdHRycy5n ZXROYW1lZEl0ZW0oIm5hbWUyIik7CiAgICB2YXIgbmFtZTJBdHRyQ2hpbGROb2RlcyA9IG5hbWUy QXR0ci5jaGlsZE5vZGVzOwogICAgdmFyIHRleHROb2RlMiA9IGRvY3VtZW50LmNyZWF0ZVRleHRO b2RlKCJ0ZXJkYXkiKTsKICAgIG5hbWUyQXR0ci5hcHBlbmRDaGlsZCh0ZXh0Tm9kZTIpOwoJCiAg ICBpZiAobmFtZUF0dHJDaGlsZE5vZGVzLmxlbmd0aCAhPSBuYW1lMkF0dHJDaGlsZE5vZGVzLmxl bmd0aCkKCXdpbmRvdy5hbGVydCgiRmFpbGVkOiBuYW1lQXR0ckNoaWxkTm9kZXMubGVuZ3RoPSIg KyBuYW1lQXR0ckNoaWxkTm9kZXMubGVuZ3RoICsgIiwgbmFtZTJBdHRyQ2hpbGROb2Rlcy5sZW5n dGg9IiArIG5hbWUyQXR0ckNoaWxkTm9kZXMubGVuZ3RoKTsKICAgIGVsc2UKCXdpbmRvdy5hbGVy dCgiUGFzc2VkISEiKTsKfQo8L3NjcmlwdD4KPC9oZWFkPgo8Ym9keSBvbmxvYWQ9Im9uTG9hZCgp OyI+CjxkaXYgaWQ9InRoZURpdiIgbmFtZT0iWWVzIiBuYW1lMj0iWWVzIj5IZWxsbzwvZGl2Pgo8 L2JvZHk+CjwvaHRtbD4K