[Sync] Updated CookieJarMismatch to handle multiple sync cycles during OpenTabAndGetLocalWindows.

[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
Committed: https://crrev.com/e1f491522b22424a488d6255ec1f066b0af45f8c
Cr-Commit-Position: refs/heads/master@{#434024}

[skym, 2016-11-22 19:20:24]

Description was changed from

==========
[Sync] Updatated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

BUG=663547
==========

to

==========
[Sync] Updatated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

[skym, 2016-11-22 19:20:24]

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

[skym, 2016-11-22 19:20:40]

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

[skym, 2016-11-22 19:21:07]

PTAL

[commit-bot: I haz the power, 2016-11-22 19:21:18]

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

[skym, 2016-11-22 19:38:41]

Description was changed from

==========
[Sync] Updatated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

to

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

[skym, 2016-11-22 19:41:53]

Description was changed from

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

to

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

[commit-bot: I haz the power, 2016-11-22 20:21:42]

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

[commit-bot: I haz the power, 2016-11-22 20:21:43]

Dry run: This issue passed the CQ dry run.

[maxbogue, 2016-11-22 21:55:52]

lgtm

[skym, 2016-11-22 23:13:41]

The CQ bit was checked by skym@chromium.org

[commit-bot: I haz the power, 2016-11-22 23:14:32]

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-22 23:47:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479856421708080, "parent_rev":
"40c7843484d5a2cc75e6872afead04e2070e69f5", "commit_rev":
"75137a59dbb3f44829a86e4c74d2620fc684c06c"}

[commit-bot: I haz the power, 2016-11-22 23:47:47]

Description was changed from

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

to

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

[commit-bot: I haz the power, 2016-11-22 23:47:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-22 23:50:15]

Description was changed from

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547
==========

to

==========
[Sync] Updated CookieJarMismatch to handle multiple sync cycles during
OpenTabAndGetLocalWindows.

CookieJarMismatch occasionally failed because two sync cycles would happen for
the upper or lower logic block, and then the histogram checks would fail. It is
a little bit unclear if the two sync cycles were a result of the implementation
of OpenTabAndGetLocalWindows(), or the extra TriggerSyncForModelTypes()
afterwards. Regardless, we now check the histograms in a manner that is
resilient to multiple sync cycles that duplicate the histogram emissions.

BUG=663547

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

[commit-bot: I haz the power, 2016-11-22 23:50:16]

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