Reset BluetoothHostPairingController in an earlier stage for normal user login flow.

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
Committed: https://crrev.com/0532863ace38dac68940ee426576b46cb0baa548
Cr-Commit-Position: refs/heads/master@{#392763}

[xdai1, 2016-05-10 04:24:55]

xdai@chromium.org changed reviewers:
+ achuith@chromium.org

[xdai1, 2016-05-10 04:24:55]

achuith@, could you help review this CL please? Thanks!

[xdai1, 2016-05-10 18:44:10]

On 2016/05/10 04:24:55, xdai1 wrote:
> achuith@, could you help review this CL please? Thanks!

FYI: I'm working on adding a test for this now. But I expect the test could land
in a separate following CL.

[achuithb, 2016-05-10 22:04:55]

The CQ bit was checked by achuith@chromium.org to run a CQ dry run

[achuithb, 2016-05-10 22:05:19]

lgtm

Looking forward to the tests.

https://codereview.chromium.org/1958393002/diff/20001/components/pairing/shar...
File components/pairing/shark_connection_listener.h (right):

https://codereview.chromium.org/1958393002/diff/20001/components/pairing/shar...
components/pairing/shark_connection_listener.h:34: void
ResetHostPairingController();
ResetController is sufficient I think.

[commit-bot: I haz the power, 2016-05-10 22:05:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1958393002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1958393002/20001

[commit-bot: I haz the power, 2016-05-10 23:07:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-10 23:07:02]

Dry run: This issue passed the CQ dry run.

[xdai1, 2016-05-10 23:10:04]

The CQ bit was checked by xdai@chromium.org

[commit-bot: I haz the power, 2016-05-10 23:10:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1958393002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1958393002/20001

[commit-bot: I haz the power, 2016-05-10 23:15:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-10 23:18:28]

Description was changed from

==========
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
==========

to

==========
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

Committed: https://crrev.com/0532863ace38dac68940ee426576b46cb0baa548
Cr-Commit-Position: refs/heads/master@{#392763}
==========

[commit-bot: I haz the power, 2016-05-10 23:18:29]

Patchset 2 (id:??) landed as
https://crrev.com/0532863ace38dac68940ee426576b46cb0baa548
Cr-Commit-Position: refs/heads/master@{#392763}

[achuithb, 2016-05-10 23:44:16]

https://codereview.chromium.org/1958393002/diff/20001/components/pairing/shar...
File components/pairing/shark_connection_listener.h (right):

https://codereview.chromium.org/1958393002/diff/20001/components/pairing/shar...
components/pairing/shark_connection_listener.h:34: void
ResetHostPairingController();
On 2016/05/10 22:05:19, achuithb wrote:
> ResetController is sufficient I think.

ping?

[xdai1, 2016-05-10 23:46:42]

On 2016/05/10 23:44:16, achuithb wrote:
>
https://codereview.chromium.org/1958393002/diff/20001/components/pairing/shar...
> File components/pairing/shark_connection_listener.h (right):
> 
>
https://codereview.chromium.org/1958393002/diff/20001/components/pairing/shar...
> components/pairing/shark_connection_listener.h:34: void
> ResetHostPairingController();
> On 2016/05/10 22:05:19, achuithb wrote:
> > ResetController is sufficient I think.
> 
> ping?

Ahh sorry! I missed this comment... only saw lgtm :( 
Will address it in another CL in a couple minutes.