[hterm:crosh] Add support for terminal window resizing to terminal API

[HTerm-Crosh] Add support for terminal window resizing to terminal API

Add method terminalPrivate.onTerminalResize that will be called by
hterm when its window gets resized.
This is needed by some app we might run in pseudo-terminal that
need to know window size to work properly.

BUG=chromium-os:25947
TEST=Open some file in crosh using vi, and verify that the file is shown
     in full terminal screen.

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

[tbarzic, 2012-01-27 04:14:27]

Hey guys, can you take a look?

https://chromiumcodereview.appspot.com/9121035/diff/2001/chrome/browser/chrom...
File chrome/browser/chromeos/process_proxy/process_proxy.cc (right):

https://chromiumcodereview.appspot.com/9121035/diff/2001/chrome/browser/chrom...
chrome/browser/chromeos/process_proxy/process_proxy.cc:160: if (width < 0 ||
height < 0)
we might check upper bound, too (but I'm not sure what it should be).
Robert, what do you think?

[tbarzic, 2012-02-03 19:44:02]

ping?

[rginda, 2012-02-06 22:49:43]

lgtm

http://codereview.chromium.org/9121035/diff/2001/chrome/browser/chromeos/proc...
File chrome/browser/chromeos/process_proxy/process_proxy.cc (right):

http://codereview.chromium.org/9121035/diff/2001/chrome/browser/chromeos/proc...
chrome/browser/chromeos/process_proxy/process_proxy.cc:160: if (width < 0 ||
height < 0)
On 2012/01/27 04:14:27, tbarzic wrote:
> we might check upper bound, too (but I'm not sure what it should be).
> Robert, what do you think?

I'm not sure it makes sense to set an upper bound.  A tiny font on a large
display may not be very useful as a traditional terminal, but it seems excessive
to impose an arbitrary limit, especially in native code.

If the UI can't handle something above a certain size, it should impose the
limit itself.

[Aaron Boodman, 2012-02-13 22:46:53]

lgtm