New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 658385 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Support set Bluetooth signal strength per advertisement

Project Member Reported by puthik@chromium.org, Oct 21 2016

Issue description

CTS-Verifier want us to support signal strength per each advertisement slot when we declared that multi advertisement is support
 

Comment 1 by st...@chromium.org, Oct 21 2016

Don't we need to provide multi-advertisement to pass other tests?

Comment 2 by puthik@chromium.org, Oct 21 2016

multi-advertisement is "should have" in Android CDD. So failing the CTS verifier override that.
Project Member

Comment 3 by bugdroid1@chromium.org, 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

Project Member

Comment 4 by bugdroid1@chromium.org, Oct 22 2016

Labels: merge-merged-2883
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

Project Member

Comment 5 by bugdroid1@chromium.org, Oct 22 2016

Labels: merge-merged-2840
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

Project Member

Comment 6 by bugdroid1@chromium.org, 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

Project Member

Comment 7 by bugdroid1@chromium.org, 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

Project Member

Comment 8 by sheriffbot@chromium.org, Oct 30 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
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