Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(113)

Issue 1971743004: [Merge to M50] Reset BluetoothHostPairingController in an earlier stage for normal user login flow. (Closed)

Created:
4 years, 7 months ago by xdai1
Modified:
4 years, 7 months ago
Reviewers:
achuithb
CC:
chromium-reviews, dzhioev+watch_chromium.org, achuith+watch_chromium.org, oshima+watch_chromium.org, davemoore+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@2661
Target Ref:
refs/pending/branch-heads/2661
Project:
chromium
Visibility:
Public.

Description

[Merge to M50] Reset BluetoothHostPairingController in an earlier stage for normal user login flow. In normal user login flow, the Reset of BluetoothHostPairingController always happens when the controller deletes itself, which makes PowerOffAdapterIfApplicable() not guaranteed to be called. In order to prevent that, we reset the controller in an earlier stage. BUG=chrome-os-partner:52828 TBR=achuith@chromium.org Review-Url: https://codereview.chromium.org/1958393002 Cr-Commit-Position: refs/heads/master@{#392763} (cherry picked from commit 0532863ace38dac68940ee426576b46cb0baa548) Committed: https://chromium.googlesource.com/chromium/src/+/30ecdf5e171a459c00bf658872d7a1081c0eed30

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -1 line) Patch
M chrome/browser/chromeos/login/wizard_controller.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M components/pairing/bluetooth_host_pairing_controller.h View 2 chunks +1 line, -1 line 0 comments Download
M components/pairing/fake_host_pairing_controller.h View 1 chunk +1 line, -0 lines 0 comments Download
M components/pairing/fake_host_pairing_controller.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M components/pairing/host_pairing_controller.h View 1 chunk +3 lines, -0 lines 0 comments Download
M components/pairing/shark_connection_listener.h View 1 chunk +2 lines, -0 lines 0 comments Download
M components/pairing/shark_connection_listener.cc View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
xdai1
4 years, 7 months ago (2016-05-11 20:14:42 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
30ecdf5e171a459c00bf658872d7a1081c0eed30 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698