Bug 80284 - fast/text/international/spaces-combined-in-vertical-text.html has incorrect expected results
Summary: fast/text/international/spaces-combined-in-vertical-text.html has incorrect e...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-05 07:32 PST by mitz
Modified: 2012-03-05 10:37 PST (History)
3 users (show)

See Also:


Attachments
Update the expected results (1.13 KB, patch)
2012-03-05 07:35 PST, mitz
ap: review+
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2012-03-05 07:32:56 PST
fast/text/international/spaces-combined-in-vertical-text.html has incorrect expected results
Comment 1 mitz 2012-03-05 07:35:07 PST
Created attachment 130128 [details]
Update the expected results
Comment 2 WebKit Review Bot 2012-03-05 08:13:35 PST
Comment on attachment 130128 [details]
Update the expected results

Attachment 130128 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11819030

New failing tests:
fast/text/international/spaces-combined-in-vertical-text.html
Comment 3 mitz 2012-03-05 08:14:54 PST
(In reply to comment #2)
> (From update of attachment 130128 [details])
> Attachment 130128 [details] did not pass chromium-ews (chromium-xvfb):
> Output: http://queues.webkit.org/results/11819030
> 
> New failing tests:
> fast/text/international/spaces-combined-in-vertical-text.html

There’s nothing at <http://queues.webkit.org/results/11819030> about fast/text/international/spaces-combined-in-vertical-text.html.
Comment 4 mitz 2012-03-05 08:19:50 PST
<http://queues.webkit.org/results/11817046> does show the test failing, but doesn’t say what the results are. I suppose it produces the currently-checked-in expected results, which are clearly wrong. I can move them to platform/chromium when landing this patch, and file a bug about this apparent failure.
Comment 5 mitz 2012-03-05 08:23:42 PST
Filed bug 80293 about the Chromium failure.
Comment 6 Alexey Proskuryakov 2012-03-05 10:01:13 PST
Comment on attachment 130128 [details]
Update the expected results

r=me assuming that this makes the test pass in some ports.

It may be more appropriate for chromium to mark the test as expected failure, not to land platform specific results.
Comment 7 mitz 2012-03-05 10:37:32 PST
(In reply to comment #6)
> (From update of attachment 130128 [details])
> r=me assuming that this makes the test pass in some ports.
> 
> It may be more appropriate for chromium to mark the test as expected failure, not to land platform specific results.

Given that there’s a bug tracking the incorrectness of the results in the Chromium port, I kept them in the tree so that any change would be detected.

Fixed in <http://trac.webkit.org/r109767>.