Identity API: Use AccountTrackerService for account details

Identity API: Use AccountTrackerService for account details

This changes the chrome.identity.getProfileUserInfo
implementation to use the signin AccountTrackerService as
its source of account details, rather than prefs which
are unavailable on Chrome OS.

BUG=430290
R=rogerta@chromium.org

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

[Ken Rockot(use gerrit already), 2014-11-25 20:50:23]

Roger could you please take a look?

[Roger Tawa OOO till Jul 10th, 2014-11-26 12:32:20]

Lgtm

Thanks for fixing email too. Can you run extensions using identity api in
incognito profiles? If so, what happens? Before and after this change.

[Ken Rockot(use gerrit already), 2014-11-26 17:51:28]

On 2014/11/26 12:32:20, Roger Tawa wrote:
> Lgtm
> 
> Thanks for fixing email too. Can you run extensions using identity api in
> incognito profiles? If so, what happens? Before and after this change.

This change doesn't affect the API within incognito profiles, as
chrome.identity.getProfileUserInfo is not allowed incognito.

[Ken Rockot(use gerrit already), 2014-11-26 18:00:51]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-11-26 18:01:49]

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

[commit-bot: I haz the power, 2014-11-26 19:05:03]

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

[commit-bot: I haz the power, 2014-11-26 19:05:04]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Roger Tawa OOO till Jul 10th, 2014-11-26 19:22:31]

The tests are failing because they set the prefs, but the code no longer uses
the prefs.  You should update the tests to call
AccountTrackerService::SeedAccountInfo() instead.

[Ken Rockot(use gerrit already), 2014-11-26 19:27:57]

On 2014/11/26 19:22:31, Roger Tawa wrote:
> The tests are failing because they set the prefs, but the code no longer uses
> the prefs.  You should update the tests to call
> AccountTrackerService::SeedAccountInfo() instead.

Yep, fix forthcoming. Thanks!

[Ken Rockot(use gerrit already), 2014-11-26 22:44:27]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-11-26 22:45:18]

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

[Ken Rockot(use gerrit already), 2014-11-26 23:43:10]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2014-11-27 00:16:46]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-11-27 00:18:08]

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

[commit-bot: I haz the power, 2014-11-27 01:08:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-11-27 01:09:17]

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