Reland "gpu, cmaa: reduce one-copy", "gpu, cmaa: copy RGBA8 texture via glCopyTexSubImage2D() inste…

Reland "gpu, cmaa: reduce one-copy", "gpu, cmaa: copy RGBA8 texture via glCopyTexSubImage2D() instead of imageStore()."

Original CL:
https://codereview.chromium.org/2298613010 : use glCopyTexSubImage2D, and fix Pri1 bug
https://codereview.chromium.org/2405893002 : optimize further

Revert CL:
https://codereview.chromium.org/2456913002 : glCopyTexSubImage2D regresses performance and optimization CL is reverted due to conflict.

Changes:
- fix Pri1 bug as-is
- optimize further as-is
- don't introduce glCopyTexSubImage2D, and use the existing copy shader.

BUG=535198, 642290, 659438
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

Committed: https://crrev.com/6cd0a67f8256ecb9e0fb899528bdffc74f0e7042
Cr-Commit-Position: refs/heads/master@{#428826}

[dshwang, 2016-10-31 17:26:12]

Description was changed from

==========
Reland "gpu, cmaa: reduce one-copy", "gpu, cmaa: copy RGBA8 texture via
glCopyTexSubImage2D() instead of imageStore()."

Original CL:
https://codereview.chromium.org/2298613010 : use glCopyTexSubImage2D, and fix
Pri1 bug
https://codereview.chromium.org/2405893002 : optimize further

Revert CL:
https://codereview.chromium.org/2456913002 : glCopyTexSubImage2D regresses
performance and optimization CL is reverted due to conflict.

Changes:
- fix Pri1 bug as-is
- optimize further as-is
- don't introduce glCopyTexSubImage2D, and use the existing copy shader.

BUG=535198, 642290, 659438
==========

to

==========
Reland "gpu, cmaa: reduce one-copy", "gpu, cmaa: copy RGBA8 texture via
glCopyTexSubImage2D() instead of imageStore()."

Original CL:
https://codereview.chromium.org/2298613010 : use glCopyTexSubImage2D, and fix
Pri1 bug
https://codereview.chromium.org/2405893002 : optimize further

Revert CL:
https://codereview.chromium.org/2456913002 : glCopyTexSubImage2D regresses
performance and optimization CL is reverted due to conflict.

Changes:
- fix Pri1 bug as-is
- optimize further as-is
- don't introduce glCopyTexSubImage2D, and use the existing copy shader.

BUG=535198, 642290, 659438
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
==========

[inactive_dshwang_plz_cc_intel, 2016-10-31 17:26:19]

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

[commit-bot: I haz the power, 2016-10-31 17:26:32]

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

[inactive_dshwang_plz_cc_intel, 2016-10-31 17:27:14]

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

[commit-bot: I haz the power, 2016-10-31 17:27:43]

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

[dshwang, 2016-10-31 17:31:34]

dongseong.hwang@intel.com changed reviewers:
+ piman@chromium.org

[dshwang, 2016-10-31 17:31:34]

piman, could you review to reland reverted Pri1 bug fix. crbug.com/642290

https://codereview.chromium.org/2456913002 reverts two CLs together because
webgl perf regression because glCopyTexSubImage2D slowness.

This CL is reland 2 CLs + https://codereview.chromium.org/2460973002/

I'll reuse CopyTextureCHROMIUMResourceManager in CMAA in
https://codereview.chromium.org/2460973002/ to make reland CL succinct.

[commit-bot: I haz the power, 2016-10-31 18:36:06]

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

[commit-bot: I haz the power, 2016-10-31 18:36:06]

Dry run: This issue passed the CQ dry run.

[piman, 2016-10-31 21:13:13]

lgtm

[dshwang, 2016-10-31 21:18:14]

The CQ bit was checked by dongseong.hwang@intel.com

[commit-bot: I haz the power, 2016-10-31 21:18:55]

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-31 21:38:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-31 21:40:58]

Description was changed from

==========
Reland "gpu, cmaa: reduce one-copy", "gpu, cmaa: copy RGBA8 texture via
glCopyTexSubImage2D() instead of imageStore()."

Original CL:
https://codereview.chromium.org/2298613010 : use glCopyTexSubImage2D, and fix
Pri1 bug
https://codereview.chromium.org/2405893002 : optimize further

Revert CL:
https://codereview.chromium.org/2456913002 : glCopyTexSubImage2D regresses
performance and optimization CL is reverted due to conflict.

Changes:
- fix Pri1 bug as-is
- optimize further as-is
- don't introduce glCopyTexSubImage2D, and use the existing copy shader.

BUG=535198, 642290, 659438
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
==========

to

==========
Reland "gpu, cmaa: reduce one-copy", "gpu, cmaa: copy RGBA8 texture via
glCopyTexSubImage2D() instead of imageStore()."

Original CL:
https://codereview.chromium.org/2298613010 : use glCopyTexSubImage2D, and fix
Pri1 bug
https://codereview.chromium.org/2405893002 : optimize further

Revert CL:
https://codereview.chromium.org/2456913002 : glCopyTexSubImage2D regresses
performance and optimization CL is reverted due to conflict.

Changes:
- fix Pri1 bug as-is
- optimize further as-is
- don't introduce glCopyTexSubImage2D, and use the existing copy shader.

BUG=535198, 642290, 659438
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

Committed: https://crrev.com/6cd0a67f8256ecb9e0fb899528bdffc74f0e7042
Cr-Commit-Position: refs/heads/master@{#428826}
==========

[commit-bot: I haz the power, 2016-10-31 21:40:59]

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