Bug 71022 - [GTK] fast/canvas/canvas-composite-image.html and -canvas.html fails
Summary: [GTK] fast/canvas/canvas-composite-image.html and -canvas.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-27 08:53 PDT by Philippe Normand
Modified: 2012-05-05 10:02 PDT (History)
4 users (show)

See Also:

Attachments
Test diff (3.37 KB, text/plain)
2011-10-27 08:53 PDT, Philippe Normand 		no flags Details
Added skipped test (1.06 KB, patch)
2011-11-06 16:13 PST, Ben Wells 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 2011-10-27 08:53:41 PDT 
Created attachment 112690 [details]
Test diff

Consistent failure. Diff attached.
Comment 1 Ben Wells 2011-11-06 16:02:53 PST 
(In reply to comment #0)
> Created an attachment (id=112690) [details]
> Test diff
> 
> Consistent failure. Diff attached.

fast/canvas/canvas-composite-canvas.html was just added and also fails. This test is almost identical to the canvas-composite-image.html test (testing drawImage(canvas) instead of drawImage(image) in an identifcal manner).

I will add to skip file in same section. As I don't have cq access I'll push through this bug and then reopen.
Comment 2 Ben Wells 2011-11-06 16:13:48 PST 
Created attachment 113811 [details]
Added skipped test
Comment 3 WebKit Review Bot 2011-11-06 22:48:37 PST 
Comment on attachment 113811 [details]
Added skipped test

Clearing flags on attachment: 113811

Committed r99396: <http://trac.webkit.org/changeset/99396>
Comment 4 WebKit Review Bot 2011-11-06 22:48:42 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Philippe Normand 2011-11-06 23:46:27 PST 
(In reply to comment #1)

> I will add to skip file in same section. As I don't have cq access I'll push through this bug and then reopen.

Strange way to use the cq for skipping tests...
Comment 6 Ben Wells 2011-11-07 00:16:40 PST 
(In reply to comment #5)
> Strange way to use the cq for skipping tests...
Would it be better to use a new bug?
Comment 7 Zan Dobersek 2012-05-05 10:02:55 PDT 
The tests covered by this bug were found to be passing consistently after moving from using the Skipped list to using test_expectations.txt. Their expectations were removed in r116122[1] (covered by bug #85591). Closing the bug.

1: http://trac.webkit.org/changeset/116122