[Sync] Integ tests can't compare progress markers during configuration.

[Sync] Integ tests can't compare progress markers during configuration.

GetActiveDataTypes() returns an empty set during configuration, so
ProgressMarkersMatch could return a false positive if either service was
in the middle of configuration during a check.

BUG=637023
Review-Url: https://codereview.chromium.org/2643643011
Cr-Commit-Position: refs/heads/master@{#445430}
Committed: https://chromium.googlesource.com/chromium/src/+/c4706b070fab0bee61323c702cb91e3405070cc5

[maxbogue, 2017-01-20 18:59:42]

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

[commit-bot: I haz the power, 2017-01-20 18:59:58]

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

[maxbogue, 2017-01-20 19:44:00]

maxbogue@chromium.org changed reviewers:
+ pavely@chromium.org

[maxbogue, 2017-01-20 19:44:00]

Pavel, PTAL!

[commit-bot: I haz the power, 2017-01-20 19:44:58]

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

[commit-bot: I haz the power, 2017-01-20 19:44:59]

Dry run: This issue passed the CQ dry run.

[skym, 2017-01-20 20:06:57]

skym@chromium.org changed reviewers:
+ skym@chromium.org

[skym, 2017-01-20 20:06:58]

lgtm

[pavely, 2017-01-23 17:59:53]

lgtm

[maxbogue, 2017-01-23 18:15:35]

The CQ bit was checked by maxbogue@chromium.org

[commit-bot: I haz the power, 2017-01-23 18:16:10]

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-01-23 19:18:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485195335221490, "parent_rev":
"ea5734bd3270765de34fc7f37b64759fe1d1c1bd", "commit_rev":
"c4706b070fab0bee61323c702cb91e3405070cc5"}

[commit-bot: I haz the power, 2017-01-23 19:18:50]

Description was changed from

==========
[Sync] Integ tests can't compare progress markers during configuration.

GetActiveDataTypes() returns an empty set during configuration, so
ProgressMarkersMatch could return a false positive if either service was
in the middle of configuration during a check.

BUG=637023
==========

to

==========
[Sync] Integ tests can't compare progress markers during configuration.

GetActiveDataTypes() returns an empty set during configuration, so
ProgressMarkersMatch could return a false positive if either service was
in the middle of configuration during a check.

BUG=637023

Review-Url: https://codereview.chromium.org/2643643011
Cr-Commit-Position: refs/heads/master@{#445430}
Committed:
https://chromium.googlesource.com/chromium/src/+/c4706b070fab0bee61323c702cb9...
==========

[commit-bot: I haz the power, 2017-01-23 19:18:51]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c4706b070fab0bee61323c702cb9...