Serialize calling of characteristic.writeValue in LayoutTests.

Serialize calling of characteristic.writeValue in LayoutTests.

characteristic.writeValue can not be called multiple times. If the UA is
currently using the Bluetooth system (for example, if we are doing a writeValue)
we may reject promise with a NetworkError. These "GATT operation in progress"
errors may be addressed in the future (like having the UA magically serialize).
See https://github.com/WebBluetoothCG/web-bluetooth/issues/188 for the spec
issue.

Here we are fixing the tests to ensure that we are serializing. Futher work is
required to test for GATT operation in progress errors.

BUG=None

R=ortuno

Review-Url: https://codereview.chromium.org/2664003002
Cr-Commit-Position: refs/heads/master@{#447139}
Committed: https://chromium.googlesource.com/chromium/src/+/490c280a382a28d12640ead9e6d56671b59f7ef6

[dougt, 2017-01-30 23:19:52]

The CQ bit was checked by dougt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-30 23:20:18]

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

[dougt, 2017-01-30 23:22:15]

ortuno, ptal

[ortuno, 2017-01-30 23:28:52]

lgtm

https://codereview.chromium.org/2664003002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/bluetooth/characteristic/writeValue/write-succeeds.html
(right):

https://codereview.chromium.org/2664003002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/bluetooth/characteristic/writeValue/write-succeeds.html:20:
.then(() => characteristic.writeValue(new DataView(new ArrayBuffer(length))))
nit: you are missing a semicolon.

[dougt, 2017-01-30 23:33:35]

The CQ bit was checked by dougt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-30 23:33:46]

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

[dougt, 2017-01-31 00:10:59]

The CQ bit was unchecked by dougt@chromium.org

[dougt, 2017-01-31 00:11:00]

The CQ bit was checked by dougt@chromium.org

[dougt, 2017-01-31 00:11:01]

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/2664003002/#ps20001
(title: "nit fixed")

[commit-bot: I haz the power, 2017-01-31 00:11:28]

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-31 00:48:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485821460952190,
"parent_rev": "c40e9f12fb5165909aaef26dc59b3cc3d6540408", "commit_rev":
"490c280a382a28d12640ead9e6d56671b59f7ef6"}

[commit-bot: I haz the power, 2017-01-31 00:49:04]

Description was changed from

==========
Serialize calling of characteristic.writeValue in LayoutTests.

characteristic.writeValue can not be called multiple times. If the UA is
currently using the Bluetooth system (for example, if we are doing a writeValue)
we may reject promise with a NetworkError. These "GATT operation in progress"
errors may be addressed in the future (like having the UA magically serialize).
See https://github.com/WebBluetoothCG/web-bluetooth/issues/188 for the spec
issue.

Here we are fixing the tests to ensure that we are serializing. Futher work is
required to test for GATT operation in progress errors.

BUG=None

R=ortuno
==========

to

==========
Serialize calling of characteristic.writeValue in LayoutTests.

characteristic.writeValue can not be called multiple times. If the UA is
currently using the Bluetooth system (for example, if we are doing a writeValue)
we may reject promise with a NetworkError. These "GATT operation in progress"
errors may be addressed in the future (like having the UA magically serialize).
See https://github.com/WebBluetoothCG/web-bluetooth/issues/188 for the spec
issue.

Here we are fixing the tests to ensure that we are serializing. Futher work is
required to test for GATT operation in progress errors.

BUG=None

R=ortuno

Review-Url: https://codereview.chromium.org/2664003002
Cr-Commit-Position: refs/heads/master@{#447139}
Committed:
https://chromium.googlesource.com/chromium/src/+/490c280a382a28d12640ead9e6d5...
==========

[commit-bot: I haz the power, 2017-01-31 00:49:04]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/490c280a382a28d12640ead9e6d5...