40661 2010-06-16 02:06:04 -0700 [Qt] fast/encoding/japanese-encoding-mix.html fails on Qt after enabling HTML5 Lexer 2011-10-20 03:32:13 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) PC All RESOLVED FIXED Qt, QtTriaged P2 Normal --- 1 eric webkit-unassigned abarth ossy zarvai oldest_to_newest 238742 0 eric 2010-06-16 02:06:04 -0700 fast/encoding/japanese-encoding-mix.html fails on Qt after enabling HTML5 Lexer I suspect this has to do with Qt's custom encoders. It looks like the comment isn't properly getting closed, likely because Qt's decoder isn't recognizing the "--" correctly. Trac also fails to display the "--" before the >, but vi, and other decoders seem to see it fine: http://trac.webkit.org/browser/trunk/LayoutTests/fast/encoding/japanese-encoding-mix.html If the decoder isn't passing -- correctly to the HTML5DocumentParser then of course the unclosed comment would eat the rest of the document. Will need a Qt person to investigate. --- /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/encoding/japanese-encoding-mix-expected.txt 2010-06-16 01:31:13.436391978 -0700 +++ /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/encoding/japanese-encoding-mix-actual.txt 2010-06-16 01:31:13.436391978 -0700 @@ -1,5 +1,12 @@ -Some text here is encoded as EUC-JP, and some (in comment) as Shift_JIS. Since there is an explicit encoding declaration, auto-detection shouldn't change the encoding. - -ニッポンテレビ - -Should be EUC-JP: EUC-JP +layer at (0,0) size 800x600 + RenderView at (0,0) size 800x600 +layer at (0,0) size 800x600 + RenderBlock {HTML} at (0,0) size 800x600 + RenderBody {BODY} at (8,8) size 784x576 + RenderBlock {P} at (0,0) size 784x40 + RenderText {#text} at (0,0) size 717x40 + text run at (0,0) width 717: "Some text here is encoded as EUC-JP, and some (in comment) as Shift_JIS. Since there is an explicit" + text run at (0,20) width 483: "encoding declaration, auto-detection shouldn't change the encoding." + RenderBlock {P} at (0,56) size 784x20 + RenderText {#text} at (0,0) size 28x20 + text run at (0,0) width 28: "\x{30CB}\x{30C3}\x{30DD}\x{30F3}\x{30C6}\x{30EC}\x{30D3}" 238744 1 58863 eric 2010-06-16 02:09:44 -0700 Created attachment 58863 Skipping for now 238747 2 eric 2010-06-16 02:12:22 -0700 Committed r61246: <http://trac.webkit.org/changeset/61246> 238987 3 eric 2010-06-16 11:28:53 -0700 Bug 40725 is the same failure except for Tiger (which also has different decoders). 487452 4 zarvai 2011-10-20 03:30:07 -0700 This test is no longer on skipped list. It passes: http://build.webkit.org/builders/Qt%20Linux%20Release/builds/38749/steps/layout-test/logs/stdio 58863 2010-06-16 02:09:44 -0700 2010-06-16 02:11:30 -0700 Skipping for now bug-40661-20100616020942.patch text/plain 1763 eric ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCAwOGE0NTBlNDIwNmQ5MDM3NWZjNjQ4OTQ4YTU4N2YzNWZkMjQ2MWExLi5iMTk1YzZh YmUyNGNiOTk2OTI3MTJlYTI4NjQ2OWZmYjZiZjUyY2JmIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0 cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjAgQEAK IDIwMTAtMDYtMTYgIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgogCisgICAgICAgIFVu cmV2aWV3ZWQuICBEaXNhYmxpbmcgYSB0ZXN0IHdoaWNoIGR1ZSB0byBhIFF0IGRlY29kaW5nIGJ1 ZworICAgICAgICBpcyBub3cgZmFpbGluZyB3aXRoIHRoZSBIVE1MNSBsZXhlci4KKworICAgICAg ICBmYXN0L2VuY29kaW5nL2phcGFuZXNlLWVuY29kaW5nLW1peC5odG1sIGZhaWxzIG9uIFF0IGFm dGVyIGVuYWJsaW5nIEhUTUw1IExleGVyCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD00MDY2MQorCisgICAgICAgIFF0J3MgZGVjb2RlcnMgZG9uJ3Qgc2Vl bSB0byBiZSBwYXNzaW5nIHRoZSAiLS0iIG9mIHRoZSAiLS0+IiB0byB0aGUKKyAgICAgICAgSFRN TDVEb2N1bWVudFBhcnNlciBjb3JyZWN0bHksIHRodXMgdGhlIGNvbW1lbnQgaXMgbmV2ZXIgY2xv c2VkCisgICAgICAgIGFuZCB0aGUgcmVzdCBvZiB0aGUgZG9jdW1lbnQgZ2V0cyBlYXRlbi4KKwor ICAgICAgICAqIHBsYXRmb3JtL3F0L1NraXBwZWQ6CisgICAgICAgICAtIFNraXAgdGhlIHRlc3Qg dW50aWwgYSBRdCBwZXJzb24gY2FuIHRha2UgYSBsb29rLgorCisyMDEwLTA2LTE2ICBFcmljIFNl aWRlbCAgPGVyaWNAd2Via2l0Lm9yZz4KKwogICAgICAgICBVbnJldmlld2VkLCBqdXN0IHVwZGF0 aW5nIHF0IHBsYXRmb3JtIHJlc3VsdCB0byBtYXRjaCBuZXcgZXhwZWN0YXRpb25zLgogCiAgICAg ICAgIEVuYWJsZSBIVE1MNSBsZXhlcgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0v cXQvU2tpcHBlZCBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0L1NraXBwZWQKaW5kZXggZTMwYjFh OTZhZDQ4M2Y1ZTc3MTkzNWI4MjVjMGRmMzI0NmJjNWM2YS4uNTRiYjA5YWRjNjZiYTBhZWRlYjY0 ZGI3ZmY4YTAwZTNmOGMxMTJkOCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQv U2tpcHBlZAorKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkCkBAIC01NDEyLDUg KzU0MTIsOSBAQCBzdmcvVzNDLVNWRy0xLjEvZmlsdGVycy1saWdodC0wNC1mLnN2ZwogIyA9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PSAjCiAjIG5ldyBza2lwcGVkIHRlc3RzIHlldCB0byBiZSBzb3J0ZWQK ICMgPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0gIwotZmFzdC9kb20vR2VvbG9jYXRpb24vY2FsbGJhY2st dG8tcmVtb3RlLWNvbnRleHQyLmh0bWwgCitmYXN0L2RvbS9HZW9sb2NhdGlvbi9jYWxsYmFjay10 by1yZW1vdGUtY29udGV4dDIuaHRtbAorCisjIEZhaWxpbmcgYWZ0ZXIgZW5hYmxpbmcgSFRNTDUg cGFyc2VyIGluIHI2MTIzNAorIyBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/ aWQ9NDA2NjEKK2Zhc3QvZW5jb2RpbmcvamFwYW5lc2UtZW5jb2RpbmctbWl4Lmh0bWwKIAo=