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

Unified Diff: chrome/browser/ui/ash/window_positioner_unittest.cc

Issue 13139004: Deprecate Browser::TYPE_PANEL (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 9 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
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_item_controller.cc ('k') | chrome/browser/ui/browser.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/ash/window_positioner_unittest.cc
diff --git a/chrome/browser/ui/ash/window_positioner_unittest.cc b/chrome/browser/ui/ash/window_positioner_unittest.cc
index 04b1c91138bc4afc83def14291f990781a65cf1d..776c7bad6216b6ae21561bd80ee65e8ad50479e7 100644
--- a/chrome/browser/ui/ash/window_positioner_unittest.cc
+++ b/chrome/browser/ui/ash/window_positioner_unittest.cc
@@ -55,7 +55,7 @@ TestBrowserWindowAura::~TestBrowserWindowAura() {}
} // namespace
// A test class for preparing window positioner tests - it creates a testing
-// base by adding a window, a popup and a panel which can be independently
+// base by adding a window and a popup which can be independently
// positioned to see where the positioner will place the window.
class WindowPositionerTest : public AshTestBase {
public:
@@ -68,11 +68,9 @@ class WindowPositionerTest : public AshTestBase {
protected:
aura::Window* window() { return window_.get(); }
aura::Window* popup() { return popup_.get(); }
- aura::Window* panel() { return panel_.get(); }
Browser* window_browser() { return window_owning_browser_.get(); }
Browser* popup_browser() { return popup_owning_browser_.get(); }
- Browser* panel_browser() { return panel_owning_browser_.get(); }
WindowPositioner* window_positioner() { return window_positioner_; }
@@ -89,15 +87,12 @@ class WindowPositionerTest : public AshTestBase {
// These get created for each session.
scoped_ptr<aura::Window> window_;
scoped_ptr<aura::Window> popup_;
- scoped_ptr<aura::Window> panel_;
scoped_ptr<BrowserWindow> browser_window_;
scoped_ptr<BrowserWindow> browser_popup_;
- scoped_ptr<BrowserWindow> browser_panel_;
scoped_ptr<Browser> window_owning_browser_;
scoped_ptr<Browser> popup_owning_browser_;
- scoped_ptr<Browser> panel_owning_browser_;
DISALLOW_COPY_AND_ASSIGN(WindowPositionerTest);
};
@@ -126,8 +121,6 @@ void WindowPositionerTest::SetUp() {
window_->SetBounds(gfx::Rect(16, 32, 640, 320));
popup_.reset(CreateTestWindowInShellWithId(1));
popup_->SetBounds(gfx::Rect(16, 32, 128, 256));
- panel_.reset(CreateTestWindowInShellWithId(2));
- panel_->SetBounds(gfx::Rect(32, 48, 256, 512));
// Create a browser for the window.
browser_window_.reset(new TestBrowserWindowAura(window_.get()));
@@ -143,17 +136,10 @@ void WindowPositionerTest::SetUp() {
popup_params.window = browser_popup_.get();
popup_owning_browser_.reset(new Browser(popup_params));
- // Creating a browser for the panel.
- browser_panel_.reset(new TestBrowserWindowAura(panel_.get()));
- Browser::CreateParams panel_params(Browser::TYPE_PANEL, profile_.get(),
- chrome::HOST_DESKTOP_TYPE_ASH);
- panel_params.window = browser_panel_.get();
- panel_owning_browser_.reset(new Browser(panel_params));
// We hide all windows upon start - each user is required to set it up
// as he needs it.
window()->Hide();
popup()->Hide();
- panel()->Hide();
window_positioner_ = new WindowPositioner();
}
@@ -162,15 +148,12 @@ void WindowPositionerTest::TearDown() {
// also delete them before we tear it down.
window_owning_browser_.reset(NULL);
popup_owning_browser_.reset(NULL);
- panel_owning_browser_.reset(NULL);
browser_window_.reset(NULL);
browser_popup_.reset(NULL);
- browser_panel_.reset(NULL);
window_.reset(NULL);
popup_.reset(NULL);
- panel_.reset(NULL);
AshTestBase::TearDown();
delete window_positioner_;
@@ -299,22 +282,6 @@ TEST_F(WindowPositionerTest, filling) {
top_right);
}
-TEST_F(WindowPositionerTest, blockedByPanel) {
- const gfx::Rect work_area =
- Shell::GetScreen()->GetPrimaryDisplay().work_area();
-
- gfx::Rect pop_position(0, 0, 200, 200);
- // Let the panel cover everything.
- panel()->SetBounds(work_area);
- panel()->Show();
-
- // Check that the popup does cascade due to the panel's existence.
- gfx::Rect top_right = window_positioner()->GetPopupPosition(pop_position);
- EXPECT_EQ(gfx::Rect(work_area.x() + grid_size_, work_area.y() + grid_size_,
- pop_position.width(), pop_position.height()),
- top_right);
-}
-
TEST_F(WindowPositionerTest, biggerThenBorder) {
const gfx::Rect work_area =
Shell::GetScreen()->GetPrimaryDisplay().work_area();
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_item_controller.cc ('k') | chrome/browser/ui/browser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698