Remove sequential_promise_test helper, just use promise_test

Remove sequential_promise_test helper, just use promise_test

The upstream implementation of promise_test() in web-platform-test has
changed to run the tests sequentially, removing the need for the
sequential_promise_test() helper. Tests now just use promise_test()
directly.

BUG=451338
R=ortuno@chromium.org

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

[jsbell, 2015-08-18 20:43:44]

ortuno@ - please take a look?

[ortuno, 2015-08-18 21:14:44]

Thanks!

bluetooth LGTM

I would also take a look at this CL that unified the implementations:
https://codereview.chromium.org/1211623005

Also I had this CL that I never got to land which also removes
testharness-helpers from
tests that included it because of sequential_promise_test:

https://codereview.chromium.org/1216203003

[jsbell, 2015-08-18 22:03:45]

On 2015/08/18 21:14:44, ortuno wrote:
> I would also take a look at this CL that unified the implementations:
> https://codereview.chromium.org/1211623005

What should I look for there?
 
> Also I had this CL that I never got to land which also removes
> testharness-helpers from
> tests that included it because of sequential_promise_test:

Ah, that's a good point. I'll try and remove those too.

[ortuno, 2015-08-18 22:10:07]

On 2015/08/18 at 22:03:45, jsbell wrote:
> On 2015/08/18 21:14:44, ortuno wrote:
> > I would also take a look at this CL that unified the implementations:
> > https://codereview.chromium.org/1211623005
> 
> What should I look for there?

I would look at the reviewers and the discussion since it seems some of those
tests are generated. Also for any files including testharness-helpers because of
the sequential_promise_test

> > Also I had this CL that I never got to land which also removes
> > testharness-helpers from
> > tests that included it because of sequential_promise_test:
> 
> Ah, that's a good point. I'll try and remove those too.

[jsbell, 2015-08-18 22:21:13]

Okay - for now I'm just removing uses of testharness-helpers.js from bluetooth,
as fetch, serviceworker and navigatorconnect has other uses of helper functions.


I have a follow-up CL https://crrev.com/1295633003 that removes most of the
stuff in testharness-helpers.js and uses a PRESUBMIT ensures that all copies
stay in sync. Once that's we'll be down to 3 helpers and I'll work to eliminate
them entirely.

[jsbell, 2015-08-18 22:21:44]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2015-08-18 22:21:44]

The patchset sent to the CQ was uploaded after l-g-t-m from ortuno@chromium.org
Link to the patchset: https://codereview.chromium.org/1295373003/#ps40001
(title: "remove testharness-helpers.js inclusion from bluetooth tests")

[commit-bot: I haz the power, 2015-08-18 22:21:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1295373003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1295373003/40001

[commit-bot: I haz the power, 2015-08-18 23:16:09]

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