Support set Bluetooth signal strength per advertisement |
||||
Issue descriptionCTS-Verifier want us to support signal strength per each advertisement slot when we declared that multi advertisement is support
,
Oct 21 2016
multi-advertisement is "should have" in Android CDD. So failing the CTS verifier override that.
,
Oct 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4438f8fe974c55d1d8c4f515feab9763aad2b530 commit 4438f8fe974c55d1d8c4f515feab9763aad2b530 Author: puthik <puthik@chromium.org> Date: Sat Oct 22 02:49:55 2016 arc: bluetooth: Set advertisement max slot to 1 CTS Verifier want us to also support multiple signal strength when we declared that multi-advertisement is supported. Disable multiadvertisement until that is done. BUG= 658387 ,658385 TEST=CTS Verifier LE signal strength test won't show up Review-Url: https://chromiumcodereview.appspot.com/2438373002 Cr-Commit-Position: refs/heads/master@{#426958} [modify] https://crrev.com/4438f8fe974c55d1d8c4f515feab9763aad2b530/components/arc/bluetooth/arc_bluetooth_bridge.h [modify] https://crrev.com/4438f8fe974c55d1d8c4f515feab9763aad2b530/components/arc/bluetooth/arc_bluetooth_bridge_unittest.cc
,
Oct 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1c82de058b24a0bf58d9303ea06e884f7f49af87 commit 1c82de058b24a0bf58d9303ea06e884f7f49af87 Author: Elijah Taylor <elijahtaylor@google.com> Date: Sat Oct 22 04:10:00 2016 arc: bluetooth: Set advertisement max slot to 1 CTS Verifier want us to also support multiple signal strength when we declared that multi-advertisement is supported. Disable multiadvertisement until that is done. BUG= 658387 ,658385 TEST=CTS Verifier LE signal strength test won't show up Review-Url: https://chromiumcodereview.appspot.com/2438373002 Cr-Commit-Position: refs/heads/master@{#426958} (cherry picked from commit 4438f8fe974c55d1d8c4f515feab9763aad2b530) Review URL: https://codereview.chromium.org/2442133003 . Cr-Commit-Position: refs/branch-heads/2883@{#236} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [modify] https://crrev.com/1c82de058b24a0bf58d9303ea06e884f7f49af87/components/arc/bluetooth/arc_bluetooth_bridge.h [modify] https://crrev.com/1c82de058b24a0bf58d9303ea06e884f7f49af87/components/arc/bluetooth/arc_bluetooth_bridge_unittest.cc
,
Oct 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/debab5eebcaf1650dbca02fc1a8702fa41ea41ce commit debab5eebcaf1650dbca02fc1a8702fa41ea41ce Author: Elijah Taylor <elijahtaylor@google.com> Date: Sat Oct 22 04:44:39 2016 [M54] arc: bluetooth: Set advertisement max slot to 1 CTS Verifier want us to also support multiple signal strength when we declared that multi-advertisement is supported. Disable multiadvertisement until that is done. BUG= 658387 , 658385 TEST=CTS Verifier LE signal strength test won't show up (spiritual cherry-pick of 4438f8fe974c55d1d8c4f515feab9763aad2b530) R=puthik@chromium.org Review URL: https://codereview.chromium.org/2444673002 . Cr-Commit-Position: refs/branch-heads/2840@{#768} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/debab5eebcaf1650dbca02fc1a8702fa41ea41ce/components/arc/bluetooth/arc_bluetooth_bridge.cc
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1c82de058b24a0bf58d9303ea06e884f7f49af87 commit 1c82de058b24a0bf58d9303ea06e884f7f49af87 Author: Elijah Taylor <elijahtaylor@google.com> Date: Sat Oct 22 04:10:00 2016 arc: bluetooth: Set advertisement max slot to 1 CTS Verifier want us to also support multiple signal strength when we declared that multi-advertisement is supported. Disable multiadvertisement until that is done. BUG= 658387 ,658385 TEST=CTS Verifier LE signal strength test won't show up Review-Url: https://chromiumcodereview.appspot.com/2438373002 Cr-Commit-Position: refs/heads/master@{#426958} (cherry picked from commit 4438f8fe974c55d1d8c4f515feab9763aad2b530) Review URL: https://codereview.chromium.org/2442133003 . Cr-Commit-Position: refs/branch-heads/2883@{#236} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [modify] https://crrev.com/1c82de058b24a0bf58d9303ea06e884f7f49af87/components/arc/bluetooth/arc_bluetooth_bridge.h [modify] https://crrev.com/1c82de058b24a0bf58d9303ea06e884f7f49af87/components/arc/bluetooth/arc_bluetooth_bridge_unittest.cc
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/debab5eebcaf1650dbca02fc1a8702fa41ea41ce commit debab5eebcaf1650dbca02fc1a8702fa41ea41ce Author: Elijah Taylor <elijahtaylor@google.com> Date: Sat Oct 22 04:44:39 2016 [M54] arc: bluetooth: Set advertisement max slot to 1 CTS Verifier want us to also support multiple signal strength when we declared that multi-advertisement is supported. Disable multiadvertisement until that is done. BUG= 658387 , 658385 TEST=CTS Verifier LE signal strength test won't show up (spiritual cherry-pick of 4438f8fe974c55d1d8c4f515feab9763aad2b530) R=puthik@chromium.org Review URL: https://codereview.chromium.org/2444673002 . Cr-Commit-Position: refs/branch-heads/2840@{#768} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/debab5eebcaf1650dbca02fc1a8702fa41ea41ce/components/arc/bluetooth/arc_bluetooth_bridge.cc
,
Oct 30 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
||||
►
Sign in to add a comment |
||||
Comment 1 by st...@chromium.org
, Oct 21 2016