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 #8 id:140001 of https://codereview.chromium.org/1714883002/ )

Reason for revert:
Broke gles2_conform on mac/linux GPU bots on FYI waterfall

GTFNativeCreateWindow failed

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}

TBR=piman@chromium.org,thakis@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/0df7774c8d57adb794693f98733c610715c0879c
Cr-Commit-Position: refs/heads/master@{#377641}

[Zhenyao Mo, 2016-02-25 19:14:47]

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

[commit-bot: I haz the power, 2016-02-25 19:17:02]

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

[commit-bot: I haz the power, 2016-02-25 19:24:47]

Description was changed from

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

Reason for revert:
Broke gles2_conform on mac/linux GPU bots on FYI waterfall

GTFNativeCreateWindow failed

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}

TBR=piman@chromium.org,thakis@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
==========

to

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

Reason for revert:
Broke gles2_conform on mac/linux GPU bots on FYI waterfall

GTFNativeCreateWindow failed

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}

TBR=piman@chromium.org,thakis@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
==========

[commit-bot: I haz the power, 2016-02-25 19:24:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-25 19:26:57]

Description was changed from

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

Reason for revert:
Broke gles2_conform on mac/linux GPU bots on FYI waterfall

GTFNativeCreateWindow failed

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}

TBR=piman@chromium.org,thakis@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
==========

to

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

Reason for revert:
Broke gles2_conform on mac/linux GPU bots on FYI waterfall

GTFNativeCreateWindow failed

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}

TBR=piman@chromium.org,thakis@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/0df7774c8d57adb794693f98733c610715c0879c
Cr-Commit-Position: refs/heads/master@{#377641}
==========

[commit-bot: I haz the power, 2016-02-25 19:26:58]

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