New issue
Advanced search Search tips

Issue 639995 link

Starred by 3 users
Status: Verified
Owner:
r...@chromium.org
Closed: Aug 2016
Cc:
r...@chromium.org
puthik@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Merge ARC++ Bluetooth bug fix to M53

Project Member Reported by puthik@chromium.org, Aug 22 2016 
Ask for merge request for CLs below. 
This fixed the wrong API call in Bluetooth ARC++ for BT low enegy scan.

http://crrev.com/2244703005

arc: bluetooth: Fix advertised uuid

device->GetUUIDs() is the correct way to get the advertised uuid
for recreate advertise data.

BUG=None
TEST=Build

Committed: https://crrev.com/6de80cb2538a41d1144cae3622011ee2e17c1119
Cr-Commit-Position: refs/heads/master@{#412065}

Comment 1 by dimu@chromium.org, Aug 23 2016

Labels: -Merge-Request-53 Merge-Review-53 Hotlist-Merge-Review
[Automated comment] Less than 2 weeks to go before stable on M53, manual review required.

Comment 2 by st...@chromium.org, Aug 24 2016

Owner: st...@chromium.org

Comment 3 by st...@chromium.org, Aug 25 2016

Cc: -puthik@chromium.org st...@chromium.org
Owner: puthik@chromium.org
Opal, let me know when this is merge approved and I'll do the merge.

Comment 4 by puthik@chromium.org, Aug 25 2016

Cc: puthik@chromium.org
Owner: keta...@chromium.org
+ketakid
Can you look at the merge request?

Rahul, look like I meed to manually add TPM for merge request.

Comment 5 by keta...@chromium.org, Aug 29 2016 

Approving merge to M53 cros.

Comment 6 by keta...@chromium.org, Aug 29 2016

Labels: -Hotlist-Merge-review -Merge-Review-53 Merge-Approved-53

Comment 7 by keta...@chromium.org, Aug 29 2016

Labels: Week-1635 Week-1636

Comment 8 by puthik@chromium.org, Aug 29 2016

Owner: st...@chromium.org
This is approved now

Comment 9 by kathrelk...@chromium.org, Aug 29 2016

Components: OS>Systems>Bluetooth

Comment 10 by st...@chromium.org, Aug 31 2016

Labels: Merge-Merged
Status: Fixed (was: Started)
Merged with: https://codereview.chromium.org/2297383002/
Project Member

Comment 11 by sheriffbot@chromium.org, Sep 2 2016 

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 12 by keta...@chromium.org, Sep 2 2016 

Approving merge to M53 cros. steel@ Please merge this in asap.

Comment 13 by st...@chromium.org, Sep 2 2016 

This is merged. I linked the CL that merged it above.

Comment 14 by puthik@chromium.org, Sep 2 2016

Labels: -Merge-Approved-53
I think you need to remove Merge approve label when it is merged.

Comment 15 by kathrelk...@chromium.org, Sep 28 2016

Status: Verified (was: Fixed)

Comment 16 by st...@chromium.org, Mar 3 2017

Cc: r...@chromium.org

Comment 17 by st...@chromium.org, Mar 3 2017

Cc: -st...@chromium.org

Comment 18 by st...@chromium.org, Mar 3 2017

Owner: r...@chromium.org
►

Sign in to add a comment