bluetooth: Only add new devices, connected devices and devices that changed.

bluetooth: Only add new devices, connected devices and devices that changed.

Last of three patches to improve devices in the chooser:

 [1] Replace old advertised uuids (http://crrev.com/2205693003)
 [2] Allow updates on chooser items (http://crrev.com/2210873003)
 [3] This patch

BUG=508904
Committed: https://crrev.com/7992351858fe9bb262e18527c590bbb301243b61
Cr-Commit-Position: refs/heads/master@{#410983}

[ortuno, 2016-08-04 19:55:52]

Description was changed from

==========
bluetooth: Only add new devices, connected devices and devices that changed.

BUG=508904
==========

to

==========
bluetooth: Only add new devices, connected devices and devices that changed.

Last of three patches to improve devices in the chooser:

 [1] Replace old advertised uuids (http://crrev.com/2210873003)
 [2] Allow updates on chooser items (http://crrev.com/2210873003)
 [3] This patch

BUG=508904
==========

[ortuno, 2016-08-05 17:49:56]

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

[commit-bot: I haz the power, 2016-08-05 17:50:22]

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-05 19:16:23]

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

[commit-bot: I haz the power, 2016-08-05 19:16:24]

Dry run: This issue passed the CQ dry run.

[ortuno, 2016-08-08 23:23:16]

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

[ortuno, 2016-08-08 23:23:16]

scheib: PTAL

[ortuno, 2016-08-08 23:23:20]

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

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

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-09 03:39:50]

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

[commit-bot: I haz the power, 2016-08-09 03:39:51]

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-08-09 20:33:43]

LGTM

https://codereview.chromium.org/2217573002/diff/80001/content/shell/browser/l...
File content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h
(right):

https://codereview.chromium.org/2217573002/diff/80001/content/shell/browser/l...
content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h:163:
//      - No UUIDs (A Generic Access UUID (0x1800) us added by
us -> is

[ortuno, 2016-08-09 20:33:46]

Description was changed from

==========
bluetooth: Only add new devices, connected devices and devices that changed.

Last of three patches to improve devices in the chooser:

 [1] Replace old advertised uuids (http://crrev.com/2210873003)
 [2] Allow updates on chooser items (http://crrev.com/2210873003)
 [3] This patch

BUG=508904
==========

to

==========
bluetooth: Only add new devices, connected devices and devices that changed.

Last of three patches to improve devices in the chooser:

 [1] Replace old advertised uuids (http://crrev.com/2205693003)
 [2] Allow updates on chooser items (http://crrev.com/2210873003)
 [3] This patch

BUG=508904
==========

[ortuno, 2016-08-09 22:03:45]

Thanks!

https://codereview.chromium.org/2217573002/diff/80001/content/shell/browser/l...
File content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h
(right):

https://codereview.chromium.org/2217573002/diff/80001/content/shell/browser/l...
content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.h:163:
//      - No UUIDs (A Generic Access UUID (0x1800) us added by
On 2016/08/09 at 20:33:43, scheib wrote:
> us -> is

Done.

[ortuno, 2016-08-09 22:04:00]

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

[commit-bot: I haz the power, 2016-08-09 22:04:25]

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-10 00:49:06]

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

[commit-bot: I haz the power, 2016-08-10 00:49:08]

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_...)

[ortuno, 2016-08-10 05:09:07]

The CQ bit was checked by ortuno@chromium.org

[ortuno, 2016-08-10 05:09:08]

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/2217573002/#ps100001
(title: "Fix typo")

[commit-bot: I haz the power, 2016-08-10 05:09:30]

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-10 06:41:50]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-08-10 06:43:09]

Description was changed from

==========
bluetooth: Only add new devices, connected devices and devices that changed.

Last of three patches to improve devices in the chooser:

 [1] Replace old advertised uuids (http://crrev.com/2205693003)
 [2] Allow updates on chooser items (http://crrev.com/2210873003)
 [3] This patch

BUG=508904
==========

to

==========
bluetooth: Only add new devices, connected devices and devices that changed.

Last of three patches to improve devices in the chooser:

 [1] Replace old advertised uuids (http://crrev.com/2205693003)
 [2] Allow updates on chooser items (http://crrev.com/2210873003)
 [3] This patch

BUG=508904

Committed: https://crrev.com/7992351858fe9bb262e18527c590bbb301243b61
Cr-Commit-Position: refs/heads/master@{#410983}
==========

[commit-bot: I haz the power, 2016-08-10 06:43:11]

Patchset 6 (id:??) landed as
https://crrev.com/7992351858fe9bb262e18527c590bbb301243b61
Cr-Commit-Position: refs/heads/master@{#410983}