Revert of command_buffer_gles2: Implement EGL default Display as a global object

Revert of command_buffer_gles2: Implement EGL default Display as a global object (patchset #21 id:400001 of https://codereview.chromium.org/1714883002/ )

Reason for revert:
Seems to be failing the gles2_conform_test of all flavours on all bots.

First fail:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NVIDIA%29/builds/24198
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28NVIDIA%20GeForce%20730%29/builds/840

Original issue's description:
> command_buffer_gles2: Implement EGL default Display as a global object
>
> Make EGL Display a global. eglGetDisplay(EGL_DEFAULT_DISPLAY) will
> return the same object.
>
> Store EGL state in a new thread-local object (ThreadState). One
> display, multiple surfaces, multiple contexts.
>
> Fix the currently implemented entry points to return specified errors
> and behave as specified. Notable changes:
> * eglGetDisplay() does not create new displays for each invocation
> * eglTerminate() invalidates the created egl objects but leaves the
>   current context intact as specified
> * eglMakeCurrent() is implemented more correctly with refcount semantics
> * eglQueryStrings() works without display for some names
> * eglReleaseThread() is implemeted
> * Config are now separate for pbuffers and windows, so that contexts
>   can be created separate from surfaces
> * Context can be made current with different surfaces
> * Catches errors related to context being set current to multiple
>   threads
> * Tries to be thread-safe
>
> In the implementation, the Display contains a lock ensuring all thread
> accesses are consistent. All functions using or changing thread-local
> state have explicit egl::ThreadState parameter.
>
> BUG=581634
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Committed: https://crrev.com/de9aa3be41bca094454babbd62907815fad7f5c2
> Cr-Commit-Position: refs/heads/master@{#377535}
>
> Committed: https://crrev.com/7b01a0867be0895009b120f37fcd4166a84f404b
> Cr-Commit-Position: refs/heads/master@{#386613}
>
> Committed: https://crrev.com/f9d5c7e6fd53570c0decc63e8ec51d30e0701c89
> Cr-Commit-Position: refs/heads/master@{#387871}

TBR=piman@chromium.org,thakis@chromium.org,vabr@chromium.org,kkinnunen@nvidia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=581634
Committed: https://crrev.com/50e5602ba08b92a495c9855f3118898591e33dce
Cr-Commit-Position: refs/heads/master@{#388784}

[Jamie Madill, 2016-04-21 16:21:18]

Created Revert of command_buffer_gles2: Implement EGL default Display as a
global object

[commit-bot: I haz the power, 2016-04-21 16:21:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1900563007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1900563007/1

[commit-bot: I haz the power, 2016-04-21 16:22:20]

Description was changed from

==========
Revert of command_buffer_gles2: Implement EGL default Display as a global object
(patchset #21 id:400001 of https://codereview.chromium.org/1714883002/ )

Reason for revert:
Seems to be failing the gles2_conform_test of all flavours on all bots.

First fail:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...


Original issue's description:
> command_buffer_gles2: Implement EGL default Display as a global object
> 
> Make EGL Display a global. eglGetDisplay(EGL_DEFAULT_DISPLAY) will
> return the same object.
> 
> Store EGL state in a new thread-local object (ThreadState). One
> display, multiple surfaces, multiple contexts.
> 
> Fix the currently implemented entry points to return specified errors
> and behave as specified. Notable changes:
> * eglGetDisplay() does not create new displays for each invocation
> * eglTerminate() invalidates the created egl objects but leaves the
>   current context intact as specified
> * eglMakeCurrent() is implemented more correctly with refcount semantics
> * eglQueryStrings() works without display for some names
> * eglReleaseThread() is implemeted
> * Config are now separate for pbuffers and windows, so that contexts
>   can be created separate from surfaces
> * Context can be made current with different surfaces
> * Catches errors related to context being set current to multiple
>   threads
> * Tries to be thread-safe
> 
> In the implementation, the Display contains a lock ensuring all thread
> accesses are consistent. All functions using or changing thread-local
> state have explicit egl::ThreadState parameter.
> 
> BUG=581634
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/de9aa3be41bca094454babbd62907815fad7f5c2
> Cr-Commit-Position: refs/heads/master@{#377535}
> 
> Committed: https://crrev.com/7b01a0867be0895009b120f37fcd4166a84f404b
> Cr-Commit-Position: refs/heads/master@{#386613}
> 
> Committed: https://crrev.com/f9d5c7e6fd53570c0decc63e8ec51d30e0701c89
> Cr-Commit-Position: refs/heads/master@{#387871}

TBR=piman@chromium.org,thakis@chromium.org,vabr@chromium.org,kkinnunen@nvidia...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=581634
==========

to

==========
Revert of command_buffer_gles2: Implement EGL default Display as a global object
(patchset #21 id:400001 of https://codereview.chromium.org/1714883002/ )

Reason for revert:
Seems to be failing the gles2_conform_test of all flavours on all bots.

First fail:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...


Original issue's description:
> command_buffer_gles2: Implement EGL default Display as a global object
> 
> Make EGL Display a global. eglGetDisplay(EGL_DEFAULT_DISPLAY) will
> return the same object.
> 
> Store EGL state in a new thread-local object (ThreadState). One
> display, multiple surfaces, multiple contexts.
> 
> Fix the currently implemented entry points to return specified errors
> and behave as specified. Notable changes:
> * eglGetDisplay() does not create new displays for each invocation
> * eglTerminate() invalidates the created egl objects but leaves the
>   current context intact as specified
> * eglMakeCurrent() is implemented more correctly with refcount semantics
> * eglQueryStrings() works without display for some names
> * eglReleaseThread() is implemeted
> * Config are now separate for pbuffers and windows, so that contexts
>   can be created separate from surfaces
> * Context can be made current with different surfaces
> * Catches errors related to context being set current to multiple
>   threads
> * Tries to be thread-safe
> 
> In the implementation, the Display contains a lock ensuring all thread
> accesses are consistent. All functions using or changing thread-local
> state have explicit egl::ThreadState parameter.
> 
> BUG=581634
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/de9aa3be41bca094454babbd62907815fad7f5c2
> Cr-Commit-Position: refs/heads/master@{#377535}
> 
> Committed: https://crrev.com/7b01a0867be0895009b120f37fcd4166a84f404b
> Cr-Commit-Position: refs/heads/master@{#386613}
> 
> Committed: https://crrev.com/f9d5c7e6fd53570c0decc63e8ec51d30e0701c89
> Cr-Commit-Position: refs/heads/master@{#387871}

TBR=piman@chromium.org,thakis@chromium.org,vabr@chromium.org,kkinnunen@nvidia...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=581634
==========

[commit-bot: I haz the power, 2016-04-21 16:22:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:35:07]

Description was changed from

==========
Revert of command_buffer_gles2: Implement EGL default Display as a global object
(patchset #21 id:400001 of https://codereview.chromium.org/1714883002/ )

Reason for revert:
Seems to be failing the gles2_conform_test of all flavours on all bots.

First fail:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...


Original issue's description:
> command_buffer_gles2: Implement EGL default Display as a global object
> 
> Make EGL Display a global. eglGetDisplay(EGL_DEFAULT_DISPLAY) will
> return the same object.
> 
> Store EGL state in a new thread-local object (ThreadState). One
> display, multiple surfaces, multiple contexts.
> 
> Fix the currently implemented entry points to return specified errors
> and behave as specified. Notable changes:
> * eglGetDisplay() does not create new displays for each invocation
> * eglTerminate() invalidates the created egl objects but leaves the
>   current context intact as specified
> * eglMakeCurrent() is implemented more correctly with refcount semantics
> * eglQueryStrings() works without display for some names
> * eglReleaseThread() is implemeted
> * Config are now separate for pbuffers and windows, so that contexts
>   can be created separate from surfaces
> * Context can be made current with different surfaces
> * Catches errors related to context being set current to multiple
>   threads
> * Tries to be thread-safe
> 
> In the implementation, the Display contains a lock ensuring all thread
> accesses are consistent. All functions using or changing thread-local
> state have explicit egl::ThreadState parameter.
> 
> BUG=581634
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/de9aa3be41bca094454babbd62907815fad7f5c2
> Cr-Commit-Position: refs/heads/master@{#377535}
> 
> Committed: https://crrev.com/7b01a0867be0895009b120f37fcd4166a84f404b
> Cr-Commit-Position: refs/heads/master@{#386613}
> 
> Committed: https://crrev.com/f9d5c7e6fd53570c0decc63e8ec51d30e0701c89
> Cr-Commit-Position: refs/heads/master@{#387871}

TBR=piman@chromium.org,thakis@chromium.org,vabr@chromium.org,kkinnunen@nvidia...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=581634
==========

to

==========
Revert of command_buffer_gles2: Implement EGL default Display as a global object
(patchset #21 id:400001 of https://codereview.chromium.org/1714883002/ )

Reason for revert:
Seems to be failing the gles2_conform_test of all flavours on all bots.

First fail:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...

Original issue's description:
> command_buffer_gles2: Implement EGL default Display as a global object
>
> Make EGL Display a global. eglGetDisplay(EGL_DEFAULT_DISPLAY) will
> return the same object.
>
> Store EGL state in a new thread-local object (ThreadState). One
> display, multiple surfaces, multiple contexts.
>
> Fix the currently implemented entry points to return specified errors
> and behave as specified. Notable changes:
> * eglGetDisplay() does not create new displays for each invocation
> * eglTerminate() invalidates the created egl objects but leaves the
>   current context intact as specified
> * eglMakeCurrent() is implemented more correctly with refcount semantics
> * eglQueryStrings() works without display for some names
> * eglReleaseThread() is implemeted
> * Config are now separate for pbuffers and windows, so that contexts
>   can be created separate from surfaces
> * Context can be made current with different surfaces
> * Catches errors related to context being set current to multiple
>   threads
> * Tries to be thread-safe
>
> In the implementation, the Display contains a lock ensuring all thread
> accesses are consistent. All functions using or changing thread-local
> state have explicit egl::ThreadState parameter.
>
> BUG=581634
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Committed: https://crrev.com/de9aa3be41bca094454babbd62907815fad7f5c2
> Cr-Commit-Position: refs/heads/master@{#377535}
>
> Committed: https://crrev.com/7b01a0867be0895009b120f37fcd4166a84f404b
> Cr-Commit-Position: refs/heads/master@{#386613}
>
> Committed: https://crrev.com/f9d5c7e6fd53570c0decc63e8ec51d30e0701c89
> Cr-Commit-Position: refs/heads/master@{#387871}

TBR=piman@chromium.org,thakis@chromium.org,vabr@chromium.org,kkinnunen@nvidia...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=581634

Committed: https://crrev.com/50e5602ba08b92a495c9855f3118898591e33dce
Cr-Commit-Position: refs/heads/master@{#388784}
==========

[commit-bot: I haz the power, 2016-04-22 19:35:08]

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