Add cross platform Bluetooth testing fixture for Windows.

Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by using cross platform testing fixture.

BUG=579202
Committed: https://crrev.com/512935cc3fb94185ee159d8e60c95dadc3c001e4
Cr-Commit-Position: refs/heads/master@{#371668}

[gogerald1, 2016-01-26 15:58:28]

Description was changed from

==========
address comments


remove not applied fix


format


format


add bluetoothtest fixture

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

to

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202
==========

[gogerald1, 2016-01-26 19:17:15]

Patchset #1 (id:1) has been deleted

[gogerald1, 2016-01-26 19:17:22]

Patchset #1 (id:20001) has been deleted

[gogerald1, 2016-01-26 19:17:27]

Patchset #1 (id:40001) has been deleted

[gogerald1, 2016-01-26 19:17:31]

Patchset #1 (id:60001) has been deleted

[gogerald1, 2016-01-26 19:17:37]

Patchset #1 (id:80001) has been deleted

[gogerald1, 2016-01-26 19:23:46]

gogerald@chromium.org changed reviewers:
+ scheib@chromium.org

[gogerald1, 2016-01-26 19:25:45]

Description was changed from

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202
==========

to

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture. The original CL
https://codereview.chromium.org/1639593002/.

BUG=579202
==========

[gogerald1, 2016-01-26 19:26:01]

Description was changed from

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture. The original CL
https://codereview.chromium.org/1639593002/.

BUG=579202
==========

to

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202
==========

[gogerald1, 2016-01-26 19:46:10]

Hey Vincent, PTAL.

[scheib, 2016-01-26 20:13:47]

LGTM with fixes:

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
File device/bluetooth/test/bluetooth_test_win.cc (right):

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
device/bluetooth/test/bluetooth_test_win.cc:1: // Copyright 2015 The Chromium
Authors. All rights reserved.
2016

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
File device/bluetooth/test/bluetooth_test_win.h (right):

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
device/bluetooth/test/bluetooth_test_win.h:24: void SetUp() override;
Please remove these, they are empty.

[gogerald1, 2016-01-26 20:24:23]

The CQ bit was checked by gogerald@chromium.org

[gogerald1, 2016-01-26 20:24:23]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org
Link to the patchset: https://codereview.chromium.org/1632123003/#ps140001
(title: " ")

[gogerald1, 2016-01-26 20:25:10]

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
File device/bluetooth/test/bluetooth_test_win.cc (right):

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
device/bluetooth/test/bluetooth_test_win.cc:1: // Copyright 2015 The Chromium
Authors. All rights reserved.
On 2016/01/26 20:13:47, scheib wrote:
> 2016

Done.

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
File device/bluetooth/test/bluetooth_test_win.h (right):

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
device/bluetooth/test/bluetooth_test_win.h:24: void SetUp() override;
On 2016/01/26 20:13:47, scheib wrote:
> Please remove these, they are empty.

Done.

[scheib, 2016-01-26 20:25:12]

The CQ bit was unchecked by scheib@chromium.org

[scheib, 2016-01-26 20:25:33]

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
File device/bluetooth/test/bluetooth_test_win.h (right):

https://codereview.chromium.org/1632123003/diff/120001/device/bluetooth/test/...
device/bluetooth/test/bluetooth_test_win.h:24: void SetUp() override;
On 2016/01/26 20:25:10, gogerald1 wrote:
> On 2016/01/26 20:13:47, scheib wrote:
> > Please remove these, they are empty.
> 
> Done.

Both empty methods please.

[scheib, 2016-01-26 20:38:52]

The CQ bit was checked by scheib@chromium.org

[scheib, 2016-01-26 20:38:52]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org
Link to the patchset: https://codereview.chromium.org/1632123003/#ps160001
(title: " ")

[commit-bot: I haz the power, 2016-01-26 20:39:49]

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

[commit-bot: I haz the power, 2016-01-26 20:48:57]

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

[commit-bot: I haz the power, 2016-01-26 20:48:58]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[gogerald1, 2016-01-26 20:59:30]

The CQ bit was checked by gogerald@chromium.org

[gogerald1, 2016-01-26 20:59:30]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org
Link to the patchset: https://codereview.chromium.org/1632123003/#ps180001
(title: " ")

[commit-bot: I haz the power, 2016-01-26 20:59:59]

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

[commit-bot: I haz the power, 2016-01-26 23:41:09]

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

[commit-bot: I haz the power, 2016-01-26 23:41:10]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[scheib, 2016-01-26 23:59:37]

The CQ bit was checked by scheib@chromium.org

[commit-bot: I haz the power, 2016-01-27 00:03:57]

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

[commit-bot: I haz the power, 2016-01-27 01:11:24]

Description was changed from

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202
==========

to

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202
==========

[commit-bot: I haz the power, 2016-01-27 01:11:25]

Committed patchset #5 (id:180001)

[commit-bot: I haz the power, 2016-01-27 01:12:21]

Description was changed from

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202
==========

to

==========
Add cross platform Bluetooth testing fixture for Windows.

This CL is the first step to implement GATT service in Chrome for Windows by
using cross platform testing fixture.

BUG=579202

Committed: https://crrev.com/512935cc3fb94185ee159d8e60c95dadc3c001e4
Cr-Commit-Position: refs/heads/master@{#371668}
==========

[commit-bot: I haz the power, 2016-01-27 01:12:21]

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