[NaCl SDK] ppapi_simple: Allow initial terminal size to to be set on startup.

[NaCl SDK] ppapi_simple: Allow initial terminal size to to be set on startup.

Allow the initial terminal size to be set via the PS_TTY_ROWS
and PS_TTY_COLS environment variables.

This allows ppapi_simple application to know the TTY
size before the main function runs.

R=binji@chromium.org

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

[binji, 2013-11-09 00:55:07]

lgtm

https://codereview.chromium.org/66343004/diff/1/native_client_sdk/src/librari...
File native_client_sdk/src/libraries/ppapi_simple/ps_instance.cc (right):

https://codereview.chromium.org/66343004/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/ppapi_simple/ps_instance.cc:244:
HandleResize(atoi(tty_cols), atoi(tty_rows));
better error messaging here? What if only one is set? Or if the values are
non-integers?

[Sam Clegg, 2013-11-09 01:12:58]

https://codereview.chromium.org/66343004/diff/1/native_client_sdk/src/librari...
File native_client_sdk/src/libraries/ppapi_simple/ps_instance.cc (right):

https://codereview.chromium.org/66343004/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/ppapi_simple/ps_instance.cc:244:
HandleResize(atoi(tty_cols), atoi(tty_rows));
On 2013/11/09 00:55:08, binji wrote:
> better error messaging here? What if only one is set? Or if the values are
> non-integers?

Done.

[binji, 2013-11-09 01:33:51]

slgtm

[Sam Clegg, 2013-11-09 17:34:25]

Committed patchset #2 manually as r234130 (presubmit successful).