TPMTokenLoader: Report both the disabled and enabled state of the TPM.

TPMTokenLoader: Report both the disabled and enabled state of the TPM.

Also change from the observer pattern to the simpler callback pattern, as there is only a single notification type (TPM is enabled/disabled).
This simplifies ownership of the observer/receiver especially if multiple threads (typically UI and IO) are involved: A temporary callback with a WeakPtr can be used instead of an observer object that has to be owned by someone.

BUG=413219
Committed: https://crrev.com/a52b2d041ac1f2e2905b0b9e5d222cee5a9ff116
Cr-Commit-Position: refs/heads/master@{#294454}

[pneubeck (no reviews), 2014-09-11 13:23:24]

Patchset #1 (id:1) has been deleted

[pneubeck (no reviews), 2014-09-11 13:24:55]

pneubeck@chromium.org changed reviewers:
+ mattm@chromium.org, tbarzic@chromium.org

[pneubeck (no reviews), 2014-09-11 13:24:55]

Toni, please review this one.
I separated this smaller part from the larger CL:
https://codereview.chromium.org/419013003/

Matt, you might want to take a look as well.

[tbarzic, 2014-09-11 16:45:16]

On 2014/09/11 13:24:55, pneubeck wrote:
> Toni, please review this one.
> I separated this smaller part from the larger CL:
> https://codereview.chromium.org/419013003/
> 
> Matt, you might want to take a look as well.

lgtm

[pneubeck (no reviews), 2014-09-11 18:03:15]

pneubeck@chromium.org changed reviewers:
+ stevenjb@chromium.org

[pneubeck (no reviews), 2014-09-11 18:03:15]

Steven please owner review. Thanks!

[stevenjb, 2014-09-11 18:16:55]

lgtm

https://codereview.chromium.org/567553002/diff/20001/chrome/browser/chromeos/...
File chrome/browser/chromeos/ownership/owner_settings_service.cc (right):

https://codereview.chromium.org/567553002/diff/20001/chrome/browser/chromeos/...
chrome/browser/chromeos/ownership/owner_settings_service.cc:363: void
OwnerSettingsService::OnTPMTokenReady(bool token_enabled) {
nit: (bool /* unused token_enabled */)

[pneubeck (no reviews), 2014-09-11 18:48:50]

The CQ bit was checked by pneubeck@chromium.org

[pneubeck (no reviews), 2014-09-11 18:49:11]

https://codereview.chromium.org/567553002/diff/20001/chrome/browser/chromeos/...
File chrome/browser/chromeos/ownership/owner_settings_service.cc (right):

https://codereview.chromium.org/567553002/diff/20001/chrome/browser/chromeos/...
chrome/browser/chromeos/ownership/owner_settings_service.cc:363: void
OwnerSettingsService::OnTPMTokenReady(bool token_enabled) {
On 2014/09/11 18:16:55, stevenjb wrote:
> nit: (bool /* unused token_enabled */)

Done.

[commit-bot: I haz the power, 2014-09-11 18:55:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/567553002/40001

[commit-bot: I haz the power, 2014-09-11 20:17:46]

Committed patchset #2 (id:40001) as 79f5826fac2c5096b0004a6a8eb0f19ec74c0c8d

[commit-bot: I haz the power, 2014-09-11 20:26:50]

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