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

Issue 804036 link

Starred by 1 user
Status: Verified
Owner:
█ sonnysasaka@chromium.org
Last visit > 30 days ago
Closed: Apr 2018
Cc:
r...@chromium.org
█ xiaoyinh@chromium.org
█ mcchou@chromium.org
█ rjahagir@chromium.org
steve...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Multiple clicks of bluetooth power toggle shouldn't be executed all one by one

Project Member Reported by sonnysasaka@chromium.org, Jan 19 2018 
Currently BluetoothPowerController queues all pending power change requests and executes all of them in order. Instead BluetoothPowerController should consolidate all pending power change requests so that only the last pending request is in effect.
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 23 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0499cad5820b0a3ee576575f4286fb2b760fdaf9

commit 0499cad5820b0a3ee576575f4286fb2b760fdaf9
Author: Sonny Sasaka <sonnysasaka@chromium.org>
Date: Tue Jan 23 03:29:19 2018

Consolidate multi pending bluetooth power change tasks.

BluetoothPowerController to consolidate all multiple pending bluetooth
power change requests and only execute the last one.

BUG= 804036 
TEST=Simulate slow bluetooth power change and test manually with
multiple clicks to bluetooth system tray toggle. Make sure that only the
last request is performed.

Change-Id: I3faec228ff9b427bc4f0a514fd22a145a006a337
Reviewed-on: https://chromium-review.googlesource.com/877388
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Sonny Sasaka <sonnysasaka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531142}
[modify] https://crrev.com/0499cad5820b0a3ee576575f4286fb2b760fdaf9/ash/system/bluetooth/bluetooth_power_controller.cc
[modify] https://crrev.com/0499cad5820b0a3ee576575f4286fb2b760fdaf9/ash/system/bluetooth/bluetooth_power_controller.h
[modify] https://crrev.com/0499cad5820b0a3ee576575f4286fb2b760fdaf9/ash/system/bluetooth/bluetooth_power_controller_unittest.cc

Comment 2 by sonnysasaka@chromium.org, Apr 4 2018

Status: Fixed (was: Assigned)

Comment 3 by pbath...@chromium.org, Apr 5 2018

Status: Verified (was: Fixed)
►

Sign in to add a comment