ES3 sampler types and their corresponding textures should be initialized before being sampled.

ES3 sampler types and their corresponding textures should be initialized before being sampled.

BUG=666145
TEST=webgl2_conformance (to be added)
R=kbr@chromium.org,piman@chromium.org
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/5e6600b56cad269e4a8640c330847a18f2b7df12
Cr-Commit-Position: refs/heads/master@{#432740}

[Zhenyao Mo, 2016-11-17 01:18:47]

Description was changed from

==========
ES3 sampler types and their corresponding textures should be initialized before
being sampled.

BUG=666145
TEST=webgl2_conformance (to be added)
R=kbr@chromium.org,piman@chromium.org
==========

to

==========
ES3 sampler types and their corresponding textures should be initialized before
being sampled.

BUG=666145
TEST=webgl2_conformance (to be added)
R=kbr@chromium.org,piman@chromium.org
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
==========

[Zhenyao Mo, 2016-11-17 01:18:56]

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

[commit-bot: I haz the power, 2016-11-17 01:20:28]

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

[Ken Russell (switch to Gerrit), 2016-11-17 01:21:46]

LGTM

I know this is covered by your new WebGL 2.0 conformance test and that we're
under tight time constraints but could you plan to add some sort of GL test for
this after the M56 branch point?

[Zhenyao Mo, 2016-11-17 01:22:54]

On 2016/11/17 01:21:46, Ken Russell wrote:
> LGTM
> 
> I know this is covered by your new WebGL 2.0 conformance test and that we're
> under tight time constraints but could you plan to add some sort of GL test
for
> this after the M56 branch point?

I will add proper test (will keep the bug open to track this)

[commit-bot: I haz the power, 2016-11-17 02:34:50]

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

[commit-bot: I haz the power, 2016-11-17 02:34:51]

Dry run: Try jobs failed on following builders:
  mac_optional_gpu_tests_rel on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_optional_gpu_...)

[Zhenyao Mo, 2016-11-17 02:40:21]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2016-11-17 02:41:32]

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-11-17 03:22:49]

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

[commit-bot: I haz the power, 2016-11-17 03:22:50]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_compile_dbg on master.tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)
  android_cronet on master.tryserver.chromium.android (JOB_TIMED_OUT, no build
URL)
  cast_shell_android on master.tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)

[Zhenyao Mo, 2016-11-17 03:23:34]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2016-11-17 03:24:49]

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-11-17 04:28:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 04:39:51]

Description was changed from

==========
ES3 sampler types and their corresponding textures should be initialized before
being sampled.

BUG=666145
TEST=webgl2_conformance (to be added)
R=kbr@chromium.org,piman@chromium.org
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

==========
ES3 sampler types and their corresponding textures should be initialized before
being sampled.

BUG=666145
TEST=webgl2_conformance (to be added)
R=kbr@chromium.org,piman@chromium.org
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/5e6600b56cad269e4a8640c330847a18f2b7df12
Cr-Commit-Position: refs/heads/master@{#432740}
==========

[commit-bot: I haz the power, 2016-11-17 04:39:52]

Patchset 1 (id:??) landed as
https://crrev.com/5e6600b56cad269e4a8640c330847a18f2b7df12
Cr-Commit-Position: refs/heads/master@{#432740}