bluetooth: Separate connection tests

bluetooth: Separate connection tests and fix one writeValue test

BUG=554240
Committed: https://crrev.com/69cf3eda0120e60cba9413eff1cf7e31dcd3b5b1
Cr-Commit-Position: refs/heads/master@{#391554}

[ortuno, 2016-05-03 20:03:42]

Description was changed from

==========
bluetooth: (WIP) Separate connection tests

BUG=
==========

to

==========
bluetooth: Separate connection tests and fix one writeValue test

BUG=554240
==========

[ortuno, 2016-05-03 20:04:54]

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

[commit-bot: I haz the power, 2016-05-03 20:05:19]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1898263002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1898263002/60001

[ortuno, 2016-05-03 20:46:56]

ortuno@chromium.org changed reviewers:
+ jyasskin@chromium.org

[ortuno, 2016-05-03 20:46:57]

jyasskin: PTAL

[commit-bot: I haz the power, 2016-05-03 21:15:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-03 21:15:11]

Dry run: This issue passed the CQ dry run.

[Jeffrey Yasskin, 2016-05-04 00:49:32]

lgtm

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/bluetooth/connect/get-same-gatt-server.html
(right):

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/bluetooth/connect/get-same-gatt-server.html:16:
});
Please add a name for this test. Maybe "Multiple connects return same object."

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/bluetooth/disconnect-frame-detached/hide-detach.html
(left):

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/bluetooth/disconnect-frame-detached/hide-detach.html:32:
}, 'Hide then detach frame. We shouldn\'t crash.');
This one's just deleted because hiding no longer disconnects, right?

[ortuno, 2016-05-04 16:41:49]

The CQ bit was checked by ortuno@chromium.org

[ortuno, 2016-05-04 16:41:50]

Thanks!

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/bluetooth/connect/get-same-gatt-server.html
(right):

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/bluetooth/connect/get-same-gatt-server.html:16:
});
On 2016/05/04 at 00:49:32, Jeffrey Yasskin wrote:
> Please add a name for this test. Maybe "Multiple connects return same object."

Done.

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/bluetooth/disconnect-frame-detached/hide-detach.html
(left):

https://codereview.chromium.org/1898263002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/bluetooth/disconnect-frame-detached/hide-detach.html:32:
}, 'Hide then detach frame. We shouldn\'t crash.');
On 2016/05/04 at 00:49:32, Jeffrey Yasskin wrote:
> This one's just deleted because hiding no longer disconnects, right?

Yup!

[ortuno, 2016-05-04 16:41:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
jyasskin@chromium.org
Link to the patchset: https://codereview.chromium.org/1898263002/#ps80001
(title: "Add name to test")

[commit-bot: I haz the power, 2016-05-04 16:42:07]

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

[commit-bot: I haz the power, 2016-05-04 17:49:05]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-05-04 17:50:13]

Description was changed from

==========
bluetooth: Separate connection tests and fix one writeValue test

BUG=554240
==========

to

==========
bluetooth: Separate connection tests and fix one writeValue test

BUG=554240

Committed: https://crrev.com/69cf3eda0120e60cba9413eff1cf7e31dcd3b5b1
Cr-Commit-Position: refs/heads/master@{#391554}
==========

[commit-bot: I haz the power, 2016-05-04 17:50:14]

Patchset 5 (id:??) landed as
https://crrev.com/69cf3eda0120e60cba9413eff1cf7e31dcd3b5b1
Cr-Commit-Position: refs/heads/master@{#391554}