Fix content upside down when call OffscreenCanvas's commit

Fix content upside down when call OffscreenCanvas's commit

The reason that the content is upside down is that we have a flag called
yflipped and it is always false. In the case of GPU compositing, this flag
should be true.

This CL also updates pixel tests for WebGL's commit, such that instead
of drawing a uniform color square, we draw a white triangle on a green
background.

BUG=655333, 655311
CQ_INCLUDE_TRYBOTS=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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Committed: https://crrev.com/8ff3caa7df9850d82ade934e1415b6ff8cb43a6b
Cr-Commit-Position: refs/heads/master@{#425571}

[xidachen, 2016-10-13 02:10:48]

Description was changed from

==========
Fix content upside down when call OffscreenCanvas's commit

The reason that the content is upside down is that we have a flag called
yflipped and it is always false. In the case of GPU compositing, this flag
should be true.

This CL also updates pixel tests for WebGL's commit, such that instead
of drawing a uniform color square, we draw a white triangle on a green
background.

BUG=655333, 655311
==========

to

==========
Fix content upside down when call OffscreenCanvas's commit

The reason that the content is upside down is that we have a flag called
yflipped and it is always false. In the case of GPU compositing, this flag
should be true.

This CL also updates pixel tests for WebGL's commit, such that instead
of drawing a uniform color square, we draw a white triangle on a green
background.

BUG=655333, 655311
CQ_INCLUDE_TRYBOTS=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;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

[xidachen, 2016-10-13 15:04:30]

xidachen@chromium.org changed reviewers:
+ junov@chromium.org, kbr@chromium.org, xlai@chromium.org, zmo@chromium.org

[xidachen, 2016-10-13 15:04:31]

PTAL

Locally I have tested this patch on Linux, Win, Mac and android and they all
work fine.

[xlai (Olivia), 2016-10-13 15:43:45]

On 2016/10/13 15:04:31, xidachen wrote:
> PTAL
> 
> Locally I have tested this patch on Linux, Win, Mac and android and they all
> work fine.

lgtm

[Justin Novosad, 2016-10-13 21:52:43]

lgtm

[Zhenyao Mo, 2016-10-15 03:33:10]

On 2016/10/13 21:52:43, Justin Novosad wrote:
> lgtm

lgtm

[xidachen, 2016-10-15 17:23:39]

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

[commit-bot: I haz the power, 2016-10-15 17:23:50]

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, 2016-10-15 17:25:58]

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

[commit-bot: I haz the power, 2016-10-15 17:25:59]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[xidachen, 2016-10-15 17:29:19]

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

[commit-bot: I haz the power, 2016-10-15 17:29:28]

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, 2016-10-15 18:52:14]

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

[commit-bot: I haz the power, 2016-10-15 18:52:15]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-10-15 19:10:49]

The CQ bit was checked by xidachen@chromium.org

[xidachen, 2016-10-15 19:10:49]

The patchset sent to the CQ was uploaded after l-g-t-m from zmo@chromium.org,
xlai@chromium.org, junov@chromium.org
Link to the patchset: https://codereview.chromium.org/2417773003/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-10-15 19:10:57]

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-10-15 19:17:08]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-15 19:20:21]

Description was changed from

==========
Fix content upside down when call OffscreenCanvas's commit

The reason that the content is upside down is that we have a flag called
yflipped and it is always false. In the case of GPU compositing, this flag
should be true.

This CL also updates pixel tests for WebGL's commit, such that instead
of drawing a uniform color square, we draw a white triangle on a green
background.

BUG=655333, 655311
CQ_INCLUDE_TRYBOTS=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;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

to

==========
Fix content upside down when call OffscreenCanvas's commit

The reason that the content is upside down is that we have a flag called
yflipped and it is always false. In the case of GPU compositing, this flag
should be true.

This CL also updates pixel tests for WebGL's commit, such that instead
of drawing a uniform color square, we draw a white triangle on a green
background.

BUG=655333, 655311
CQ_INCLUDE_TRYBOTS=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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Committed: https://crrev.com/8ff3caa7df9850d82ade934e1415b6ff8cb43a6b
Cr-Commit-Position: refs/heads/master@{#425571}
==========

[commit-bot: I haz the power, 2016-10-15 19:20:22]

Patchset 4 (id:??) landed as
https://crrev.com/8ff3caa7df9850d82ade934e1415b6ff8cb43a6b
Cr-Commit-Position: refs/heads/master@{#425571}