Bug 44773 - REGRESSION: some tests no longer painting
Summary: REGRESSION: some tests no longer painting
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nikolas Zimmermann
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-27 09:43 PDT by Tony Chang
Modified: 2010-08-30 09:17 PDT (History)
5 users (show)

See Also:


Attachments
Patch (2.10 KB, patch)
2010-08-27 23:53 PDT, Nikolas Zimmermann
krit: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2010-08-27 09:43:22 PDT
After http://trac.webkit.org/changeset/66223, three tests started failing pixel tests on the Chromium bots:

  fast/canvas/canvas-as-image.html
  fast/canvas/canvas-as-image-incremental-repaint.html
  fast/gradients/generated-gradients.html

It seems that they're not painting the background at all.

I'm building webkit-mac to see if this also happens in Safari.
Comment 2 Nikolas Zimmermann 2010-08-27 13:04:24 PDT
Thanks for the report, I'll investigate tomorrow, about to go to bed.
Are you sure these are the only tests which are broken?
Comment 3 Tony Chang 2010-08-27 13:32:23 PDT
(In reply to comment #2)
> Thanks for the report, I'll investigate tomorrow, about to go to bed.
> Are you sure these are the only tests which are broken?

These are the only layout tests that regressed on the chromium bots (across windows, mac and linux).  It's possible for there to be a test that chromium was already failing, in which case a regression would not be noticed.
Comment 4 Tony Chang 2010-08-27 18:44:23 PDT
I suspect fast/canvas/webgl/css-webkit-canvas-repaint.html and fast/canvas/webgl/css-webkit-canvas.html are failing for the same reason.  I bet the pixel results were generated before r66223 was landed.
Comment 5 Nikolas Zimmermann 2010-08-27 23:53:53 PDT
Created attachment 65813 [details]
Patch
Comment 6 Dirk Schulze 2010-08-27 23:56:25 PDT
Comment on attachment 65813 [details]
Patch

r=me
Comment 7 Nikolas Zimmermann 2010-08-27 23:57:03 PDT
Landed in r66299.

Tony, can you verify the regression is gone?
All tests you mentioned work again for me in Safari.
Comment 8 WebKit Review Bot 2010-08-28 00:30:54 PDT
http://trac.webkit.org/changeset/66299 might have broken Qt Linux Release
Comment 9 Tony Chang 2010-08-30 09:17:47 PDT
(In reply to comment #7)
> Landed in r66299.
> 
> Tony, can you verify the regression is gone?
> All tests you mentioned work again for me in Safari.

Yes, the regression is gone.  Thanks!