When rendering offscreen, prefer size 0 instead of 1.

When rendering offscreen, prefer size 0 instead of 1.

Fixes surface creation on ozone/gbm, where pbuffers are not supported.

R=jamesr@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/e5b4b95790bfbc69750a407dba421817644612a3

[cdotstout, 2015-09-11 21:33:49]

cstout@chromium.org changed reviewers:
+ jamesr@chromium.org

[jamesr, 2015-09-11 21:34:59]

lgtm

https://codereview.chromium.org/1343513002/diff/1/services/gles2/command_buff...
File services/gles2/command_buffer_driver.cc (right):

https://codereview.chromium.org/1343513002/diff/1/services/gles2/command_buff...
services/gles2/command_buffer_driver.cc:107: if (!surface_)
add { }s since the body of this conditional is multiline. maybe comment here
indicating why the code is retrying with 1x1

https://codereview.chromium.org/1343513002/diff/1/services/gles2/command_buff...
services/gles2/command_buffer_driver.cc:109: gfx::Size(1, 1),
requested_configuration_);
i think indentation is now wrong on this line

[cdotstout, 2015-09-11 21:52:54]

https://codereview.chromium.org/1343513002/diff/1/services/gles2/command_buff...
File services/gles2/command_buffer_driver.cc (right):

https://codereview.chromium.org/1343513002/diff/1/services/gles2/command_buff...
services/gles2/command_buffer_driver.cc:107: if (!surface_)
On 2015/09/11 21:34:59, jamesr wrote:
> add { }s since the body of this conditional is multiline. maybe comment here
> indicating why the code is retrying with 1x1

Done.

https://codereview.chromium.org/1343513002/diff/1/services/gles2/command_buff...
services/gles2/command_buffer_driver.cc:109: gfx::Size(1, 1),
requested_configuration_);
On 2015/09/11 21:34:59, jamesr wrote:
> i think indentation is now wrong on this line

Done.

[cdotstout, 2015-09-11 21:57:35]

Committed patchset #2 (id:20001) manually as
e5b4b95790bfbc69750a407dba421817644612a3 (presubmit successful).