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

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

Issue 10544113: Revert 141690 - Modify old Panel test files and include them in .gyp. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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/old_panel_browsertest.cc
===================================================================
--- chrome/browser/ui/panels/old_panel_browsertest.cc (revision 141691)
+++ chrome/browser/ui/panels/old_panel_browsertest.cc (working copy)
@@ -17,7 +17,7 @@
#include "chrome/browser/ui/browser_window.h"
#include "chrome/browser/ui/find_bar/find_bar.h"
#include "chrome/browser/ui/find_bar/find_bar_controller.h"
-#include "chrome/browser/ui/panels/old_base_panel_browser_test.h"
+#include "chrome/browser/ui/panels/base_panel_browser_test.h"
#include "chrome/browser/ui/panels/docked_panel_strip.h"
#include "chrome/browser/ui/panels/native_panel.h"
#include "chrome/browser/ui/panels/panel.h"
@@ -46,9 +46,9 @@
using content::WebContents;
using extensions::Extension;
-class OldPanelBrowserTest : public OldBasePanelBrowserTest {
+class PanelBrowserTest : public BasePanelBrowserTest {
public:
- OldPanelBrowserTest() : OldBasePanelBrowserTest() {
+ PanelBrowserTest() : BasePanelBrowserTest() {
}
protected:
@@ -255,7 +255,7 @@
}
};
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, CheckDockedPanelProperties) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, CheckDockedPanelProperties) {
PanelManager* panel_manager = PanelManager::GetInstance();
DockedPanelStrip* docked_strip = panel_manager->docked_strip();
@@ -316,7 +316,7 @@
panel_manager->CloseAll();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, CreatePanel) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, CreatePanel) {
PanelManager* panel_manager = PanelManager::GetInstance();
EXPECT_EQ(0, panel_manager->num_panels()); // No panels initially.
@@ -337,7 +337,7 @@
EXPECT_EQ(0, panel_manager->num_panels());
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, CreateBigPanel) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, CreateBigPanel) {
gfx::Rect work_area = PanelManager::GetInstance()->
display_settings_provider()->GetDisplayArea();
Panel* panel = CreatePanelWithBounds("BigPanel", work_area);
@@ -350,7 +350,7 @@
}
// Flaky: http://crbug.com/105445
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, DISABLED_AutoResize) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DISABLED_AutoResize) {
PanelManager* panel_manager = PanelManager::GetInstance();
panel_manager->enable_auto_sizing(true);
// Bigger space is needed by this test.
@@ -417,7 +417,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, ResizePanel) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, ResizePanel) {
PanelManager* panel_manager = PanelManager::GetInstance();
panel_manager->enable_auto_sizing(true);
@@ -450,7 +450,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, AnimateBounds) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, AnimateBounds) {
Panel* panel = CreatePanelWithBounds("PanelTest", gfx::Rect(0, 0, 100, 100));
scoped_ptr<NativePanelTesting> panel_testing(
NativePanelTesting::Create(panel->native_panel()));
@@ -472,7 +472,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, RestoredBounds) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, RestoredBounds) {
// Disable mouse watcher. We don't care about mouse movements in this test.
PanelManager* panel_manager = PanelManager::GetInstance();
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
@@ -533,7 +533,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MinimizeRestore) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MinimizeRestore) {
// We'll simulate mouse movements for test.
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
PanelManager::GetInstance()->SetMouseWatcherForTesting(mouse_watcher);
@@ -545,7 +545,7 @@
PanelManager::GetInstance()->CloseAll();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MinimizeRestoreTwoPanels) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MinimizeRestoreTwoPanels) {
// We'll simulate mouse movements for test.
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
PanelManager::GetInstance()->SetMouseWatcherForTesting(mouse_watcher);
@@ -558,7 +558,7 @@
PanelManager::GetInstance()->CloseAll();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MinimizeRestoreThreePanels) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MinimizeRestoreThreePanels) {
// We'll simulate mouse movements for test.
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
PanelManager::GetInstance()->SetMouseWatcherForTesting(mouse_watcher);
@@ -572,7 +572,7 @@
PanelManager::GetInstance()->CloseAll();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MinimizeRestoreButtonClick) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MinimizeRestoreButtonClick) {
// Test with three panels.
Panel* panel1 = CreatePanel("PanelTest1");
Panel* panel2 = CreatePanel("PanelTest2");
@@ -619,7 +619,7 @@
EXPECT_FALSE(panel3->IsMinimized());
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, RestoreAllWithTitlebarClick) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, RestoreAllWithTitlebarClick) {
// We'll simulate mouse movements for test.
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
PanelManager::GetInstance()->SetMouseWatcherForTesting(mouse_watcher);
@@ -792,7 +792,7 @@
#else
#define MAYBE_ActivatePanelOrTabbedWindow ActivatePanelOrTabbedWindow
#endif
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MAYBE_ActivatePanelOrTabbedWindow) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MAYBE_ActivatePanelOrTabbedWindow) {
CreatePanelParams params1("Panel1", gfx::Rect(), SHOW_AS_ACTIVE);
Panel* panel1 = CreatePanelWithParams(params1);
CreatePanelParams params2("Panel2", gfx::Rect(), SHOW_AS_ACTIVE);
@@ -840,7 +840,7 @@
#else
#define MAYBE_ActivateDeactivateBasic DISABLED_ActivateDeactivateBasic
#endif
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MAYBE_ActivateDeactivateBasic) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MAYBE_ActivateDeactivateBasic) {
// Create an active panel.
Panel* panel = CreatePanel("PanelTest");
scoped_ptr<NativePanelTesting> native_panel_testing(
@@ -867,7 +867,7 @@
#define MAYBE_ActivateDeactivateMultiple DISABLED_ActivateDeactivateMultiple
#endif
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, MAYBE_ActivateDeactivateMultiple) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MAYBE_ActivateDeactivateMultiple) {
BrowserWindow* tabbed_window = browser()->window();
// Create 4 panels in the following screen layout:
@@ -922,7 +922,7 @@
EXPECT_TRUE(tabbed_window->IsActive());
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, DrawAttentionBasic) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DrawAttentionBasic) {
CreatePanelParams params("Initially Inactive", gfx::Rect(), SHOW_AS_INACTIVE);
Panel* panel = CreatePanelWithParams(params);
scoped_ptr<NativePanelTesting> native_panel_testing(
@@ -959,7 +959,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, DrawAttentionWhileMinimized) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DrawAttentionWhileMinimized) {
// We'll simulate mouse movements for test.
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
PanelManager::GetInstance()->SetMouseWatcherForTesting(mouse_watcher);
@@ -1011,8 +1011,7 @@
// Verify that minimized state of a panel is correct after draw attention
// is stopped when there are other minimized panels.
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
- StopDrawingAttentionWhileMinimized) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, StopDrawingAttentionWhileMinimized) {
// We'll simulate mouse movements for test.
PanelMouseWatcher* mouse_watcher = new TestPanelMouseWatcher();
PanelManager::GetInstance()->SetMouseWatcherForTesting(mouse_watcher);
@@ -1079,7 +1078,7 @@
panel2->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, DrawAttentionWhenActive) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DrawAttentionWhenActive) {
CreatePanelParams params("Initially Active", gfx::Rect(), SHOW_AS_ACTIVE);
Panel* panel = CreatePanelWithParams(params);
scoped_ptr<NativePanelTesting> native_panel_testing(
@@ -1098,7 +1097,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, DrawAttentionResetOnActivate) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DrawAttentionResetOnActivate) {
// Create 2 panels so we end up with an inactive panel that can
// be made to draw attention.
Panel* panel = CreatePanel("test panel1");
@@ -1123,7 +1122,7 @@
panel2->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
DrawAttentionMinimizedNotResetOnActivate) {
// Create 2 panels so we end up with an inactive panel that can
// be made to draw attention.
@@ -1153,7 +1152,7 @@
panel2->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, DrawAttentionResetOnClick) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DrawAttentionResetOnClick) {
CreatePanelParams params("Initially Inactive", gfx::Rect(), SHOW_AS_INACTIVE);
Panel* panel = CreatePanelWithParams(params);
scoped_ptr<NativePanelTesting> native_panel_testing(
@@ -1177,7 +1176,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
MinimizeImmediatelyAfterRestore) {
CreatePanelParams params("Panel Test", gfx::Rect(), SHOW_AS_ACTIVE);
Panel* panel = CreatePanelWithParams(params);
@@ -1201,7 +1200,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, FocusLostOnMinimize) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, FocusLostOnMinimize) {
CreatePanelParams params("Initially Active", gfx::Rect(), SHOW_AS_ACTIVE);
Panel* panel = CreatePanelWithParams(params);
EXPECT_EQ(Panel::EXPANDED, panel->expansion_state());
@@ -1212,7 +1211,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, CreateInactiveSwitchToActive) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, CreateInactiveSwitchToActive) {
// Compiz will not activate initially inactive window.
if (SkipTestIfCompizWM())
return;
@@ -1238,7 +1237,7 @@
// When there are 2 panels and no chrome window, minimizing one panel does
// not expand/focuses another.
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
MAYBE_MinimizeTwoPanelsWithoutTabbedWindow) {
CreatePanelParams params("Initially Inactive", gfx::Rect(), SHOW_AS_INACTIVE);
Panel* panel1 = CreatePanelWithParams(params);
@@ -1278,7 +1277,7 @@
panel2->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
NonExtensionDomainPanelsCloseOnUninstall) {
// Create a test extension.
DictionaryValue empty_value;
@@ -1339,7 +1338,7 @@
panel_other->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, OnBeforeUnloadOnClose) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, OnBeforeUnloadOnClose) {
PanelManager* panel_manager = PanelManager::GetInstance();
EXPECT_EQ(0, panel_manager->num_panels()); // No panels initially.
@@ -1391,8 +1390,7 @@
// http://crbug.com/126381 - should find a better notification to wait
// for resize completion. Bounds animation could happen for all sorts of
// reasons.
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
- DISABLED_CreateWithExistingContents) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DISABLED_CreateWithExistingContents) {
PanelManager::GetInstance()->enable_auto_sizing(true);
// Load contents into regular tabbed browser.
@@ -1444,7 +1442,7 @@
tabbed_browser->window()->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, SizeClamping) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, SizeClamping) {
// Using '0' sizes is equivalent of not providing sizes in API and causes
// minimum sizes to be applied to facilitate auto-sizing.
CreatePanelParams params("Panel", gfx::Rect(), SHOW_AS_ACTIVE);
@@ -1476,7 +1474,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest, TightAutosizeAroundSingleLine) {
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, TightAutosizeAroundSingleLine) {
PanelManager::GetInstance()->enable_auto_sizing(true);
// Using 0 sizes triggers auto-sizing.
CreatePanelParams params("Panel", gfx::Rect(), SHOW_AS_ACTIVE);
@@ -1506,7 +1504,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
DefaultMaxSizeOnDisplaySettingsChange) {
Panel* panel = CreatePanelWithBounds("1", gfx::Rect(0, 0, 240, 220));
@@ -1529,7 +1527,7 @@
panel->Close();
}
-IN_PROC_BROWSER_TEST_F(OldPanelBrowserTest,
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
CustomMaxSizeOnDisplaySettingsChange) {
PanelManager* panel_manager = PanelManager::GetInstance();
Panel* panel = CreatePanelWithBounds("1", gfx::Rect(0, 0, 240, 220));
@@ -1567,9 +1565,9 @@
panel->Close();
}
-class OldPanelDownloadTest : public OldPanelBrowserTest {
+class PanelDownloadTest : public PanelBrowserTest {
public:
- OldPanelDownloadTest() : OldPanelBrowserTest() { }
+ PanelDownloadTest() : PanelBrowserTest() { }
// Creates a temporary directory for downloads that is auto-deleted
// on destruction.
@@ -1585,7 +1583,7 @@
protected:
void SetUpOnMainThread() OVERRIDE {
- OldPanelBrowserTest::SetUpOnMainThread();
+ PanelBrowserTest::SetUpOnMainThread();
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
@@ -1645,7 +1643,7 @@
// Verify that the download shelf is opened in the existing tabbed browser
// when a download is started in a Panel.
-IN_PROC_BROWSER_TEST_F(OldPanelDownloadTest, Download) {
+IN_PROC_BROWSER_TEST_F(PanelDownloadTest, Download) {
Profile* profile = browser()->profile();
ASSERT_TRUE(CreateDownloadDirectory(profile));
Browser* panel_browser = Browser::CreateWithParams(
@@ -1698,7 +1696,7 @@
// Verify that a new tabbed browser is created to display a download
// shelf when a download is started in a Panel and there is no existing
// tabbed browser.
-IN_PROC_BROWSER_TEST_F(OldPanelDownloadTest, MAYBE_DownloadNoTabbedBrowser) {
+IN_PROC_BROWSER_TEST_F(PanelDownloadTest, MAYBE_DownloadNoTabbedBrowser) {
Profile* profile = browser()->profile();
ASSERT_TRUE(CreateDownloadDirectory(profile));
Browser* panel_browser = Browser::CreateWithParams(

Powered by Google App Engine
This is Rietveld 408576698