Move AsyncTask helpers for accounts and features getters to AccountManagerHelper

Move AsyncTask helpers for accounts and features getters to AccountManagerHelper

This CL moves AsyncTask management from AccountManagerHelper delegates to
AccountManagerHelper itself (see bug for details).

BUG=698258
Review-Url: https://codereview.chromium.org/2778443002
Cr-Commit-Position: refs/heads/master@{#459736}
Committed: https://chromium.googlesource.com/chromium/src/+/4a6245e626500c412e86e74e110542d33408e679

[bsazonov, 2017-03-24 15:21:40]

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

[commit-bot: I haz the power, 2017-03-24 15:22:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-24 15:59:42]

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

[commit-bot: I haz the power, 2017-03-24 15:59:43]

Dry run: This issue passed the CQ dry run.

[bsazonov, 2017-03-24 16:17:38]

bsazonov@chromium.org changed reviewers:
+ msarda@chromium.org

[bsazonov, 2017-03-24 16:17:39]

Mihai, please take a look.

[msarda, 2017-03-27 09:09:10]

LGTM with a nit.

https://codereview.chromium.org/2778443002/diff/1/components/signin/core/brow...
File
components/signin/core/browser/android/java/src/org/chromium/components/signin/AccountManagerHelper.java
(right):

https://codereview.chromium.org/2778443002/diff/1/components/signin/core/brow...
components/signin/core/browser/android/java/src/org/chromium/components/signin/AccountManagerHelper.java:360:
String[] features = {FEATURE_IS_CHILD_ACCOUNT_KEY};
Nit: Consider in-lining the features variable (if it fits 100 characters).

[bsazonov, 2017-03-27 09:27:58]

https://codereview.chromium.org/2778443002/diff/1/components/signin/core/brow...
File
components/signin/core/browser/android/java/src/org/chromium/components/signin/AccountManagerHelper.java
(right):

https://codereview.chromium.org/2778443002/diff/1/components/signin/core/brow...
components/signin/core/browser/android/java/src/org/chromium/components/signin/AccountManagerHelper.java:360:
String[] features = {FEATURE_IS_CHILD_ACCOUNT_KEY};
On 2017/03/27 09:09:09, msarda wrote:
> Nit: Consider in-lining the features variable (if it fits 100 characters).

Done.

[bsazonov, 2017-03-27 09:28:03]

The CQ bit was checked by bsazonov@chromium.org

[bsazonov, 2017-03-27 09:28:03]

The patchset sent to the CQ was uploaded after l-g-t-m from msarda@chromium.org
Link to the patchset: https://codereview.chromium.org/2778443002/#ps20001
(title: "Addressed comment")

[commit-bot: I haz the power, 2017-03-27 09:28:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-27 10:04:15]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490606883323940,
"parent_rev": "eaf7753c64f3d2a1d334e76e2a3882b74148454b", "commit_rev":
"4a6245e626500c412e86e74e110542d33408e679"}

[commit-bot: I haz the power, 2017-03-27 10:05:00]

Description was changed from

==========
Move AsyncTask helpers for accounts and features getters to AccountManagerHelper

This CL moves AsyncTask management from AccountManagerHelper delegates to
AccountManagerHelper itself (see bug for details).

BUG=698258
==========

to

==========
Move AsyncTask helpers for accounts and features getters to AccountManagerHelper

This CL moves AsyncTask management from AccountManagerHelper delegates to
AccountManagerHelper itself (see bug for details).

BUG=698258

Review-Url: https://codereview.chromium.org/2778443002
Cr-Commit-Position: refs/heads/master@{#459736}
Committed:
https://chromium.googlesource.com/chromium/src/+/4a6245e626500c412e86e74e1105...
==========

[commit-bot: I haz the power, 2017-03-27 10:05:01]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4a6245e626500c412e86e74e1105...