Fixing typos and errors in webaudio/offlinecontext-promise.html

The layout test webaudio/offlinecontext-promise.html has few minor issues:

- missing semicolon.
- typos.
- wrong indentation and line break.
- both test branches are passing. (there is no test failure.)

BUG=460615
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190699

[hongchan, 2015-02-23 21:29:03]

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

[hongchan, 2015-02-23 21:29:33]

PTAL.

[Raymond Toy, 2015-02-23 21:37:38]

lgtm with one minor nit.

https://codereview.chromium.org/928073004/diff/1/LayoutTests/webaudio/offline...
File LayoutTests/webaudio/offlineaudiocontext-promise.html (right):

https://codereview.chromium.org/928073004/diff/1/LayoutTests/webaudio/offline...
LayoutTests/webaudio/offlineaudiocontext-promise.html:6: <script
src="resources/audio-testing.js"></script>
Does the order matter?

[hongchan, 2015-02-23 21:41:10]

On 2015/02/23 at 21:37:38, rtoy wrote:
> lgtm with one minor nit.
> 
>
https://codereview.chromium.org/928073004/diff/1/LayoutTests/webaudio/offline...
> File LayoutTests/webaudio/offlineaudiocontext-promise.html (right):
> 
>
https://codereview.chromium.org/928073004/diff/1/LayoutTests/webaudio/offline...
> LayoutTests/webaudio/offlineaudiocontext-promise.html:6: <script
src="resources/audio-testing.js"></script>
> Does the order matter?

Technically, it doesn't matter - but I would like to keep it consistent by the
order of scope: js-test library > web audio compat check > our audio testing
utility. I've done the same thing for other promise tests as well.

WDYT?

[Raymond Toy, 2015-02-23 21:53:54]

On 2015/02/23 21:41:10, hoch wrote:
> On 2015/02/23 at 21:37:38, rtoy wrote:
> > lgtm with one minor nit.
> > 
> >
>
https://codereview.chromium.org/928073004/diff/1/LayoutTests/webaudio/offline...
> > File LayoutTests/webaudio/offlineaudiocontext-promise.html (right):
> > 
> >
>
https://codereview.chromium.org/928073004/diff/1/LayoutTests/webaudio/offline...
> > LayoutTests/webaudio/offlineaudiocontext-promise.html:6: <script
> src="resources/audio-testing.js"></script>
> > Does the order matter?
> 
> Technically, it doesn't matter - but I would like to keep it consistent by the
> order of scope: js-test library > web audio compat check > our audio testing
> utility. I've done the same thing for other promise tests as well.
> 
> WDYT?

No objections if you do change the order, but I do think it's unnecessary churn
if it doesn't matter.

[hongchan, 2015-02-23 22:23:25]

The CQ bit was checked by hongchan@chromium.org

[commit-bot: I haz the power, 2015-02-23 22:24:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/928073004/1

[commit-bot: I haz the power, 2015-02-24 00:13:45]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190699