Re: Call for review: Vibration test suite update

On 24 Jun 2014, at 18:34, Zhang, Zhiqiang <zhiqiang.zhang@intel.com> wrote:

> When I tried to run the Vibration tests on Chrome 35 for Android, I found there are several issues on the current being test suite:
> 
> - manual (user interaction) tests with testharness.js reference will result in TIMEOUT though the tests pass
> - short timeout setting in asynchronous tests probably result in TIMEOUT
> - [] in test filename will get {"error": {"message": "", "code": 404}}
> 
> So I refined the tests at
> https://github.com/w3c/web-platform-tests/pull/1071
> 
> and tried the updated tests at
> <suppressed>
> 
> all the tests passed on Chrome 35 for Android.
> 
> Please help review the pull request; a simple LGTM can help me land it.

Dom reviewed the tests refined by Zhiqiang and they’ve landed at:

http://w3c-test.org/vibration/

Thanks Zhiqiang and Dom!

-Anssi

Received on Wednesday, 25 June 2014 13:28:09 UTC