gl_gl_api_implemetation: DO API check instead of null fnptr checks

gl_gl_api_implementation: Check API type instead of null fnptr

Some EGL implementation expose deprecated Desktop GL function on GL ES
contexts. RealGLApi would call them and cause a GL_INVALID_OPERATION
because the function is deprecated in core profile.

TBR=jbauman@chromium.org
BUG=686856

[Corentin Wallez, 2017-01-31 18:57:31]

Description was changed from

==========
gl_gl_api_implemetation: DO API check instead of null fnptr checks

Some EGL implementation expose deprecated Desktop GL function on GL ES
contexts. RealGLApi would call them and cause a GL_INVALID_OPERATION
because the function is deprecated in core profile.

BUG=686856
==========

to

==========
gl_gl_api_implemetation: DO API check instead of null fnptr checks

Some EGL implementation expose deprecated Desktop GL function on GL ES
contexts. RealGLApi would call them and cause a GL_INVALID_OPERATION
because the function is deprecated in core profile.

TBR=jbauman@chromium.org
BUG=686856
==========

[Corentin Wallez, 2017-01-31 18:57:31]

cwallez@chromium.org changed reviewers:
+ geofflang@chromium.org, jbauman@chromium.org

[Geoff Lang, 2017-01-31 18:59:57]

lgtm

[Geoff Lang, 2017-01-31 19:00:00]

The CQ bit was checked by geofflang@chromium.org

[commit-bot: I haz the power, 2017-01-31 19:00:39]

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

[Corentin Wallez, 2017-01-31 19:05:07]

Description was changed from

==========
gl_gl_api_implemetation: DO API check instead of null fnptr checks

Some EGL implementation expose deprecated Desktop GL function on GL ES
contexts. RealGLApi would call them and cause a GL_INVALID_OPERATION
because the function is deprecated in core profile.

TBR=jbauman@chromium.org
BUG=686856
==========

to

==========
gl_gl_api_implementation: Check API type instead of null fnptr

Some EGL implementation expose deprecated Desktop GL function on GL ES
contexts. RealGLApi would call them and cause a GL_INVALID_OPERATION
because the function is deprecated in core profile.

TBR=jbauman@chromium.org
BUG=686856
==========

[commit-bot: I haz the power, 2017-01-31 19:26:36]

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

[commit-bot: I haz the power, 2017-01-31 19:26:37]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Corentin Wallez, 2017-01-31 19:27:47]

Seems like a shard just didn't get started.

[Corentin Wallez, 2017-01-31 19:27:48]

The CQ bit was checked by cwallez@chromium.org

[commit-bot: I haz the power, 2017-01-31 19:28:41]

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-01-31 19:38:10]

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

[commit-bot: I haz the power, 2017-01-31 19:38:11]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)