Summary: | fast/encoding/japanese-encoding-mix.html fails on Tiger after enabling HTML5 Lexer | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, ap, commit-queue | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.4 | ||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2010-06-16 11:28:08 PDT
Bug 40661 is the same failure except for Qt (which also has different decoders). Skipping the test for now. This bug should remain open to track the failure (at least for as long as Tiger is still a real platform... which can't be that much longer now). Committed r61266: <http://trac.webkit.org/changeset/61266> The fix would be to just add a space before --> to insulate the test from EUC-JP decoder differences. That's fine too. Do we care about testing the encoder differences? I don't think we should try to test edge cases of complex decoders like this, unless we suspect that particular bugs may have security implications. Created attachment 58952 [details]
Patch
Comment on attachment 58952 [details]
Patch
+ Qt and Tiger seem to have encoding problems that swallow the --
I think it's only one character after an invalid sequence that gets swallowed.
r=me
Comment on attachment 58952 [details]
Patch
Wow. I don't think this was ever landed. :(
Comment on attachment 58952 [details] Patch Rejecting patch 58952 from commit-queue. Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Alexey Proskuryakov', u'--force']" exit_code: 1 Parsed 4 diffs from patch file(s). patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/fast/encoding/japanese-encoding-mix.html patching file LayoutTests/platform/mac-tiger/Skipped Hunk #1 FAILED at 194. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac-tiger/Skipped.rej patching file LayoutTests/platform/qt/Skipped Hunk #1 FAILED at 5417. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/qt/Skipped.rej Full output: http://queues.webkit.org/results/3734061 Created attachment 64136 [details]
Patch for landing
Comment on attachment 64136 [details] Patch for landing Clearing flags on attachment: 64136 Committed r65211: <http://trac.webkit.org/changeset/65211> All reviewed patches have been landed. Closing bug. |