Add layout support for invisible SingleSplitView children.

Add layout support for invisible SingleSplitView children.


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

[pfeldman, 2009-06-30 18:33:30]

This CL makes a child view occupy all the space available if the other child
view is not visible.

[sky, 2009-06-30 18:52:42]

http://codereview.chromium.org/150120/diff/1/2
File views/controls/single_split_view.cc (right):

http://codereview.chromium.org/150120/diff/1/2#newcode149
Line 149: if (GetChildViewCount() < 2)
You need to update this method too. Otherwise if none of the child views want
the mouse event, it's possible to get weird behavior depending upon the last
size of the invisible views.

[pfeldman, 2009-07-01 12:38:33]

On 2009/06/30 18:52:42, sky wrote:
> http://codereview.chromium.org/150120/diff/1/2
> File views/controls/single_split_view.cc (right):
> 
> http://codereview.chromium.org/150120/diff/1/2#newcode149
> Line 149: if (GetChildViewCount() < 2)
> You need to update this method too. Otherwise if none of the child views want
> the mouse event, it's possible to get weird behavior depending upon the last
> size of the invisible views.

Done. Thanks for review.

[sky, 2009-07-01 15:28:31]

LGTM