Display GL window system binding information in about:gpu

Display GL window system binding information in about:gpu

This patch adds information about the GL window system binding layer
implementation to about:gpu. Depending on which binding layer is used,
the following information is displayed:

- EGL: vendor, version, extension string
- GLX: vendor, version, extension string
- WGL: extension string

TEST=Check that the above shows up in about:gpu

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203167

[Sami, 2013-05-23 15:41:31]

Hi Gregg, here's something I've wanted in about:gpu for a while. WDYT?

[greggman, 2013-05-23 16:22:14]

lgtm

https://codereview.chromium.org/15890002/diff/1/ui/gl/gl_implementation_x11.cc
File ui/gl/gl_implementation_x11.cc (right):

https://codereview.chromium.org/15890002/diff/1/ui/gl/gl_implementation_x11.c...
ui/gl/gl_implementation_x11.cc:253: EGLDisplay display = eglGetCurrentDisplay();
This piece of code is repeated 3 times. Does it make any sense to put it in one
place? Probably not but I just thought I'd ask.

[Sami, 2013-05-23 19:05:19]

Thanks Gregg. Adding joi@ for content/public/common/gpu_info.h and cdn@ for
gpu_messages.h. PTAL.

On 2013/05/23 16:22:14, greggman wrote:
> This piece of code is repeated 3 times. Does it make any sense to put it in
one
> place? Probably not but I just thought I'd ask.

Great point. Looks like it fits naturally into
gl_{egl,glx,wgl}_api_implementation.cc so I've moved it there.

[Jói, 2013-05-23 20:12:08]

LGTM for //content/public/common.

[apatrick_chromium, 2013-05-23 20:13:04]

FYI, some of the files in this patch just moved. See
https://codereview.chromium.org/15745014/.

[Sami, 2013-05-24 10:17:45]

On 2013/05/23 20:13:04, apatrick_chromium wrote:
> FYI, some of the files in this patch just moved. See
> https://codereview.chromium.org/15745014/.

Thanks for the warning. I've now rebased this.

[apatrick_chromium, 2013-05-24 18:05:22]

lgtm

[Sami, 2013-05-29 11:38:55]

cdn@, friendly ping about content/common/gpu/gpu_messages.h. Thanks!

[Cris Neckar, 2013-05-29 20:20:04]

On 2013/05/29 11:38:55, Sami wrote:
> cdn@, friendly ping about content/common/gpu/gpu_messages.h. Thanks!

IPC changes LGTM

[commit-bot: I haz the power, 2013-05-30 10:06:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skyostil@chromium.org/15890002/13001

[commit-bot: I haz the power, 2013-05-30 10:22:08]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_aura.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-05-30 10:37:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skyostil@chromium.org/15890002/40001

[commit-bot: I haz the power, 2013-05-30 12:37:07]

Retried try job too often on linux_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-05-30 12:44:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skyostil@chromium.org/15890002/40001

[commit-bot: I haz the power, 2013-05-30 14:53:11]

Retried try job too often on linux_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-05-30 14:58:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skyostil@chromium.org/15890002/40001

[commit-bot: I haz the power, 2013-05-30 17:06:46]

Change committed as 203167