Bug 40661 - [Qt] fast/encoding/japanese-encoding-mix.html fails on Qt after enabling HTML5 Lexer
Summary: [Qt] fast/encoding/japanese-encoding-mix.html fails on Qt after enabling HTML...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2010-06-16 02:06 PDT by Eric Seidel (no email)
Modified: 2011-10-20 03:32 PDT (History)
3 users (show)

See Also:


Attachments
Skipping for now (1.72 KB, patch)
2010-06-16 02:09 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2010-06-16 02:06:04 PDT
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}"
Comment 1 Eric Seidel (no email) 2010-06-16 02:09:44 PDT
Created attachment 58863 [details]
Skipping for now
Comment 2 Eric Seidel (no email) 2010-06-16 02:12:22 PDT
Committed r61246: <http://trac.webkit.org/changeset/61246>
Comment 3 Eric Seidel (no email) 2010-06-16 11:28:53 PDT
Bug 40725 is the same failure except for Tiger (which also has different decoders).
Comment 4 Zoltan Arvai 2011-10-20 03:30:07 PDT
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