Bug 22498 – DRT transition pause tests need improvement

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Dean Jackson]

re LayoutTests/transitions/transition-drt-api-delay.html and transition-shorthand-delay.html

Simon says:
shouldn't use red unless there is failure
make the boxes smaller, make them green, put in a bit more explanatory text
pref. links to bugs
but not too much text
also, the test should say if it only works in DRT

[Pierre-Olivier Latour]

Another issue is that the DRT pause API is not implemented on windows, so you need to update the skip file:
LayoutTests/platform/win/Skipped

[Simon Fraser (smfr)]

What's the bug for implementing the pause API on Windows?

[Pierre-Olivier Latour]

(In reply to comment #2)
> What's the bug for implementing the pause API on Windows?
> 

https://bugs.webkit.org/show_bug.cgi?id=22239

[Dean Jackson]

Created attachment 25637 [details]
cleans up tests

I left the -expected.txt files in the same dir. While the API is mac-only at the moment, there is nothing platform specific about the output.

[Dean Jackson]

	M	LayoutTests/ChangeLog
	M	LayoutTests/platform/win/Skipped
	M	LayoutTests/transitions/transition-drt-api-delay-expected.txt
	M	LayoutTests/transitions/transition-drt-api-delay.html
	M	LayoutTests/transitions/transition-shorthand-delay-expected.txt
	M	LayoutTests/transitions/transition-shorthand-delay.html
Committed r38957