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

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

Issue 10483010: Change the visual appearance of panel on GTK per new UI design. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch Created 8 years, 6 months 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/panel_browsertest.cc
diff --git a/chrome/browser/ui/panels/panel_browsertest.cc b/chrome/browser/ui/panels/panel_browsertest.cc
index 0ac8e140da582acf1b7f067bb3a54514cacd62bf..0595764e4daa52313c117c658acbe4f3c2d8c517 100644
--- a/chrome/browser/ui/panels/panel_browsertest.cc
+++ b/chrome/browser/ui/panels/panel_browsertest.cc
@@ -291,26 +291,17 @@ IN_PROC_BROWSER_TEST_F(PanelBrowserTest, CheckDockedPanelProperties) {
EXPECT_TRUE(panel2->always_on_top());
EXPECT_TRUE(panel3->always_on_top());
- EXPECT_TRUE(panel1_testing->IsButtonVisible(
- NativePanelTesting::CLOSE_BUTTON));
- EXPECT_TRUE(panel2_testing->IsButtonVisible(
- NativePanelTesting::CLOSE_BUTTON));
- EXPECT_TRUE(panel3_testing->IsButtonVisible(
- NativePanelTesting::CLOSE_BUTTON));
-
- EXPECT_TRUE(panel1_testing->IsButtonVisible(
- NativePanelTesting::MINIMIZE_BUTTON));
- EXPECT_FALSE(panel2_testing->IsButtonVisible(
- NativePanelTesting::MINIMIZE_BUTTON));
- EXPECT_FALSE(panel3_testing->IsButtonVisible(
- NativePanelTesting::MINIMIZE_BUTTON));
-
- EXPECT_FALSE(panel1_testing->IsButtonVisible(
- NativePanelTesting::RESTORE_BUTTON));
- EXPECT_TRUE(panel2_testing->IsButtonVisible(
- NativePanelTesting::RESTORE_BUTTON));
- EXPECT_TRUE(panel3_testing->IsButtonVisible(
- NativePanelTesting::RESTORE_BUTTON));
+ EXPECT_TRUE(panel1_testing->IsButtonVisible(panel::CLOSE_BUTTON));
+ EXPECT_TRUE(panel2_testing->IsButtonVisible(panel::CLOSE_BUTTON));
+ EXPECT_TRUE(panel3_testing->IsButtonVisible(panel::CLOSE_BUTTON));
+
+ EXPECT_TRUE(panel1_testing->IsButtonVisible(panel::MINIMIZE_BUTTON));
+ EXPECT_FALSE(panel2_testing->IsButtonVisible(panel::MINIMIZE_BUTTON));
+ EXPECT_FALSE(panel3_testing->IsButtonVisible(panel::MINIMIZE_BUTTON));
+
+ EXPECT_FALSE(panel1_testing->IsButtonVisible(panel::RESTORE_BUTTON));
+ EXPECT_TRUE(panel2_testing->IsButtonVisible(panel::RESTORE_BUTTON));
+ EXPECT_TRUE(panel3_testing->IsButtonVisible(panel::RESTORE_BUTTON));
EXPECT_EQ(panel::RESIZABLE_ALL_SIDES_EXCEPT_BOTTOM,
panel1->CanResizeByMouse());

Powered by Google App Engine
This is Rietveld 408576698