Revert of Move static GL binding initialization to //ui/gl/init.

Revert of Move static GL binding initialization to //ui/gl/init. (patchset #7 id:120001 of https://codereview.chromium.org/2094513002/ )

Reason for revert:
Suspected of breaking Official "win trunk" builder:
c:\b\build\slave\win_trunk\build\src\ui\gl\init\gl_initializer_win.cc(32): fatal error C1083: Cannot open include file: 'software_renderer.h': No such file or directory
Please see crbug.com/623657 for details.

Original issue's description:
> Move static GL binding initialization to //ui/gl/init.
>
> Static GL binding initialization needs to call out to the Ozone
> platform. Move this initialization code from //ui/gl to //ui/gl/init as
> part of larger effort to break //ui/gl dep on //ui/ozone. Also move
> InitializationDebugGLBindings() and ClearGLBindings() in a similar
> fashion as they are closely linked to static initialization.
>
> Unfortunately, dynamic GL binding initialization can't be moved
> //ui/gl/init. It would be nice to have all the initialization code in
> one target but dynamic GL binding initialization is used by GLContext.
>
> The existing InitializationStaticGLBindings() functions had grown to be
> very large for some platforms. Where appropriate the code for each
> implementation has been extracted into it's own method to improve
> readability.
>
> The PRESUBMIT.py script is modified slightly here. The existing static
> GL initialization uses ScopedAllowIO on some platforms. This function is
> banned so moving the code triggers presubmit errors. The GPU main thread
> can't continue until initialization is finished anyways so moving
> blocking code to a different thread isn't helpful. Add new exemptions to
> PRESUBMIT.py.
>
> BUG=611142
> 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
>
> Committed: https://crrev.com/507532e60d484704354edd17a7b51904e7a97d88
> Cr-Commit-Position: refs/heads/master@{#402259}

TBR=piman@chromium.org,dpranke@chromium.org,kylechar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=611142
Committed: https://crrev.com/f6fb94eabd50367cd9873ca6765fab04627f819b
Cr-Commit-Position: refs/heads/master@{#402287}

[Lambros, 2016-06-27 21:19:22]

The CQ bit was checked by lambroslambrou@chromium.org

[Lambros, 2016-06-27 21:19:23]

Created Revert of Move static GL binding initialization to //ui/gl/init.

[commit-bot: I haz the power, 2016-06-27 21:20:58]

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-06-27 21:22:10]

Description was changed from

==========
Revert of Move static GL binding initialization to //ui/gl/init. (patchset #7
id:120001 of https://codereview.chromium.org/2094513002/ )

Reason for revert:
Suspected of breaking Official "win trunk" builder:
c:\b\build\slave\win_trunk\build\src\ui\gl\init\gl_initializer_win.cc(32): fatal
error C1083: Cannot open include file: 'software_renderer.h': No such file or
directory
Please see crbug.com/623657 for details.


Original issue's description:
> Move static GL binding initialization to //ui/gl/init.
> 
> Static GL binding initialization needs to call out to the Ozone
> platform. Move this initialization code from //ui/gl to //ui/gl/init as
> part of larger effort to break //ui/gl dep on //ui/ozone. Also move
> InitializationDebugGLBindings() and ClearGLBindings() in a similar
> fashion as they are closely linked to static initialization.
> 
> Unfortunately, dynamic GL binding initialization can't be moved
> //ui/gl/init. It would be nice to have all the initialization code in
> one target but dynamic GL binding initialization is used by GLContext.
> 
> The existing InitializationStaticGLBindings() functions had grown to be
> very large for some platforms. Where appropriate the code for each
> implementation has been extracted into it's own method to improve
> readability.
> 
> The PRESUBMIT.py script is modified slightly here. The existing static
> GL initialization uses ScopedAllowIO on some platforms. This function is
> banned so moving the code triggers presubmit errors. The GPU main thread
> can't continue until initialization is finished anyways so moving
> blocking code to a different thread isn't helpful. Add new exemptions to
> PRESUBMIT.py.
> 
> BUG=611142
>
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
> 
> Committed: https://crrev.com/507532e60d484704354edd17a7b51904e7a97d88
> Cr-Commit-Position: refs/heads/master@{#402259}

TBR=piman@chromium.org,dpranke@chromium.org,kylechar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=611142
==========

to

==========
Revert of Move static GL binding initialization to //ui/gl/init. (patchset #7
id:120001 of https://codereview.chromium.org/2094513002/ )

Reason for revert:
Suspected of breaking Official "win trunk" builder:
c:\b\build\slave\win_trunk\build\src\ui\gl\init\gl_initializer_win.cc(32): fatal
error C1083: Cannot open include file: 'software_renderer.h': No such file or
directory
Please see crbug.com/623657 for details.


Original issue's description:
> Move static GL binding initialization to //ui/gl/init.
> 
> Static GL binding initialization needs to call out to the Ozone
> platform. Move this initialization code from //ui/gl to //ui/gl/init as
> part of larger effort to break //ui/gl dep on //ui/ozone. Also move
> InitializationDebugGLBindings() and ClearGLBindings() in a similar
> fashion as they are closely linked to static initialization.
> 
> Unfortunately, dynamic GL binding initialization can't be moved
> //ui/gl/init. It would be nice to have all the initialization code in
> one target but dynamic GL binding initialization is used by GLContext.
> 
> The existing InitializationStaticGLBindings() functions had grown to be
> very large for some platforms. Where appropriate the code for each
> implementation has been extracted into it's own method to improve
> readability.
> 
> The PRESUBMIT.py script is modified slightly here. The existing static
> GL initialization uses ScopedAllowIO on some platforms. This function is
> banned so moving the code triggers presubmit errors. The GPU main thread
> can't continue until initialization is finished anyways so moving
> blocking code to a different thread isn't helpful. Add new exemptions to
> PRESUBMIT.py.
> 
> BUG=611142
>
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
> 
> Committed: https://crrev.com/507532e60d484704354edd17a7b51904e7a97d88
> Cr-Commit-Position: refs/heads/master@{#402259}

TBR=piman@chromium.org,dpranke@chromium.org,kylechar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=611142
==========

[commit-bot: I haz the power, 2016-06-27 21:22:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-27 21:23:49]

Description was changed from

==========
Revert of Move static GL binding initialization to //ui/gl/init. (patchset #7
id:120001 of https://codereview.chromium.org/2094513002/ )

Reason for revert:
Suspected of breaking Official "win trunk" builder:
c:\b\build\slave\win_trunk\build\src\ui\gl\init\gl_initializer_win.cc(32): fatal
error C1083: Cannot open include file: 'software_renderer.h': No such file or
directory
Please see crbug.com/623657 for details.


Original issue's description:
> Move static GL binding initialization to //ui/gl/init.
> 
> Static GL binding initialization needs to call out to the Ozone
> platform. Move this initialization code from //ui/gl to //ui/gl/init as
> part of larger effort to break //ui/gl dep on //ui/ozone. Also move
> InitializationDebugGLBindings() and ClearGLBindings() in a similar
> fashion as they are closely linked to static initialization.
> 
> Unfortunately, dynamic GL binding initialization can't be moved
> //ui/gl/init. It would be nice to have all the initialization code in
> one target but dynamic GL binding initialization is used by GLContext.
> 
> The existing InitializationStaticGLBindings() functions had grown to be
> very large for some platforms. Where appropriate the code for each
> implementation has been extracted into it's own method to improve
> readability.
> 
> The PRESUBMIT.py script is modified slightly here. The existing static
> GL initialization uses ScopedAllowIO on some platforms. This function is
> banned so moving the code triggers presubmit errors. The GPU main thread
> can't continue until initialization is finished anyways so moving
> blocking code to a different thread isn't helpful. Add new exemptions to
> PRESUBMIT.py.
> 
> BUG=611142
>
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
> 
> Committed: https://crrev.com/507532e60d484704354edd17a7b51904e7a97d88
> Cr-Commit-Position: refs/heads/master@{#402259}

TBR=piman@chromium.org,dpranke@chromium.org,kylechar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=611142
==========

to

==========
Revert of Move static GL binding initialization to //ui/gl/init. (patchset #7
id:120001 of https://codereview.chromium.org/2094513002/ )

Reason for revert:
Suspected of breaking Official "win trunk" builder:
c:\b\build\slave\win_trunk\build\src\ui\gl\init\gl_initializer_win.cc(32): fatal
error C1083: Cannot open include file: 'software_renderer.h': No such file or
directory
Please see crbug.com/623657 for details.

Original issue's description:
> Move static GL binding initialization to //ui/gl/init.
>
> Static GL binding initialization needs to call out to the Ozone
> platform. Move this initialization code from //ui/gl to //ui/gl/init as
> part of larger effort to break //ui/gl dep on //ui/ozone. Also move
> InitializationDebugGLBindings() and ClearGLBindings() in a similar
> fashion as they are closely linked to static initialization.
>
> Unfortunately, dynamic GL binding initialization can't be moved
> //ui/gl/init. It would be nice to have all the initialization code in
> one target but dynamic GL binding initialization is used by GLContext.
>
> The existing InitializationStaticGLBindings() functions had grown to be
> very large for some platforms. Where appropriate the code for each
> implementation has been extracted into it's own method to improve
> readability.
>
> The PRESUBMIT.py script is modified slightly here. The existing static
> GL initialization uses ScopedAllowIO on some platforms. This function is
> banned so moving the code triggers presubmit errors. The GPU main thread
> can't continue until initialization is finished anyways so moving
> blocking code to a different thread isn't helpful. Add new exemptions to
> PRESUBMIT.py.
>
> BUG=611142
>
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
>
> Committed: https://crrev.com/507532e60d484704354edd17a7b51904e7a97d88
> Cr-Commit-Position: refs/heads/master@{#402259}

TBR=piman@chromium.org,dpranke@chromium.org,kylechar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=611142

Committed: https://crrev.com/f6fb94eabd50367cd9873ca6765fab04627f819b
Cr-Commit-Position: refs/heads/master@{#402287}
==========

[commit-bot: I haz the power, 2016-06-27 21:23:50]

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

[Dirk Pranke, 2016-06-27 23:10:54]

lgtm