gpu_perftests: Use zero size gl surface for offscreen rendering

gpu_perftests: Use zero size gl surface for offscreen rendering

In GLX, most of the drivers can cope with both PBuffer and Surfaceless modes
for offscreen rendering. In EGL though, Intel driver only works with
Surfaceless.

This CL changes all hardware platforms to use Surfaceless by forcing a zero
size GL surface initialization. Ozone-GBM wasn't previously working because of
that but now works fine.

BUG=423481
TEST=gpu_perftests on GBM

Committed: https://crrev.com/274eb3202c59d4842c191e924577e81f14044af5
Cr-Commit-Position: refs/heads/master@{#325514}

[vignatti (out of this project), 2015-04-13 20:17:26]

tiago.vignatti@intel.com changed reviewers:
+ reveman@chromium.org

[vignatti (out of this project), 2015-04-13 20:19:00]

I believe dshwang@ will want to take a look on this one for his zero-copy
related work (crbug.com/475633).

reveman@ PTAL.

[reveman, 2015-04-13 20:29:32]

reveman@chromium.org changed reviewers:
+ dcastagna@chromium.org

[reveman, 2015-04-13 20:29:32]

+dcastagna

[Daniele Castagna, 2015-04-14 15:38:38]

https://codereview.chromium.org/1086703007/diff/1/gpu/perftests/texture_uploa...
File gpu/perftests/texture_upload_perftest.cc (right):

https://codereview.chromium.org/1086703007/diff/1/gpu/perftests/texture_uploa...
gpu/perftests/texture_upload_perftest.cc:181: base::MessageLoopForUI main_loop;
Where in the backend is this message loop used? It'd be nice to avoid this.

[vignatti (out of this project), 2015-04-14 16:39:16]

https://codereview.chromium.org/1086703007/diff/1/gpu/perftests/texture_uploa...
File gpu/perftests/texture_upload_perftest.cc (right):

https://codereview.chromium.org/1086703007/diff/1/gpu/perftests/texture_uploa...
gpu/perftests/texture_upload_perftest.cc:181: base::MessageLoopForUI main_loop;
On 2015/04/14 15:38:38, Daniele Castagna wrote:
> Where in the backend is this message loop used? It'd be nice to avoid this.

A few Ozone implementations like the GBM requires early initialization of the
message loop because GL stuff, display management and etc (most of hardware
accesses) happen in the GPU process. So we need the message loop in place
already for the GPU process initial handshake behave correctly.

That said, a couple of weeks ago I've been trying to put an end to all of this,
but it went too far from what I've originally have intended and gave up. More
info here:

https://codereview.chromium.org/1043233003/

[Daniele Castagna, 2015-04-16 19:53:36]

On 2015/04/14 at 16:39:16, tiago.vignatti wrote:
>
https://codereview.chromium.org/1086703007/diff/1/gpu/perftests/texture_uploa...
> File gpu/perftests/texture_upload_perftest.cc (right):
> 
>
https://codereview.chromium.org/1086703007/diff/1/gpu/perftests/texture_uploa...
> gpu/perftests/texture_upload_perftest.cc:181: base::MessageLoopForUI
main_loop;
> On 2015/04/14 15:38:38, Daniele Castagna wrote:
> > Where in the backend is this message loop used? It'd be nice to avoid this.
> 
> A few Ozone implementations like the GBM requires early initialization of the
message loop because GL stuff, display management and etc (most of hardware
accesses) happen in the GPU process. So we need the message loop in place
already for the GPU process initial handshake behave correctly.
> 
> That said, a couple of weeks ago I've been trying to put an end to all of
this, but it went too far from what I've originally have intended and gave up.
More info here:
> 
> https://codereview.chromium.org/1043233003/

Thanks for trying. :)

LGTM.

[vignatti (out of this project), 2015-04-16 19:59:15]

The CQ bit was checked by tiago.vignatti@intel.com

[reveman, 2015-04-16 20:00:31]

lgtm

[commit-bot: I haz the power, 2015-04-16 20:00:40]

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

[commit-bot: I haz the power, 2015-04-16 21:10:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-04-16 21:12:39]

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