Convert tabcontentscontainer to use a vbox instead of a fixed. The

Convert tabcontentscontainer to use a vbox instead of a fixed.  The
comment in the code mentions that a vbox would cause a split screen,
but looking at the code, it seems like that's not possible because
we hide the old TCVG before adding new ones.  These days the vbox
contains all TCVGs and only shows one at a time (I think in the past
we tried to remove them, but we don't have that problem now).

This simplifies the code by not having to use our own size-allocate
handler.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=30843

[Evan Stade, 2009-11-03 02:26:11]

lg if it works

to refresh my memory, the TCVG still uses a fixed for its highest level widget,
right? In my recollection the problem with showing more than one tab contents
was in TCVG where it would show an interstitial and a normal page at the same
time.

[tony, 2009-11-03 18:49:01]

On 2009/11/03 02:26:11, Evan Stade wrote:
> lg if it works
> 
> to refresh my memory, the TCVG still uses a fixed for its highest level
widget,
> right? In my recollection the problem with showing more than one tab contents
> was in TCVG where it would show an interstitial and a normal page at the same
> time.

It seems that way based on the second to last review comment here:
http://codereview.chromium.org/113590

This patch seems to work fine with interstitial pages and with dragging tabs
around (the other case where we add/remove TCVGs).