arc: Fix ArcPackageSyncableService initialization issue.

arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG=637058
TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.

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

[lgcheng, 2016-08-11 17:57:25]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.

BUG=
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.

BUG= http://b/30699487
==========

[lgcheng, 2016-08-11 18:04:42]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.

BUG= http://b/30699487
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.

BUG= http://b/30699487

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

[lgcheng, 2016-08-11 18:09:13]

The CQ bit was checked by lgcheng@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 18:10:00]

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

[lgcheng, 2016-08-11 18:10:27]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.

BUG= http://b/30699487

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is ready.

BUG= http://b/30699487

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

[lgcheng, 2016-08-11 18:10:47]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is ready.

BUG= http://b/30699487

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG= http://b/30699487

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

[lgcheng, 2016-08-11 19:50:27]

lgcheng@google.com changed reviewers:
+ pavely@chromium.org, stevenjb@chromium.org

[lgcheng, 2016-08-11 19:51:16]

Patchset #3 (id:40001) has been deleted

[lgcheng, 2016-08-11 19:56:29]

Hi Guys,

PTAL at this patch. 

Steven, could you review
c/b/ui/app_list/

Pavel, could you review
c/b/sync/

Thanks!

[lgcheng, 2016-08-11 20:17:50]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG= http://b/30699487

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG=637058

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

[stevenjb, 2016-08-11 20:55:21]

c/b/ui/app_list/ lgtm

[lgcheng, 2016-08-11 20:57:40]

On 2016/08/11 20:55:21, stevenjb wrote:
> c/b/ui/app_list/ lgtm

Thanks for review!

[pavely, 2016-08-12 16:59:52]

lgtm

[lgcheng, 2016-08-12 17:01:08]

The CQ bit was checked by lgcheng@google.com to run a CQ dry run

[lgcheng, 2016-08-12 17:01:24]

Thanks for review!

[commit-bot: I haz the power, 2016-08-12 17:01:33]

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, 2016-08-12 17:40:22]

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

[commit-bot: I haz the power, 2016-08-12 17:40:23]

Dry run: This issue passed the CQ dry run.

[lgcheng, 2016-08-12 17:40:44]

The CQ bit was checked by lgcheng@google.com

[commit-bot: I haz the power, 2016-08-12 17:41:36]

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-12 17:45:31]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG=637058

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG=637058

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

[commit-bot: I haz the power, 2016-08-12 17:45:32]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-08-12 17:47:51]

Description was changed from

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG=637058

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.
==========

to

==========
arc: Fix ArcPackageSyncableService initialization issue.
ArcPackageSyncableService should not be started before Arc service is
ready.

BUG=637058

TEST=Pass sync integration test.
TEST=Manual test. No crash when add second user.

Committed: https://crrev.com/a0627b05a72e3ddd8b9fbdba6ab93717868a9921
Cr-Commit-Position: refs/heads/master@{#411696}
==========

[commit-bot: I haz the power, 2016-08-12 17:47:52]

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