[M53-cherrypick] Migrate ArcUserDataService into ArcAuthService.

Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258, 636218
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.
NOTRY=true
NOPRESUBMIT=true
TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)

[hidehiko, 2016-08-19 06:16:35]

Patchset #2 (id:20001) has been deleted

[hidehiko, 2016-08-19 06:19:40]

Description was changed from

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

to

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

[hidehiko, 2016-08-19 06:22:50]

hidehiko@chromium.org changed reviewers:
+ dspaid@chromium.org, khmel@chromium.org, lhchavez@chromium.org,
stevenjb@chromium.org

[hidehiko, 2016-08-19 06:22:51]

Yury, thank you for pointing out compile error, and sorry for inconvenience.

This is relanding it.
PS1 is original cherry pick CL, and PS2 is the compile error fix.
PTAL.

TBRing other reviewers, as other part is not touched.

[hidehiko, 2016-08-19 06:23:34]

Description was changed from

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

to

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

[Luis Héctor Chávez, 2016-08-19 06:24:09]

lgtm if it compiles

[dspaid, 2016-08-19 06:36:38]

lgtm

[hidehiko, 2016-08-19 06:38:01]

Description was changed from

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

to

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258, 636218
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

[hidehiko, 2016-08-23 02:03:37]

On 2016/08/19 06:36:38, dspaid wrote:
> lgtm

Thank you for review.

Yury, friendly ping?

[khmel, 2016-08-24 01:05:12]

On 2016/08/23 02:03:37, hidehiko wrote:
> On 2016/08/19 06:36:38, dspaid wrote:
> > lgtm
> 
> Thank you for review.
> 
> Yury, friendly ping?

lgtm

[hidehiko, 2016-08-25 01:25:08]

The CQ bit was checked by hidehiko@chromium.org

[commit-bot: I haz the power, 2016-08-25 01:26:02]

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, 2016-08-25 01:26:03]

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

[commit-bot: I haz the power, 2016-08-25 01:26:05]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[hidehiko, 2016-08-25 01:32:42]

Description was changed from

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258, 636218
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.

TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

to

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258, 636218
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.
NOTRY=true
NOPRESUBMIT=true
TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

[hidehiko, 2016-08-25 01:32:48]

The CQ bit was checked by hidehiko@chromium.org

[commit-bot: I haz the power, 2016-08-25 01:34:05]

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, 2016-08-25 02:05:46]

Description was changed from

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258, 636218
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.
NOTRY=true
NOPRESUBMIT=true
TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

to

==========
Migrate ArcUserDataService into ArcAuthService.

There are race problems around data removing.
One of the cause is that state management is done in
ArcAuthService and ArcUserDataService separately.

This CL migrates ArcUserDataService into ArcAuthService
so that we can remove the data when necessary.

BUG=633258, 636218
TEST=Ran manually.
     - "Enable -> ARC boot wait -> Disable" triggers data removal after
       ARC stop.
     - "Enable -> Disable before ARC boot" triggers immediate data
       removal.
     - "Enable -> ARC boot wait -> Kill ARC instance" does not trigger
       data removal.
NOTRY=true
NOPRESUBMIT=true
TBR=dspaid@chromium.org, lhchavez@chromium.org, stevenjb@chromium.org

Review-Url: https://codereview.chromium.org/2209173002
Cr-Commit-Position: refs/heads/master@{#411923}
(cherry picked from commit 6d95ad318e992261c45f5a88a0a599663a6d66ae)
==========

[commit-bot: I haz the power, 2016-08-25 02:05:47]

Committed patchset #2 (id:40001)