Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky

Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky (patchset #2 id:40001 of https://codereview.chromium.org/2479313004/ )

Reason for revert:
Suspected to cause TwoClientUssSyncTest.ConflictResolution flaky failures:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=TwoClientUssSyncTest.ConflictResolution&testType=sync_integration_tests

Original issue's description:
> [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky
>
> Reduce flakiness in TwoClientBookmarksSyncTest by:
> 1) Reducing the nudge delay for sync to 1 ms
> 2) Switching to BookmarksMatchChecker.Wait(), which isn't prone to the
> "stale snapshot" race condition.
>
> Reducing the nudge delay applies to and should speed up all the sync
> integration tests, not just the TwoClientBookmarks ones.
>
> Only pairs of
> AwaitQuiescence|AwaitMutualSyncCycleCompletion
> AllModelsMatch(Verifier?)()
>
> Were converted to use the new pattern.
>
> BUG=660655
>
> R=zea@chromium.org, pavely@chromium.org
>
> Committed: https://crrev.com/3b8f13960ccb28bf854a91f05878b95529c32d79
> Cr-Commit-Position: refs/heads/master@{#431376}

TBR=pavely@chromium.org,zea@chromium.org,pnoland@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660655
Committed: https://crrev.com/09f9c8e02d8918a23099fc8d7402aa39f24ea1fe
Cr-Commit-Position: refs/heads/master@{#431525}

[hiroshige, 2016-11-11 08:28:34]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2016-11-11 08:28:34]

Created Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky

[commit-bot: I haz the power, 2016-11-11 08:28:43]

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-11-11 08:29:35]

Description was changed from

==========
Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky (patchset #2
id:40001 of https://codereview.chromium.org/2479313004/ )

Reason for revert:
Suspected to cause TwoClientUssSyncTest.ConflictResolution flaky failures:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=Tw...


Original issue's description:
> [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky
> 
> Reduce flakiness in TwoClientBookmarksSyncTest by:
> 1) Reducing the nudge delay for sync to 1 ms
> 2) Switching to BookmarksMatchChecker.Wait(), which isn't prone to the
> "stale snapshot" race condition.
> 
> Reducing the nudge delay applies to and should speed up all the sync
> integration tests, not just the TwoClientBookmarks ones.
> 
> Only pairs of
> AwaitQuiescence|AwaitMutualSyncCycleCompletion
> AllModelsMatch(Verifier?)()
> 
> Were converted to use the new pattern.
> 
> BUG=660655
> 
> R=zea@chromium.org, pavely@chromium.org
> 
> Committed: https://crrev.com/3b8f13960ccb28bf854a91f05878b95529c32d79
> Cr-Commit-Position: refs/heads/master@{#431376}

TBR=pavely@chromium.org,zea@chromium.org,pnoland@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660655
==========

to

==========
Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky (patchset #2
id:40001 of https://codereview.chromium.org/2479313004/ )

Reason for revert:
Suspected to cause TwoClientUssSyncTest.ConflictResolution flaky failures:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=Tw...


Original issue's description:
> [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky
> 
> Reduce flakiness in TwoClientBookmarksSyncTest by:
> 1) Reducing the nudge delay for sync to 1 ms
> 2) Switching to BookmarksMatchChecker.Wait(), which isn't prone to the
> "stale snapshot" race condition.
> 
> Reducing the nudge delay applies to and should speed up all the sync
> integration tests, not just the TwoClientBookmarks ones.
> 
> Only pairs of
> AwaitQuiescence|AwaitMutualSyncCycleCompletion
> AllModelsMatch(Verifier?)()
> 
> Were converted to use the new pattern.
> 
> BUG=660655
> 
> R=zea@chromium.org, pavely@chromium.org
> 
> Committed: https://crrev.com/3b8f13960ccb28bf854a91f05878b95529c32d79
> Cr-Commit-Position: refs/heads/master@{#431376}

TBR=pavely@chromium.org,zea@chromium.org,pnoland@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660655
==========

[commit-bot: I haz the power, 2016-11-11 08:29:36]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky (patchset #2
id:40001 of https://codereview.chromium.org/2479313004/ )

Reason for revert:
Suspected to cause TwoClientUssSyncTest.ConflictResolution flaky failures:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=Tw...


Original issue's description:
> [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky
> 
> Reduce flakiness in TwoClientBookmarksSyncTest by:
> 1) Reducing the nudge delay for sync to 1 ms
> 2) Switching to BookmarksMatchChecker.Wait(), which isn't prone to the
> "stale snapshot" race condition.
> 
> Reducing the nudge delay applies to and should speed up all the sync
> integration tests, not just the TwoClientBookmarks ones.
> 
> Only pairs of
> AwaitQuiescence|AwaitMutualSyncCycleCompletion
> AllModelsMatch(Verifier?)()
> 
> Were converted to use the new pattern.
> 
> BUG=660655
> 
> R=zea@chromium.org, pavely@chromium.org
> 
> Committed: https://crrev.com/3b8f13960ccb28bf854a91f05878b95529c32d79
> Cr-Commit-Position: refs/heads/master@{#431376}

TBR=pavely@chromium.org,zea@chromium.org,pnoland@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660655
==========

to

==========
Revert of [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky (patchset #2
id:40001 of https://codereview.chromium.org/2479313004/ )

Reason for revert:
Suspected to cause TwoClientUssSyncTest.ConflictResolution flaky failures:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=Tw...

Original issue's description:
> [sync] "TwoClientBookmarksSyncTest.Sanity" is flaky
>
> Reduce flakiness in TwoClientBookmarksSyncTest by:
> 1) Reducing the nudge delay for sync to 1 ms
> 2) Switching to BookmarksMatchChecker.Wait(), which isn't prone to the
> "stale snapshot" race condition.
>
> Reducing the nudge delay applies to and should speed up all the sync
> integration tests, not just the TwoClientBookmarks ones.
>
> Only pairs of
> AwaitQuiescence|AwaitMutualSyncCycleCompletion
> AllModelsMatch(Verifier?)()
>
> Were converted to use the new pattern.
>
> BUG=660655
>
> R=zea@chromium.org, pavely@chromium.org
>
> Committed: https://crrev.com/3b8f13960ccb28bf854a91f05878b95529c32d79
> Cr-Commit-Position: refs/heads/master@{#431376}

TBR=pavely@chromium.org,zea@chromium.org,pnoland@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=660655

Committed: https://crrev.com/09f9c8e02d8918a23099fc8d7402aa39f24ea1fe
Cr-Commit-Position: refs/heads/master@{#431525}
==========

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

Patchset 1 (id:??) landed as
https://crrev.com/09f9c8e02d8918a23099fc8d7402aa39f24ea1fe
Cr-Commit-Position: refs/heads/master@{#431525}