Wait for GPU to finish before committing first video overlay frame.

Wait for GPU to finish before committing first video overlay frame.

The current code to do CopyResource helps in some cases, but with 4k
videos it still may flicker black before the first frame. Waiting for
the GPU commands to execute before the commit seems to help.

BUG=654631
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2892123002
Cr-Commit-Position: refs/heads/master@{#473373}
Committed: https://chromium.googlesource.com/chromium/src/+/7b6d8a3148e1279efa7fac40870557a5ca77d7dc

[jbauman, 2017-05-19 03:55:17]

Description was changed from

==========
Wait for GPU to finish before committing first video overlay frame.

The current code to do CopyResource helps in some cases, but with 4k
videos it still may flicker black before the first frame. Waiting for
the GPU commands to execute before the commit seems to help.

BUG=654631
==========

to

==========
Wait for GPU to finish before committing first video overlay frame.

The current code to do CopyResource helps in some cases, but with 4k
videos it still may flicker black before the first frame. Waiting for
the GPU commands to execute before the commit seems to help.

BUG=654631
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[jbauman, 2017-05-19 03:55:22]

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

[commit-bot: I haz the power, 2017-05-19 03:56:10]

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-05-19 05:00:43]

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

[commit-bot: I haz the power, 2017-05-19 05:00:44]

Dry run: This issue passed the CQ dry run.

[jbauman, 2017-05-19 22:28:53]

jbauman@chromium.org changed reviewers:
+ sunnyps@chromium.org

[sunnyps, 2017-05-19 22:40:05]

lgtm

[jbauman, 2017-05-19 22:44:54]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2017-05-19 22:45:41]

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-05-19 22:51:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495233894308320, "parent_rev":
"6ae7018ea85eabd5c341126227cb3403a2c1e5b6", "commit_rev":
"7b6d8a3148e1279efa7fac40870557a5ca77d7dc"}

[commit-bot: I haz the power, 2017-05-19 22:51:29]

Description was changed from

==========
Wait for GPU to finish before committing first video overlay frame.

The current code to do CopyResource helps in some cases, but with 4k
videos it still may flicker black before the first frame. Waiting for
the GPU commands to execute before the commit seems to help.

BUG=654631
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Wait for GPU to finish before committing first video overlay frame.

The current code to do CopyResource helps in some cases, but with 4k
videos it still may flicker black before the first frame. Waiting for
the GPU commands to execute before the commit seems to help.

BUG=654631
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2892123002
Cr-Commit-Position: refs/heads/master@{#473373}
Committed:
https://chromium.googlesource.com/chromium/src/+/7b6d8a3148e1279efa7fac408705...
==========

[commit-bot: I haz the power, 2017-05-19 22:51:30]

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