bluetooth: Add optional 'name' support to MockBluetoothDevice.

bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However,
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438
Refactor TBR for proximity test:
TBR=msarda@chromium.org

Committed: https://crrev.com/16585eebf8d69b201c64127acf3f47300d1013e5
Cr-Commit-Position: refs/heads/master@{#409298}

[scheib, 2016-07-29 22:03:21]

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

[commit-bot: I haz the power, 2016-07-29 22:03:40]

Dry run: 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, 2016-07-30 01:54:32]

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

[commit-bot: I haz the power, 2016-07-30 01:54:33]

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

[scheib, 2016-07-30 02:39:02]

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

[commit-bot: I haz the power, 2016-07-30 02:39:21]

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

[scheib, 2016-07-30 02:41:57]

Description was changed from

==========
bluetooth: Add optional name support to MockBluetoothDevice.

The mock previously always provided a name value.

BUG=586438
==========

to

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Immediately needed for:
 bluetooth: Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002 Web Bluetooth updated

BUG=586438
==========

[scheib, 2016-07-30 02:41:57]

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

[scheib, 2016-07-30 02:45:07]

Description was changed from

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Immediately needed for:
 bluetooth: Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002 Web Bluetooth updated

BUG=586438
==========

to

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 bluetooth: Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002 Web Bluetooth updated

BUG=586438
==========

[scheib, 2016-07-30 02:45:27]

Description was changed from

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 bluetooth: Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002 Web Bluetooth updated

BUG=586438
==========

to

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438
==========

[commit-bot: I haz the power, 2016-07-30 04:14:30]

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

[commit-bot: I haz the power, 2016-07-30 04:14:31]

Dry run: This issue passed the CQ dry run.

[scheib, 2016-08-02 00:55:01]

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

[commit-bot: I haz the power, 2016-08-02 00:55:16]

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

[scheib, 2016-08-02 00:57:37]

ping ortuno

[commit-bot: I haz the power, 2016-08-02 02:06:53]

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

[commit-bot: I haz the power, 2016-08-02 02:06:54]

Dry run: This issue passed the CQ dry run.

[scheib, 2016-08-02 03:17:54]

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

[commit-bot: I haz the power, 2016-08-02 03:18:16]

Dry run: 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, 2016-08-02 03:27:33]

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

[commit-bot: I haz the power, 2016-08-02 03:27:33]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[scheib, 2016-08-02 18:39:56]

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

[scheib, 2016-08-02 18:40:12]

Patchset #3 (id:40001) has been deleted

[scheib, 2016-08-02 18:40:26]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-08-02 18:40:30]

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

[scheib, 2016-08-02 18:40:35]

Patchset #1 (id:20001) has been deleted

[scheib, 2016-08-02 18:59:26]

PTAL ortuno

[ortuno, 2016-08-02 19:11:42]

lgtm

[scheib, 2016-08-02 20:09:04]

The CQ bit was unchecked by scheib@chromium.org

[scheib, 2016-08-02 20:09:05]

The CQ bit was checked by scheib@chromium.org

[scheib, 2016-08-02 20:09:12]

The CQ bit was unchecked by scheib@chromium.org

[scheib, 2016-08-02 20:10:37]

Description was changed from

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438
==========

to

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438

Refactor TBR for proximity test:
TBR=msarda@chromium.org
==========

[scheib, 2016-08-02 20:10:37]

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

[scheib, 2016-08-02 20:10:47]

The CQ bit was checked by scheib@chromium.org

[commit-bot: I haz the power, 2016-08-02 20:11:45]

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, 2016-08-02 20:16:35]

Description was changed from

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438

Refactor TBR for proximity test:
TBR=msarda@chromium.org
==========

to

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438

Refactor TBR for proximity test:
TBR=msarda@chromium.org
==========

[commit-bot: I haz the power, 2016-08-02 20:16:36]

Committed patchset #1 (id:60001)

[commit-bot: I haz the power, 2016-08-02 20:18:06]

Description was changed from

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However, 
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438

Refactor TBR for proximity test:
TBR=msarda@chromium.org
==========

to

==========
bluetooth: Add optional 'name' support to MockBluetoothDevice.

The mock previously always provided a name value. However,
the base supports the name not existing, and tests need
this mock to be able to return nullopt as well.

Split out from this following patch:
 Web Bluetooth can filter by empty device names, doesn't leak MACs.
 http://crrev.com/2014473002

BUG=586438

Refactor TBR for proximity test:
TBR=msarda@chromium.org

Committed: https://crrev.com/16585eebf8d69b201c64127acf3f47300d1013e5
Cr-Commit-Position: refs/heads/master@{#409298}
==========

[commit-bot: I haz the power, 2016-08-02 20:18:07]

Patchset 1 (id:??) landed as
https://crrev.com/16585eebf8d69b201c64127acf3f47300d1013e5
Cr-Commit-Position: refs/heads/master@{#409298}