Bug 84813 - Layout Test css3/filters/custom/custom-filter-shader-cache.html is failing
Summary: Layout Test css3/filters/custom/custom-filter-shader-cache.html is failing
Status: RESOLVED DUPLICATE of bug 82372
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Hin-Chung Lam
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-24 18:06 PDT by Hin-Chung Lam
Modified: 2012-07-20 07:05 PDT (History)
4 users (show)

See Also:


Attachments
Patch (64.03 KB, patch)
2012-04-24 18:20 PDT, Hin-Chung Lam
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hin-Chung Lam 2012-04-24 18:06:57 PDT
The following layout test is failing on chromium:

css3/filters/custom/custom-filter-shader-cache.html
css3/filters/custom/effect-custom-combined-missing.html
css3/filters/custom/effect-custom-parameters.html
css3/filters/custom/effect-custom.html

Probable cause:

http://trac.webkit.org/changeset/115123 updates code that does this, possibly a rebaseline is needed.
Comment 1 Hin-Chung Lam 2012-04-24 18:20:37 PDT
Created attachment 138711 [details]
Patch
Comment 2 Hin-Chung Lam 2012-04-24 18:21:33 PDT
I left out css3/filters/custom/effect-custom-parameters.html because the actual results are different in chromium-mac and chromium-linux.
Comment 3 Alexandru Chiculita 2012-04-24 18:30:12 PDT
(In reply to comment #2)
> I left out css3/filters/custom/effect-custom-parameters.html because the actual results are different in chromium-mac and chromium-linux.

This one may be related to https://bugs.webkit.org/show_bug.cgi?id=84067 . I've just seen the bug today.

Is there an easy way to generate expected results on all chromium platforms? CSS Shaders in general render slightly different from one platform to the other or even one GPU to the other.
Comment 4 Hin-Chung Lam 2012-04-24 18:42:16 PDT
On Linux you can run Tools/Scripts/webkit-patch garden-o-matic and click Rebaseline without the suppression in test_expectations.txt. It will will show you expected, actual results and differences.
Comment 5 Alexandru Chiculita 2012-04-25 13:32:54 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > I left out css3/filters/custom/effect-custom-parameters.html because the actual results are different in chromium-mac and chromium-linux.
> 
> This one may be related to https://bugs.webkit.org/show_bug.cgi?id=84067 . I've just seen the bug today.
> 
> Is there an easy way to generate expected results on all chromium platforms? CSS Shaders in general render slightly different from one platform to the other or even one GPU to the other.

I've just run css3/filters/custom/effect-custom-parameters.html on Chromium on Mac and the difference is only about the color. I think we are ok with that result. Can you please include it in your patch?
Comment 6 Hin-Chung Lam 2012-04-25 13:35:01 PDT
Will do.(In reply to comment #5)
> (In reply to comment #3)
> > (In reply to comment #2)
> > > I left out css3/filters/custom/effect-custom-parameters.html because the actual results are different in chromium-mac and chromium-linux.
> > 
> > This one may be related to https://bugs.webkit.org/show_bug.cgi?id=84067 . I've just seen the bug today.
> > 
> > Is there an easy way to generate expected results on all chromium platforms? CSS Shaders in general render slightly different from one platform to the other or even one GPU to the other.
> 
> I've just run css3/filters/custom/effect-custom-parameters.html on Chromium on Mac and the difference is only about the color. I think we are ok with that result. Can you please include it in your patch?

Wiill do.
Comment 7 Alexandru Chiculita 2012-04-25 13:40:36 PDT
It seems there's a duplicate of that. https://bugs.webkit.org/show_bug.cgi?id=82372
Comment 8 Tony Chang 2012-04-27 10:56:13 PDT
@hclam - did this patch land or did you want a review?
Comment 9 Tony Chang 2012-07-19 15:57:07 PDT
Did chromium end up enabling CSS Filters?  Should we rebaseline the tests in css3/filters/custom?

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=css3%2Ffilters%2Fcustom

TestExpectations currently has:

// CSS custom() filters are not currently supported
BUGWK71392 WIN : css3/filters/custom = IMAGE IMAGE+TEXT PASS

We should remove that, no?
Comment 10 Hin-Chung Lam 2012-07-19 15:59:56 PDT
This review was sitting for a long time let me go over this again.

(In reply to comment #9)
> Did chromium end up enabling CSS Filters?  Should we rebaseline the tests in css3/filters/custom?
> 
> http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=css3%2Ffilters%2Fcustom
> 
> TestExpectations currently has:
> 
> // CSS custom() filters are not currently supported
> BUGWK71392 WIN : css3/filters/custom = IMAGE IMAGE+TEXT PASS
> 
> We should remove that, no?
Comment 11 Hin-Chung Lam 2012-07-19 16:06:37 PDT
The patch I uploaded was a gardening effort. 

Closing this bug as this is a duplicate of 82372 introduced by  Chiculita's patch.

*** This bug has been marked as a duplicate of bug 82372 ***
Comment 12 Stephen White 2012-07-20 07:03:43 PDT
(In reply to comment #9)
> Did chromium end up enabling CSS Filters?  Should we rebaseline the tests in css3/filters/custom?

Just to be clear:  CSS Filters are enabled in Chrome, but the custom() GLSL ones (aka CSS Shaders) are not, until the security issues are resolved.

DumpRenderTree has everything enabled, however, so we should be able to rebaseline the tests.

> http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=css3%2Ffilters%2Fcustom
> 
> TestExpectations currently has:
> 
> // CSS custom() filters are not currently supported
> BUGWK71392 WIN : css3/filters/custom = IMAGE IMAGE+TEXT PASS
> 
> We should remove that, no?
Comment 13 Stephen White 2012-07-20 07:05:45 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > I left out css3/filters/custom/effect-custom-parameters.html because the actual results are different in chromium-mac and chromium-linux.
> 
> This one may be related to https://bugs.webkit.org/show_bug.cgi?id=84067 . I've just seen the bug today.
> 
> Is there an easy way to generate expected results on all chromium platforms? CSS Shaders in general render slightly different from one platform to the other or even one GPU to the other.

BTW, Chromium's DumpRenderTree runs GPU tests using the Mesa software renderer, so its results should be independent of the GPU installed on the machine.