Do additional blit and Present on first draw to overlay.

Do additional blit and Present on first draw to overlay.

It's unclear why this is necessary, but it seems to fix
Pixel_DirectComposition_Video_MP4.
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/2813103004
Cr-Commit-Position: refs/heads/master@{#464601}
Committed: https://chromium.googlesource.com/chromium/src/+/7c2bb41a8275988ed20cbf8f5dbaba08a0f9f222

[jbauman, 2017-04-13 02:58:31]

Description was changed from

==========
Do additional blit and Present on first draw to overlay.

It's unclear why this is necessary, but it seems to fix
Pixel_DirectComposition_Video_MP4.
==========

to

==========
Do additional blit and Present on first draw to overlay.

It's unclear why this is necessary, but it seems to fix
Pixel_DirectComposition_Video_MP4.
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-04-13 02:58:36]

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

[commit-bot: I haz the power, 2017-04-13 02:59:14]

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-04-13 03:41:00]

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

[commit-bot: I haz the power, 2017-04-13 03:41:01]

Dry run: This issue passed the CQ dry run.

[jbauman, 2017-04-13 20:54:47]

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

[commit-bot: I haz the power, 2017-04-13 20:55:28]

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

[jbauman, 2017-04-13 20:55:32]

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

[sunnyps, 2017-04-13 21:03:13]

lgtm % nits

https://codereview.chromium.org/2813103004/diff/20001/gpu/ipc/service/direct_...
File gpu/ipc/service/direct_composition_surface_win.cc (right):

https://codereview.chromium.org/2813103004/diff/20001/gpu/ipc/service/direct_...
gpu/ipc/service/direct_composition_surface_win.cc:421: // lost somehow and it
shows the wrong buffer.In that case copy the
nit: space after period

https://codereview.chromium.org/2813103004/diff/20001/gpu/ipc/service/direct_...
gpu/ipc/service/direct_composition_surface_win.cc:426: swap_chain_->GetBuffer(0,
IID_PPV_ARGS(dest_texture.Receive()));
nit: CHECK if these calls succeeded?

[commit-bot: I haz the power, 2017-04-13 21:49:31]

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

[commit-bot: I haz the power, 2017-04-13 21:49:32]

Dry run: This issue passed the CQ dry run.

[jbauman, 2017-04-13 22:08:31]

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

[jbauman, 2017-04-13 22:08:50]

The CQ bit was unchecked by jbauman@chromium.org

[jbauman, 2017-04-13 22:09:15]

The CQ bit was checked by jbauman@chromium.org

[jbauman, 2017-04-13 22:09:16]

The patchset sent to the CQ was uploaded after l-g-t-m from sunnyps@chromium.org
Link to the patchset: https://codereview.chromium.org/2813103004/#ps40001
(title: "improve")

[commit-bot: I haz the power, 2017-04-13 22:09:53]

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-04-13 23:18:55]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1492121355644180,
"parent_rev": "945a5edc332c030923556edde964cce5a4e24795", "commit_rev":
"7c2bb41a8275988ed20cbf8f5dbaba08a0f9f222"}

[commit-bot: I haz the power, 2017-04-13 23:19:47]

Description was changed from

==========
Do additional blit and Present on first draw to overlay.

It's unclear why this is necessary, but it seems to fix
Pixel_DirectComposition_Video_MP4.
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

==========
Do additional blit and Present on first draw to overlay.

It's unclear why this is necessary, but it seems to fix
Pixel_DirectComposition_Video_MP4.
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/2813103004
Cr-Commit-Position: refs/heads/master@{#464601}
Committed:
https://chromium.googlesource.com/chromium/src/+/7c2bb41a8275988ed20cbf8f5dba...
==========

[commit-bot: I haz the power, 2017-04-13 23:19:49]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/7c2bb41a8275988ed20cbf8f5dba...