Panels test cleanup.

Panels test cleanup.

Make MinimizeImmediatelyAfterRestore set up signal prior to action.
Simplify expansion state checks. They're not async.
More waiting in AutoResize test to avoid resizing content before it has finished sizing itself.

BUG=None
TEST=Updated/cleaned up.


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

[jennb, 2012-08-17 23:53:54]

Trying to make some panel tests less flaky...

[jianli, 2012-08-18 00:07:57]

lgtm

http://codereview.chromium.org/10836324/diff/1/chrome/browser/ui/panels/panel...
File chrome/browser/ui/panels/panel_gtk.cc (right):

http://codereview.chromium.org/10836324/diff/1/chrome/browser/ui/panels/panel...
chrome/browser/ui/panels/panel_gtk.cc:775: } else if (is_shown_) {
Is this if necessary?

[jennb, 2012-08-18 00:28:04]

http://codereview.chromium.org/10836324/diff/1/chrome/browser/ui/panels/panel...
File chrome/browser/ui/panels/panel_gtk.cc (right):

http://codereview.chromium.org/10836324/diff/1/chrome/browser/ui/panels/panel...
chrome/browser/ui/panels/panel_gtk.cc:775: } else if (is_shown_) {
On 2012/08/18 00:07:57, jianli wrote:
> Is this if necessary?

Good catch!

[commit-bot: I haz the power, 2012-08-18 01:01:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jennb@chromium.org/10836324/7002

[commit-bot: I haz the power, 2012-08-18 02:32:30]

Change committed as 152225