Remove AsyncSetOwnerUser() from cryptohome library

Remove AsyncSetOwnerUser() from cryptohome library

The fix for issue chromium-os:25464 will make AsyncSetOwnerUser()
obsolete. This CL removes the method and any calls to it.

BUG=chromium:111328
TEST=browser_test ExistingUserControllerTest.*
CQ-DEPEND=I2c085b927fd5412edf02f9026cad5dd7fc608ccc


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=120688

[use bartfab instead, 2012-01-30 15:15:12]

Hi Dave, could you have a look at this? The Chrome OS side has been picked up by
the Commit Queue already, I would like to continue with the Chrome changes.

[Nikita (slow), 2012-02-03 12:14:32]

Denis Glotov is a better reviewer.

[Nikita (slow), 2012-02-03 12:14:45]

as he added this call.

[use bartfab instead, 2012-02-03 12:20:12]

I saw that Denis added the call but figured I should ask one of the owners for
review. Denis, if you don't mind, could you review this?

[glotov, 2012-02-06 11:41:30]

LGTM.

Just a question. How does cryptohomed know the owner now?

[use bartfab instead, 2012-02-06 11:43:27]

Cryptohomed now uses libpolicy which extracts this information from the device's
policy blob, see:

http://crosbug.com/25464

[commit-bot: I haz the power, 2012-02-06 11:43:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@google.com/9286003/11002

[commit-bot: I haz the power, 2012-02-06 11:43:50]

Can't process patch for file
chrome/browser/chromeos/login/existing_user_controller_browsertest.cc.
File's status is None, patchset upload is incomplete.

[commit-bot: I haz the power, 2012-02-06 11:51:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@google.com/9286003/9009

[commit-bot: I haz the power, 2012-02-06 11:51:42]

Presubmit check for 9286003-9009 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for:
chrome/browser/chromeos/cros/mock_cryptohome_library.h,chrome/browser/chromeos/cros/mock_cryptohome_library.cc,chrome/browser/chromeos/cros/cryptohome_library.h,chrome/browser/chromeos/cros/cryptohome_library.cc

Presubmit checks took 1.5s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[use bartfab instead, 2012-02-06 11:53:26]

Hi Satoru, could you review with your cros hat on?

[satorux1, 2012-02-06 16:16:45]

LGTM. Removing code from 'cros' is awesome. :)

[commit-bot: I haz the power, 2012-02-06 16:21:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@google.com/9286003/9009

[commit-bot: I haz the power, 2012-02-06 17:35:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@google.com/9286003/9009

[commit-bot: I haz the power, 2012-02-06 20:13:38]

Try job failure for 9286003-9009 (retry) on mac_rel for steps "browser_tests,
ui_tests".
It's a second try, previously, steps "base_unittests, browser_tests, ui_tests"
failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-02-06 20:25:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@google.com/9286003/9009

[commit-bot: I haz the power, 2012-02-06 22:40:20]

Try job failure for 9286003-9009 (retry) on mac_rel for step "ui_tests".
It's a second try, previously, step "ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-02-06 22:43:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bartfab@google.com/9286003/9009

[commit-bot: I haz the power, 2012-02-07 02:06:27]

Change committed as 120688