Bug 92627

This is because I made compositing the default directory for pixel testing. Maybe it was not the best idea. My idea was that maybe we want to run pixel test by default but only for the test cases that affects accelerated compositing. I made it to be a default so that developers run the same set of pixel tests as bots (without the need to know some special configuration). One workaround could be to accept globs, so one could say --pixel-test-directory *. However, given that it's still not very clear how we want to go on with pixel testing, I could rather just remove this default behavior, so that --pixel-tests will turn on pixel testing for all tests again.

Created attachment 155257 [details] Patch

(In reply to comment #1) > This is because I made compositing the default directory for pixel testing. Maybe it was not the best idea. My idea was that maybe we want to run pixel test by default but only for the test cases that affects accelerated compositing. I made it to be a default so that developers run the same set of pixel tests as bots (without the need to know some special configuration). > One workaround could be to accept globs, so one could say --pixel-test-directory *. However, given that it's still not very clear how we want to go on with pixel testing, I could rather just remove this default behavior, so that --pixel-tests will turn on pixel testing for all tests again. Ah, I see it now. The idea was good to switch on pixel tests by default for compositing dir. But unfortunately its sideeffect was that we can't run pixel tests for other dirs without passing --pixel-test-directory ... Now I don't have idea which workaround would be better ... Don't hurry, we have time to find the best solution. ;-) I'm examining the status, stability, ... of pixel testing after the big rebaseline.

> Ah, I see it now. The idea was good to switch on pixel tests by default for > compositing dir. No, I was not enabled pixel testing by default, but restricted --pixel-tests to compositing (by default).

Comment on attachment 155257 [details] Patch Clearing flags on attachment: 155257 Committed r124020: <http://trac.webkit.org/changeset/124020>

All reviewed patches have been landed. Closing bug.

It seems like there's four different things we might want to do ... 1) don't run any pixel tests 2) run a default set 3) run a custom set 4) run all of them One option would be to introduce a magic directory name (e.g., "default"), and you could then say "--pixel-test-directory default --pixel-test-directory svg" , etc. Another option (that I like less) would be to add another command line flag. Another option would be to not have the default set, but it seems like if this "only run some of them" idea is a good permanent solution we'll really want a default set.