Remove flakiness from one copy partial raster tests

Remove flakiness from one copy partial raster tests

The one copy path relied on GL command buffer progress in order for
staging resources to become available for partial raster. This wasn't
guaranteed. This change adds a flush before we attempt to partial
raster, ensuring that the staging resources are available.

R=danakj@chromium.org
BUG=676852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2649043010
Cr-Commit-Position: refs/heads/master@{#446205}
Committed: https://chromium.googlesource.com/chromium/src/+/afec1af82eb9103f39034c0909e32e9c0341f9aa

[ericrk, 2017-01-25 22:13:49]

Description was changed from

==========
Remove flakiness from one copy partial raster tests

The one copy path relied on GL command buffer progress in order for
staging resources to become available for partial raster. This wasn't
guaranteed. This change adds a flush before we attempt to partial
raster, ensuring that the staging resources are available.

R=danakj@chromium.org
BUG=676852
==========

to

==========
Remove flakiness from one copy partial raster tests

The one copy path relied on GL command buffer progress in order for
staging resources to become available for partial raster. This wasn't
guaranteed. This change adds a flush before we attempt to partial
raster, ensuring that the staging resources are available.

R=danakj@chromium.org
BUG=676852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ericrk, 2017-01-25 22:15:06]

Another general fix factored out of the on-hold ForceReclaimResources patch.

[ericrk, 2017-01-25 22:15:10]

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

[danakj, 2017-01-25 22:15:37]

LGTM

[commit-bot: I haz the power, 2017-01-25 22:15:55]

Dry run: 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-25 23:24:39]

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

[commit-bot: I haz the power, 2017-01-25 23:24:40]

Dry run: This issue passed the CQ dry run.

[ericrk, 2017-01-26 01:57:36]

The CQ bit was checked by ericrk@chromium.org

[commit-bot: I haz the power, 2017-01-26 01:58:16]

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-26 02:05:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485395856550630, "parent_rev":
"3c72dd4fcdb2ece2e571bf9b92a23949b9ea438b", "commit_rev":
"afec1af82eb9103f39034c0909e32e9c0341f9aa"}

[commit-bot: I haz the power, 2017-01-26 02:06:40]

Description was changed from

==========
Remove flakiness from one copy partial raster tests

The one copy path relied on GL command buffer progress in order for
staging resources to become available for partial raster. This wasn't
guaranteed. This change adds a flush before we attempt to partial
raster, ensuring that the staging resources are available.

R=danakj@chromium.org
BUG=676852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Remove flakiness from one copy partial raster tests

The one copy path relied on GL command buffer progress in order for
staging resources to become available for partial raster. This wasn't
guaranteed. This change adds a flush before we attempt to partial
raster, ensuring that the staging resources are available.

R=danakj@chromium.org
BUG=676852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

[commit-bot: I haz the power, 2017-01-26 02:06:41]

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