Move token handle obtaining to UserSessionManager

Move token handle obtaining to UserSessionManager

BUG=484690, 484734
R=nkostylev@chromium.org

Committed: https://crrev.com/71ede7fdaf6404ec0f70a60d60f679fee07902a9
Cr-Commit-Position: refs/heads/master@{#329868}

Committed: https://crrev.com/d116d1094079601c3adefc30a597f95690635ecb
Cr-Commit-Position: refs/heads/master@{#330110}

[Nikita (slow), 2015-05-14 16:27:01]

lgtm although TH backfill has to be multi-profiles safe as users may sign in
while offline

[Denis Kuznetsov (DE-MUC), 2015-05-14 16:48:22]

The CQ bit was checked by antrim@chromium.org

[commit-bot: I haz the power, 2015-05-14 16:49:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1129293004/1

[commit-bot: I haz the power, 2015-05-14 17:26:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-14 17:27:49]

Patchset 1 (id:??) landed as
https://crrev.com/71ede7fdaf6404ec0f70a60d60f679fee07902a9
Cr-Commit-Position: refs/heads/master@{#329868}

[Evan Stade, 2015-05-14 19:00:49]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1140823007/ by estade@chromium.org.

The reason for reverting is: most likely culprit for interactive ui test
failures here:
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%....

[Nikita (slow), 2015-05-15 04:05:08]

https://codereview.chromium.org/1129293004/diff/1/chrome/browser/chromeos/log...
File chrome/browser/chromeos/login/session/user_session_manager.cc (right):

https://codereview.chromium.org/1129293004/diff/1/chrome/browser/chromeos/log...
chrome/browser/chromeos/login/session/user_session_manager.cc:1678:
token_handle_util_.reset(
I think you need to make sure that when test instances are shut down underlying
fetcher is destroyed explicitly.

Since this class is singleton you most likely need to define Shutdown() method
and call on Chrome shutdown where we shutdown all services.

[Denis Kuznetsov (DE-MUC), 2015-05-15 15:40:04]

Patchset #2 (id:20001) has been deleted

[Nikita (slow), 2015-05-15 16:14:22]

https://codereview.chromium.org/1129293004/diff/60001/chrome/browser/chromeos...
File chrome/browser/chromeos/chrome_browser_main_chromeos.cc (right):

https://codereview.chromium.org/1129293004/diff/60001/chrome/browser/chromeos...
chrome/browser/chromeos/chrome_browser_main_chromeos.cc:726: // Make sure that
there is no pending URLRequests.
Please move this to line 772.

[Nikita (slow), 2015-05-15 16:17:22]

lgtm

[Nikita (slow), 2015-05-15 16:21:24]

The CQ bit was checked by nkostylev@chromium.org

[Nikita (slow), 2015-05-15 16:21:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
nkostylev@chromium.org
Link to the patchset: https://codereview.chromium.org/1129293004/#ps80001
(title: ".")

[commit-bot: I haz the power, 2015-05-15 16:23:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1129293004/80001

[commit-bot: I haz the power, 2015-05-15 16:55:41]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2015-05-15 16:56:46]

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