Update widget size from StatusAreaView::Layout()

Update widget size from StatusAreaView::Layout()

Change-Id: I461b53554bb67f6fe9681d04940e2131e2a9224c

BUG=107309
TEST=See issue


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

[stevenjb, 2011-12-13 02:29:02]

If we only call GetWidget()->SetSize() from ChildPreferredSizeChanged(), we
won't catch size changes due to visibility changes. Instead, call
GetWidget()->SetSize() from Layout() which is called from both
ChildPreferredSizeChanged() and UpdateButtonVisibility().

[sky, 2011-12-13 16:40:41]

http://codereview.chromium.org/8932001/diff/1/chrome/browser/chromeos/status/...
File chrome/browser/chromeos/status/status_area_view.cc (right):

http://codereview.chromium.org/8932001/diff/1/chrome/browser/chromeos/status/...
chrome/browser/chromeos/status/status_area_view.cc:92: if (GetWidget())
This makes it easy to get stuck in a loop. Can we instead invoke
PreferredSizeChanged when toggling visibility?

[stevenjb, 2011-12-14 02:10:51]

http://codereview.chromium.org/8932001/diff/1/chrome/browser/chromeos/status/...
File chrome/browser/chromeos/status/status_area_view.cc (right):

http://codereview.chromium.org/8932001/diff/1/chrome/browser/chromeos/status/...
chrome/browser/chromeos/status/status_area_view.cc:92: if (GetWidget())
On 2011/12/13 16:40:41, sky wrote:
> This makes it easy to get stuck in a loop. Can we instead invoke
> PreferredSizeChanged when toggling visibility?

We already do; I will override PreferredSizeChanged() and move this chunk of
code there.
Done.

[sky, 2011-12-14 15:33:55]

LGTM

[commit-bot: I haz the power, 2011-12-14 20:26:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/8932001/12001

[commit-bot: I haz the power, 2011-12-14 22:13:00]

Change committed as 114501

[Nikita (slow), 2011-12-26 10:44:53]

lgtm