Convert AudioBufferSource sample-accurate-scheduling to testharness

Convert AudioBufferSource sample-accurate-scheduling to testharness

Manually convert test to use testharness and new Audit.

BUG=676394
TEST=sample-accurate-scheduling.html

Review-Url: https://codereview.chromium.org/2635223004
Cr-Commit-Position: refs/heads/master@{#445227}
Committed: https://chromium.googlesource.com/chromium/src/+/65eceecf7418a59d7ee03e32f01fbb8dede34bc6

[Raymond Toy, 2017-01-17 17:36:20]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org

[Raymond Toy, 2017-01-17 17:36:21]

PTAL.

The new output is:

This is a testharness.js-based test.
PASS # AUDIT TASK RUNNER STARTED. 
PASS > [test] Test sample-accurate scheduling 
PASS   Content of left and right channels match and is identical to the array
[1,0,0,1,0,0,0,0,0,0,0,0,0,0,0,0...]. 
PASS   Non-zero sample found at sample offset 0 is true. 
PASS   Non-zero sample found at sample offset 3 is true. 
PASS   Non-zero sample found at sample offset 512 is true. 
PASS   Non-zero sample found at sample offset 517 is true. 
PASS   Non-zero sample found at sample offset 1000 is true. 
PASS   Non-zero sample found at sample offset 1005 is true. 
PASS   Non-zero sample found at sample offset 20000 is true. 
PASS   Non-zero sample found at sample offset 21234 is true. 
PASS   Non-zero sample found at sample offset 37590 is true. 
PASS   Number of impulses found is equal to 9. 
PASS   bad offset is equal to 0. 
PASS < [test] All assertions passed. (total 12 assertions) 
PASS # AUDIT TASK RUNNER FINISHED: 1 tasks ran successfully. 
Harness: the test ran to completion.

[hongchan, 2017-01-18 18:25:15]

https://codereview.chromium.org/2635223004/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html
(right):

https://codereview.chromium.org/2635223004/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html:76:
success = false;
Is this |success| used some where?

[Raymond Toy, 2017-01-18 19:21:36]

https://codereview.chromium.org/2635223004/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html
(right):

https://codereview.chromium.org/2635223004/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html:76:
success = false;
On 2017/01/18 18:25:15, hongchan (OOO 1.17-1.19) wrote:
> Is this |success| used some where?

Nope. I'll remove it.

[hongchan, 2017-01-20 18:31:17]

https://codereview.chromium.org/2635223004/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html
(right):

https://codereview.chromium.org/2635223004/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html:93:
context = new OfflineAudioContext(2, sampleRate * lengthInSeconds, sampleRate);
Wrap at 80.

https://codereview.chromium.org/2635223004/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html:105:
.then(task.done.bind(task));;
Two semicolon.

[Raymond Toy, 2017-01-20 19:37:31]

https://codereview.chromium.org/2635223004/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html
(right):

https://codereview.chromium.org/2635223004/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html:93:
context = new OfflineAudioContext(2, sampleRate * lengthInSeconds, sampleRate);
On 2017/01/20 18:31:17, hongchan wrote:
> Wrap at 80.

Done.

https://codereview.chromium.org/2635223004/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/sample-accurate-scheduling.html:105:
.then(task.done.bind(task));;
On 2017/01/20 18:31:17, hongchan wrote:
> Two semicolon.

Done.

[hongchan, 2017-01-20 23:19:20]

lgtm

[Raymond Toy, 2017-01-20 23:24:30]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-01-20 23:24:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-21 00:33:06]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1484954670204640,
"parent_rev": "68c7753dd291fbf573dc67c258284440f5aad6ff", "commit_rev":
"65eceecf7418a59d7ee03e32f01fbb8dede34bc6"}

[commit-bot: I haz the power, 2017-01-21 00:33:39]

Description was changed from

==========
Convert AudioBufferSource sample-accurate-scheduling to testharness

Manually convert test to use testharness and new Audit.

BUG=676394
TEST=sample-accurate-scheduling.html
==========

to

==========
Convert AudioBufferSource sample-accurate-scheduling to testharness

Manually convert test to use testharness and new Audit.

BUG=676394
TEST=sample-accurate-scheduling.html

Review-Url: https://codereview.chromium.org/2635223004
Cr-Commit-Position: refs/heads/master@{#445227}
Committed:
https://chromium.googlesource.com/chromium/src/+/65eceecf7418a59d7ee03e32f01f...
==========

[commit-bot: I haz the power, 2017-01-21 00:33:40]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/65eceecf7418a59d7ee03e32f01f...