[sync] Apply sync config changes only after sync backend is initialized

[sync] Apply sync config changes only after sync backend is initialized

During chrome startup on Win 8, if CredentialCacheService detects sync config changes in the alternate profile, we try to apply them immediately. Instead, we must try to apply config changes only after the sync backend is initialized, and preferably, after local sync configuration is done.

This patch does the following:

1) Modify the constraints in MayReconfigureSync to include the fact that the sync backend has been initialized.
2) In case of sync auto-start, we try to consume config changes, if any, from the alternate profile once local configuration is done.
3) Update ScheduleNextReadFromAlternateCredentialCache to take a delay parameter, so that we can schedule a read of the alternate cache immediately or in the future.

BUG=144302
TEST=Sign in on desktop; launch metro (so it signs in); close desktop; reconfigure metro; restart desktop; make sure config changes from metro are immediately applied on desktop.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=153123

[Raghu Simha, 2012-08-23 00:03:37]

Nicolas, please review. Thanks.

[Nicolas Zea, 2012-08-23 23:00:48]

http://codereview.chromium.org/10872025/diff/1/chrome/browser/sync/credential...
File chrome/browser/sync/credential_cache_service_win.cc (right):

http://codereview.chromium.org/10872025/diff/1/chrome/browser/sync/credential...
chrome/browser/sync/credential_cache_service_win.cc:852:
service->sync_initialized();
would should push changes be more appropriate here?

[Raghu Simha, 2012-08-23 23:15:20]

PTAL.

http://codereview.chromium.org/10872025/diff/1/chrome/browser/sync/credential...
File chrome/browser/sync/credential_cache_service_win.cc (right):

http://codereview.chromium.org/10872025/diff/1/chrome/browser/sync/credential...
chrome/browser/sync/credential_cache_service_win.cc:852:
service->sync_initialized();
On 2012/08/23 23:00:48, nzea wrote:
> would should push changes be more appropriate here?

Good idea. Makes sense to wait for local config to be done before applying any
external config changes. Done.

[Nicolas Zea, 2012-08-23 23:20:18]

lgtm