16760 2008-01-05 19:17:13 -0800 Incorrect handling of <object> data MIME types (affects Acid3) 2008-02-20 18:00:25 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All RESOLVED FIXED http://acid3.acidtests.org/ HasReduction P2 Normal --- 17064 1 eric webkit-unassigned gavin.sharp julian.reschke jwalden+bwo webkit oldest_to_newest 66331 0 eric 2008-01-05 19:17:13 -0800 Red square covers Acid3 test in Safari http://www.hixie.ch/tests/evil/acid/003/NOT_READY_PLEASE_DO_NOT_USE.html I'm not even entirely sure what the red square is. I think it's the <map>. The inspector wasn't very helpful due to bug 16755. 66332 1 eric 2008-01-05 19:17:53 -0800 I will note that neither Opera or FF display the red square. 66363 2 eric 2008-01-06 00:31:46 -0800 FYI the red square is from the <object> element, which has this CSS: object { position: fixed; left: 130.5px; top: 84.3px; background: transparent; } /* show objects if they have content */ Hixie made a comment in IRC about this begin a test of <object> Content-Type support. Looking at the sent headers: curl -I "http://www.hixie.ch/tests/evil/acid/003/support-a.png" HTTP/1.1 404 Not Found Date: Sun, 06 Jan 2008 08:29:39 GMT Server: Apache/2.0.61 (Unix) PHP/4.4.7 mod_ssl/2.0.61 OpenSSL/0.9.7e mod_fastcgi/2.4.2 DAV/2 SVN/1.4.2 Last-Modified: Thu, 01 Jun 2006 06:09:43 GMT ETag: "13bef5e-908-839d37c0" Accept-Ranges: bytes Content-Length: 2312 X-Pingback: http://tracking.damowmow.com/ Content-Language: en-GB-x-Hixie Content-Type: image/png I guess the issue there is the 404 which we're not respecting? 66373 3 julian.reschke 2008-01-06 07:01:26 -0800 One could argue that the 404 response itself should be fixed. For instance, why would a server claim there is no representation (404) and then include an ETag??? 66454 4 eric 2008-01-07 07:34:24 -0800 So this is the test in question: function () { // test 16: <object> handling and HTTP status codes var oC = document.createElement('object'); oC.appendChild(document.createTextNode("FAIL")); var oB = document.createElement('object'); var oA = document.createElement('object'); oA.data = "support-a.png"; oB.data = "support-b.png"; oB.appendChild(oC); oC.data = "support-c.png"; oA.appendChild(oB); document.getElementsByTagName("map")[0].appendChild(oA); // assuming the above didn't raise any exceptions, this test has passed // (the real test is whether the rendering is correct) return 1; }, http://www.hixie.ch/tests/evil/acid/003/support-a.png // gives a 404 http://www.hixie.ch/tests/evil/acid/003/support-b.png // returns wrong content type http://www.hixie.ch/tests/evil/acid/003/support-c.png // returns identical to a, (including Etag), but but returns 200 instead of 404. 67259 5 eric 2008-01-14 16:04:38 -0800 This is at least related to bug 15242, possibly even dependent on. It wasn't clear to me from my brief trip through the FrameLoader code over the weekend, how we recognize 404s down in WebCore. 68102 6 eric 2008-01-24 00:49:39 -0800 If we're to implement this the way Mark started to, then we'll need to add a new callback to CachedResourceClient. It could even be as specific as respondedWithError() or similar. Hooking into the notifyFinished() callback would be too late, since who knows, the 404 page could be 10mb and there is no need to download the whole page only to trip over into <object> fallback (or ignore the CachedStyleSheet) 68385 7 18724 webkit 2008-01-27 15:09:12 -0800 Created attachment 18724 Minimal test case 69400 8 hyatt 2008-02-04 15:03:46 -0800 We do respect 404s on object fallback. That was required for Acid2. I think the issue is with the middle PNG (the one with the wrong content type). I'm not sure what's going on though. 69402 9 hyatt 2008-02-04 15:06:47 -0800 I don't understand what is wrong with support-c.png. I also don't understand why support-b.png would not be used. Shouldn't it be used? 69403 10 hyatt 2008-02-04 15:07:36 -0800 Aha, I get it. support-b.png should be honored as text/html (and thus render nothing). This test is exposing the problems with not honoring the true content type of the data. 69438 11 18927 webkit 2008-02-04 23:09:02 -0800 Created attachment 18927 Standalone test case from Acid 3 suite 69439 12 18928 webkit 2008-02-04 23:09:48 -0800 Created attachment 18928 Static test case from Acid3 suite 69440 13 18929 webkit 2008-02-04 23:11:38 -0800 Created attachment 18929 test case with all 3 cases separated 69442 14 webkit 2008-02-04 23:17:55 -0800 Dave, based on "test case with all 3 cases separated" it's clear that Webkit has a problem only with support-a.png and support-b.png. support-c.png is correct, and should be displayed, but in Acid3 this picture is covered by "blank" support-b.png. 69444 15 18931 webkit 2008-02-04 23:27:43 -0800 Created attachment 18931 test case with all 3 cases separated some issues in test case corrected. 69465 16 18935 webkit 2008-02-05 05:24:44 -0800 Created attachment 18935 test for treating text/html file as CSS file Another test from Acid3 (a word Acid3 should not be red). It shows that Webkit treats text/html file as text/css and incorrectly applies styles from it to current document. 69466 17 webkit 2008-02-05 05:30:29 -0800 I wonder if the last test case should be filed on separate bug report? However the main problem relies on incorrectly handling MIME types and HTTP status codes. 71319 18 webkit 2008-02-20 15:01:47 -0800 *** Bug 16690 has been marked as a duplicate of this bug. *** 71321 19 webkit 2008-02-20 15:04:30 -0800 Another variant of the issue: http://biesi.damowmow.com/object/011.html 71322 20 webkit 2008-02-20 15:08:10 -0800 Many various tests for <object> MIME type handling are available on: http://biesi.damowmow.com/object/ 71336 21 hyatt 2008-02-20 18:00:25 -0800 Fixed in r30438. 18724 2008-01-27 15:09:12 -0800 2008-01-27 15:09:12 -0800 Minimal test case bug-16760-minimal-test-case.html text/html 315 webkit PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIiAiaHR0cDov L3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+CjxodG1sPgo8aGVhZD4KPHRpdGxlPk9C SkVDVCA0MDQgZmFsbGJhY2sgY29udGVudCB0ZXN0PC90aXRsZT4KPC9oZWFkPgo8Ym9keT4KPHA+ VGhlcmUgc2hvdWxkIGJlIG5vIFJFRCBvbiB0aGUgcGFnZTwvcD4KPHA+PG9iamVjdCBkYXRhPSJo dHRwOi8vd3d3LmhpeGllLmNoL3Rlc3RzL2V2aWwvYWNpZC8wMDMvc3VwcG9ydC1hLnBuZyI+UEFT Uzwvb2JqZWN0PjwvcD4KPC9ib2R5Pgo8L2h0bWw+ 18927 2008-02-04 23:09:02 -0800 2008-02-04 23:09:02 -0800 Standalone test case from Acid 3 suite test-case-object-handling-js.html text/html 773 webkit PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KCTx0aXRsZT5EeW5hbWljIHRlc3QgZm9yIE9C SkVDVCBoYW5kbGluZyBhbmQgSFRUUCBzdGF0dXMgY29kZXM8L3RpdGxlPgo8L2hlYWQ+Cjxib2R5 Pgo8ZGl2PjwvZGl2Pgo8c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlwdCI+CiAgIC8vIHRlc3Q6 IDxvYmplY3Q+IGhhbmRsaW5nIGFuZCBIVFRQIHN0YXR1cyBjb2RlcwogICAgICB2YXIgb0MgPSBk b2N1bWVudC5jcmVhdGVFbGVtZW50KCdvYmplY3QnKTsKICAgICAgb0MuYXBwZW5kQ2hpbGQoZG9j dW1lbnQuY3JlYXRlVGV4dE5vZGUoIkZBSUwiKSk7CiAgICAgIHZhciBvQiA9IGRvY3VtZW50LmNy ZWF0ZUVsZW1lbnQoJ29iamVjdCcpOwogICAgICB2YXIgb0EgPSBkb2N1bWVudC5jcmVhdGVFbGVt ZW50KCdvYmplY3QnKTsKICAgICAgb0EuZGF0YSA9ICJodHRwOi8vd3d3LmhpeGllLmNoL3Rlc3Rz L2V2aWwvYWNpZC8wMDMvc3VwcG9ydC1hLnBuZyI7CiAgICAgIG9CLmRhdGEgPSAiaHR0cDovL3d3 dy5oaXhpZS5jaC90ZXN0cy9ldmlsL2FjaWQvMDAzL3N1cHBvcnQtYi5wbmciOwogICAgICBvQi5h cHBlbmRDaGlsZChvQyk7CiAgICAgIG9DLmRhdGEgPSAiaHR0cDovL3d3dy5oaXhpZS5jaC90ZXN0 cy9ldmlsL2FjaWQvMDAzL3N1cHBvcnQtYy5wbmciOwogICAgICBvQS5hcHBlbmRDaGlsZChvQik7 CiAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRzQnlUYWdOYW1lKCJkaXYiKVswXS5hcHBlbmRDaGls ZChvQSk7Cjwvc2NyaXB0Pgo8L2JvZHk+CjwvaHRtbD4= 18928 2008-02-04 23:09:48 -0800 2008-02-04 23:09:48 -0800 Static test case from Acid3 suite test-case-object-handling.html text/html 376 webkit PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KCTx0aXRsZT5UZXN0IGZvciBPQkpFQ1QgaGFu ZGxpbmcgYW5kIEhUVFAgc3RhdHVzIGNvZGVzPC90aXRsZT4KPC9oZWFkPgo8Ym9keT4KPG9iamVj dCBkYXRhPSJodHRwOi8vd3d3LmhpeGllLmNoL3Rlc3RzL2V2aWwvYWNpZC8wMDMvc3VwcG9ydC1h LnBuZyI+Cgk8b2JqZWN0IGRhdGE9Imh0dHA6Ly93d3cuaGl4aWUuY2gvdGVzdHMvZXZpbC9hY2lk LzAwMy9zdXBwb3J0LWIucG5nIj4KCQk8b2JqZWN0IGRhdGE9Imh0dHA6Ly93d3cuaGl4aWUuY2gv dGVzdHMvZXZpbC9hY2lkLzAwMy9zdXBwb3J0LWMucG5nIj4KCQkJRkFJTAoJCTwvb2JqZWN0PgoJ PG9iamVjdD4KPC9vYmplY3Q+CjwvYm9keT4KPC9odG1sPg== 18929 2008-02-04 23:11:38 -0800 2008-02-04 23:27:43 -0800 test case with all 3 cases separated minimal-test-case.html text/html 801 webkit PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIiAiaHR0cDov L3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+CjxodG1sPgo8aGVhZD4KPHRpdGxlPk9C SkVDVCA0MDQgZmFsbGJhY2sgY29udGVudCB0ZXN0PC90aXRsZT4KPC9oZWFkPgo8Ym9keT4KPGRp diBzdHlsZT0iYm9yZGVyOjFweCBzb2xpZCBibHVlOyBtYXJnaW46IDFlbTsiPgo8cD5UZXN0IDE6 IHlvdSBzaG91bGQgc2VlIFBBU1Mgd29yZCBiZWxvdzwvcD4KPHA+PG9iamVjdCBkYXRhPSJodHRw Oi8vd3d3LmhpeGllLmNoL3Rlc3RzL2V2aWwvYWNpZC8wMDMvc3VwcG9ydC1hLnBuZyI+UEFTUzwv b2JqZWN0PjwvcD4KPC9kaXY+CjxkaXYgc3R5bGU9ImJvcmRlcjoxcHggc29saWQgYmx1ZTsgbWFy Z2luOiAxZW07Ij4KPHA+VGVzdCAyOiB5b3Ugc2hvdWxkIHNlZSBibGFuayBzcGFjZSBiZWxvdyAo d2l0aG91dCBpbWFnZSBwbGFjZWhvbGRlciEpPC9wPgo8cD48b2JqZWN0IGRhdGE9Imh0dHA6Ly93 d3cuaGl4aWUuY2gvdGVzdHMvZXZpbC9hY2lkLzAwMy9zdXBwb3J0LWIucG5nIj48L29iamVjdD48 L3A+CjwvZGl2Pgo8ZGl2IHN0eWxlPSJib3JkZXI6MXB4IHNvbGlkIGJsdWU7IG1hcmdpbjogMWVt OyI+CjxwPlRlc3QgMjogeW91IHNob3VsZCBzZWUgYSBmYWNlIG9mIGNhdCBvbiByZWQgYmFja2dy b3VuZDwvcD4KPHA+PG9iamVjdCBkYXRhPSJodHRwOi8vd3d3LmhpeGllLmNoL3Rlc3RzL2V2aWwv YWNpZC8wMDMvc3VwcG9ydC1jLnBuZyI+PC9vYmplY3Q+PC9wPgo8L2Rpdj4KPC9ib2R5Pgo8L2h0 bWw+ 18931 2008-02-04 23:27:43 -0800 2008-02-04 23:27:43 -0800 test case with all 3 cases separated test-case-object-handling-1.html text/html 809 webkit PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIiAiaHR0cDov L3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+CjxodG1sPgo8aGVhZD4KPHRpdGxlPk9C SkVDVCA0MDQgZmFsbGJhY2sgY29udGVudCB0ZXN0PC90aXRsZT4KPC9oZWFkPgo8Ym9keT4KPGRp diBzdHlsZT0iYm9yZGVyOjFweCBzb2xpZCBibHVlOyBtYXJnaW46IDFlbTsiPgo8cD5UZXN0IDE6 IHlvdSBzaG91bGQgc2VlIFBBU1Mgd29yZCBiZWxvdzwvcD4KPHA+PG9iamVjdCBkYXRhPSJodHRw Oi8vd3d3LmhpeGllLmNoL3Rlc3RzL2V2aWwvYWNpZC8wMDMvc3VwcG9ydC1hLnBuZyI+UEFTUzwv b2JqZWN0PjwvcD4KPC9kaXY+CjxkaXYgc3R5bGU9ImJvcmRlcjoxcHggc29saWQgYmx1ZTsgbWFy Z2luOiAxZW07Ij4KPHA+VGVzdCAyOiB5b3Ugc2hvdWxkIHNlZSBibGFuayBzcGFjZSBiZWxvdyAo d2l0aG91dCBpbWFnZSBwbGFjZWhvbGRlciEpPC9wPgo8cD48b2JqZWN0IGRhdGE9Imh0dHA6Ly93 d3cuaGl4aWUuY2gvdGVzdHMvZXZpbC9hY2lkLzAwMy9zdXBwb3J0LWIucG5nIj5GQUlMPC9vYmpl Y3Q+PC9wPgo8L2Rpdj4KPGRpdiBzdHlsZT0iYm9yZGVyOjFweCBzb2xpZCBibHVlOyBtYXJnaW46 IDFlbTsiPgo8cD5UZXN0IDM6IHlvdSBzaG91bGQgc2VlIGEgZmFjZSBvZiBjYXQgb24gcmVkIGJh Y2tncm91bmQ8L3A+CjxwPjxvYmplY3QgZGF0YT0iaHR0cDovL3d3dy5oaXhpZS5jaC90ZXN0cy9l dmlsL2FjaWQvMDAzL3N1cHBvcnQtYy5wbmciPkZBSUw8L29iamVjdD48L3A+CjwvZGl2Pgo8L2Jv ZHk+CjwvaHRtbD4= 18935 2008-02-05 05:24:44 -0800 2008-02-05 05:24:44 -0800 test for treating text/html file as CSS file test-case.html text/html 355 webkit PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIiAiaHR0cDov L3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+CjxodG1sPgo8aGVhZD4KCTx0aXRsZT5U ZXN0IGNhc2UgZm9yIGJ1ZyAxNjc2MDwvdGl0bGU+Cgk8bGluayByZWw9InN0eWxlc2hlZXQiIGhy ZWY9Imh0dHA6Ly9hY2lkMy5hY2lkdGVzdHMub3JnL2VtcHR5LmNzcyI+PCEtLSB0ZXh0L2h0bWwg ZmlsZSAoc2hvdWxkIGJlIGlnbm9yZWQsIDxoMT4gd2lsbCBnbyByZWQgaWYgaXQgaXNuJ3QpIC0t Pgo8L2hlYWQ+Cjxib2R5PgoJPGgxPlRoaXMgdGV4dCBzaG91bGQgTk9UIGJlIFJFRDwvaDE+Cjwv Ym9keT4KPC9odG1sPg==