save vertex attribute divisor values for each program

If there are two shader programs, one vertex attribute's divisor of the first program is non-zero, other vertex attribute's divisor of the second program is zero, when the two vetext attributes' location value is the same. the latter vertex attribute's divisor value is non-zero, and override by the former one. So save vertex attribute divisor values in program instead of vertex attribute manager.

BUG=625960
TEST=all/deqp/functional/gles3/instancedrendering.html
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

[xinghua.cao, 2016-07-06 08:56:28]

Description was changed from

==========
Revert vertex attribute divisor value when change program

BUG=
==========

to

==========
Revert vertex attribute divisor value when change program

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xinghua.cao, 2016-07-06 10:05:27]

Description was changed from

==========
Revert vertex attribute divisor value when change program

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Revert vertex attribute divisor value when change program

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xinghua.cao, 2016-07-06 10:05:27]

xinghua.cao@intel.com changed reviewers:
+ qiankun.miao@intel.com, yunchao.he@intel.com

[xinghua.cao, 2016-07-06 10:45:54]

Description was changed from

==========
Revert vertex attribute divisor value when change program

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Revert vertex attribute divisor value when change program

BUG=625960
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xinghua.cao, 2016-07-07 02:47:41]

Please help to review it, thank you.

[qiankun, 2016-07-07 06:41:31]

On 2016/07/07 02:47:41, xinghua.cao wrote:
> Please help to review it, thank you.

Can you point the context of this patch. Which test case does it fix?
Reset is more suitable than revert for the function name.

[xinghua.cao, 2016-07-07 06:54:52]

Description was changed from

==========
Revert vertex attribute divisor value when change program

BUG=625960
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Reset vertex attribute divisor value when change program
There are two shader programs, one vertex attribute's divisor of the first
program is non-zero, other vertex attribute's divisor of the second program is
zero, when the two vetext attributes' location value is the same. the latter
vertex attribute's divisor value is non-zero. It is effected by the former one,
so need to reset divisor value when switching program.

BUG=625960
TEST=all/deqp/functional/gles3/instancedrendering.html
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[yunchao, 2016-07-07 10:33:34]

On 2016/07/07 06:41:31, qiankun wrote:
> On 2016/07/07 02:47:41, xinghua.cao wrote:
> > Please help to review it, thank you.
> 
> Can you point the context of this patch. Which test case does it fix?
> Reset is more suitable than revert for the function name.

Xinghua, could you please run trybots and/or dryrun before you ask someone to
review your patch.

[xinghua.cao, 2016-07-07 10:40:29]

On 2016/07/07 10:33:34, yunchao wrote:
> On 2016/07/07 06:41:31, qiankun wrote:
> > On 2016/07/07 02:47:41, xinghua.cao wrote:
> > > Please help to review it, thank you.
> > 
> > Can you point the context of this patch. Which test case does it fix?
> > Reset is more suitable than revert for the function name.
> 
> Xinghua, could you please run trybots and/or dryrun before you ask someone to
> review your patch.

Ok. I Always thought that it must be reviewed before allowing to CQ dry run.

[xinghua.cao, 2016-07-07 10:49:20]

The CQ bit was checked by xinghua.cao@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-07 10:49:45]

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-07-07 11:22:07]

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

[commit-bot: I haz the power, 2016-07-07 11:22:08]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[xinghua.cao, 2016-07-11 06:44:13]

Description was changed from

==========
Reset vertex attribute divisor value when change program
There are two shader programs, one vertex attribute's divisor of the first
program is non-zero, other vertex attribute's divisor of the second program is
zero, when the two vetext attributes' location value is the same. the latter
vertex attribute's divisor value is non-zero. It is effected by the former one,
so need to reset divisor value when switching program.

BUG=625960
TEST=all/deqp/functional/gles3/instancedrendering.html
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
If there are two shader programs, one vertex attribute's divisor of the first
program is non-zero, other vertex attribute's divisor of the second program is
zero, when the two vetext attributes' location value is the same. the latter
vertex attribute's divisor value is non-zero, and override by the former one. So
save vertex attribute divisor values in program instead of vertex attribute
manager.

BUG=625960
TEST=all/deqp/functional/gles3/instancedrendering.html
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[xinghua.cao, 2016-07-12 07:58:17]

The CQ bit was checked by xinghua.cao@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 07:58:54]

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-07-12 08:49:09]

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

[commit-bot: I haz the power, 2016-07-12 08:49:10]

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

[qiankun, 2016-07-20 10:04:22]

Can we close this now since it was fixed in WebGL conformance test:
https://github.com/KhronosGroup/WebGL/pull/1906.