cc: Do not verify resource sync tokens unnecessarily.

cc: Do not verify resource sync tokens unnecessarily.

ResourceProvider::delegated_sync_points_required_ and
ResourceProvider::Child::needs_sync_tokens only toggle the creation of
sync tokens by ResourceProvider but ResourceProvider always tries to
verify sync tokens already associated with resources e.g. from the
worker thread. This CL ensures that we don't verify sync tokens if those
flags are off.

R=piman@chromium.org
BUG=626193
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/91978e02d9bd4c39d10de4a2a1698742d50b3ffb
Cr-Commit-Position: refs/heads/master@{#404435}

[sunnyps, 2016-07-07 01:19:45]

Description was changed from

==========
cc: Do not verify resource sync tokens unnecessarily.

ResourceProvider::delegated_sync_points_required_ and
ResourceProvider::Child::needs_sync_tokens only toggle the creation of
sync tokens by ResourceProvider but ResourceProvider always tries to
verify sync tokens already associated with resources e.g. from the
worker thread. This CL ensures that we don't verify sync tokens if those
flags are off.

R=piman@chromium.org
BUG=626193
==========

to

==========
cc: Do not verify resource sync tokens unnecessarily.

ResourceProvider::delegated_sync_points_required_ and
ResourceProvider::Child::needs_sync_tokens only toggle the creation of
sync tokens by ResourceProvider but ResourceProvider always tries to
verify sync tokens already associated with resources e.g. from the
worker thread. This CL ensures that we don't verify sync tokens if those
flags are off.

R=piman@chromium.org
BUG=626193
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[sunnyps, 2016-07-07 01:37:37]

PTAL

[sunnyps, 2016-07-07 22:38:03]

Setting the sync token (for resources which didn't have sync tokens) before
verification was causing crashes due to failed IPC checks. Fixed that in the
second patch. PTAL

[piman, 2016-07-07 23:00:14]

lgtm

[sunnyps, 2016-07-08 17:24:25]

The CQ bit was checked by sunnyps@chromium.org

[commit-bot: I haz the power, 2016-07-08 17:25:12]

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-07-08 18:04:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-08 18:04:23]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-08 18:06:58]

Description was changed from

==========
cc: Do not verify resource sync tokens unnecessarily.

ResourceProvider::delegated_sync_points_required_ and
ResourceProvider::Child::needs_sync_tokens only toggle the creation of
sync tokens by ResourceProvider but ResourceProvider always tries to
verify sync tokens already associated with resources e.g. from the
worker thread. This CL ensures that we don't verify sync tokens if those
flags are off.

R=piman@chromium.org
BUG=626193
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Do not verify resource sync tokens unnecessarily.

ResourceProvider::delegated_sync_points_required_ and
ResourceProvider::Child::needs_sync_tokens only toggle the creation of
sync tokens by ResourceProvider but ResourceProvider always tries to
verify sync tokens already associated with resources e.g. from the
worker thread. This CL ensures that we don't verify sync tokens if those
flags are off.

R=piman@chromium.org
BUG=626193
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/91978e02d9bd4c39d10de4a2a1698742d50b3ffb
Cr-Commit-Position: refs/heads/master@{#404435}
==========

[commit-bot: I haz the power, 2016-07-08 18:06:58]

Patchset 2 (id:??) landed as
https://crrev.com/91978e02d9bd4c39d10de4a2a1698742d50b3ffb
Cr-Commit-Position: refs/heads/master@{#404435}