Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6304)

Unified Diff: chrome/browser/ui/panels/base_panel_browser_test.cc

Issue 8802021: Fix a couple panel overflow related bugs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/panels/base_panel_browser_test.cc
diff --git a/chrome/browser/ui/panels/base_panel_browser_test.cc b/chrome/browser/ui/panels/base_panel_browser_test.cc
index 626d6b731ca4a2c5eee6f0120188ba3659907e24..82eb2e570ba30a8fa9fab2f108012a38803dafb7 100644
--- a/chrome/browser/ui/panels/base_panel_browser_test.cc
+++ b/chrome/browser/ui/panels/base_panel_browser_test.cc
@@ -254,6 +254,17 @@ void BasePanelBrowserTest::WaitForBoundsAnimationFinished(Panel* panel) {
EXPECT_TRUE(!panel_testing->IsAnimatingBounds());
}
+void BasePanelBrowserTest::WaitForExpansionStateChanged(
+ Panel* panel, Panel::ExpansionState expansion_state) {
+ ui_test_utils::WindowedNotificationObserver signal(
+ chrome::NOTIFICATION_PANEL_CHANGED_EXPANSION_STATE,
+ content::Source<Panel>(panel));
+ if (panel->expansion_state() == expansion_state)
+ return;
+ signal.Wait();
+ EXPECT_EQ(expansion_state, panel->expansion_state());
+}
+
Panel* BasePanelBrowserTest::CreatePanelWithParams(
const CreatePanelParams& params) {
#if defined(OS_MACOSX)

Powered by Google App Engine
This is Rietveld 408576698