| Index: ash/shelf/shelf_view_unittest.cc
|
| diff --git a/ash/shelf/shelf_view_unittest.cc b/ash/shelf/shelf_view_unittest.cc
|
| index ec1468ab10dfd89f9a8de9d64f994afac178f51c..81913790e5a476754bbca9ef080422567910fb40 100644
|
| --- a/ash/shelf/shelf_view_unittest.cc
|
| +++ b/ash/shelf/shelf_view_unittest.cc
|
| @@ -8,11 +8,11 @@
|
| #include <vector>
|
|
|
| #include "ash/ash_switches.h"
|
| -#include "ash/launcher/launcher.h"
|
| #include "ash/launcher/launcher_types.h"
|
| #include "ash/root_window_controller.h"
|
| #include "ash/shelf/overflow_bubble.h"
|
| #include "ash/shelf/overflow_bubble_view.h"
|
| +#include "ash/shelf/shelf.h"
|
| #include "ash/shelf/shelf_button.h"
|
| #include "ash/shelf/shelf_icon_observer.h"
|
| #include "ash/shelf/shelf_item_delegate_manager.h"
|
| @@ -23,8 +23,8 @@
|
| #include "ash/shell.h"
|
| #include "ash/shell_window_ids.h"
|
| #include "ash/test/ash_test_base.h"
|
| -#include "ash/test/launcher_test_api.h"
|
| #include "ash/test/overflow_bubble_view_test_api.h"
|
| +#include "ash/test/shelf_test_api.h"
|
| #include "ash/test/shelf_view_test_api.h"
|
| #include "ash/test/shell_test_api.h"
|
| #include "ash/test/test_shelf_delegate.h"
|
| @@ -56,16 +56,16 @@ namespace test {
|
|
|
| class TestShelfIconObserver : public ShelfIconObserver {
|
| public:
|
| - explicit TestShelfIconObserver(Launcher* launcher)
|
| - : launcher_(launcher),
|
| + explicit TestShelfIconObserver(Shelf* shelf)
|
| + : shelf_(shelf),
|
| change_notified_(false) {
|
| - if (launcher_)
|
| - launcher_->AddIconObserver(this);
|
| + if (shelf_)
|
| + shelf_->AddIconObserver(this);
|
| }
|
|
|
| virtual ~TestShelfIconObserver() {
|
| - if (launcher_)
|
| - launcher_->RemoveIconObserver(this);
|
| + if (shelf_)
|
| + shelf_->RemoveIconObserver(this);
|
| }
|
|
|
| // ShelfIconObserver implementation.
|
| @@ -77,7 +77,7 @@ class TestShelfIconObserver : public ShelfIconObserver {
|
| void Reset() { change_notified_ = false; }
|
|
|
| private:
|
| - Launcher* launcher_;
|
| + Shelf* shelf_;
|
| bool change_notified_;
|
|
|
| DISALLOW_COPY_AND_ASSIGN(TestShelfIconObserver);
|
| @@ -90,11 +90,11 @@ class ShelfViewIconObserverTest : public AshTestBase {
|
|
|
| virtual void SetUp() OVERRIDE {
|
| AshTestBase::SetUp();
|
| - Launcher* launcher = Launcher::ForPrimaryDisplay();
|
| - observer_.reset(new TestShelfIconObserver(launcher));
|
| + Shelf* shelf = Shelf::ForPrimaryDisplay();
|
| + observer_.reset(new TestShelfIconObserver(shelf));
|
|
|
| - shelf_view_test_.reset(new ShelfViewTestAPI(
|
| - LauncherTestAPI(launcher).shelf_view()));
|
| + shelf_view_test_.reset(
|
| + new ShelfViewTestAPI(ShelfTestAPI(shelf).shelf_view()));
|
| shelf_view_test_->SetAnimationDuration(1);
|
| }
|
|
|
| @@ -109,8 +109,8 @@ class ShelfViewIconObserverTest : public AshTestBase {
|
| return shelf_view_test_.get();
|
| }
|
|
|
| - Launcher* LauncherForSecondaryDisplay() {
|
| - return Launcher::ForWindow(Shell::GetAllRootWindows()[1]);
|
| + Shelf* ShelfForSecondaryDisplay() {
|
| + return Shelf::ForWindow(Shell::GetAllRootWindows()[1]);
|
| }
|
|
|
| private:
|
| @@ -152,10 +152,10 @@ TEST_F(ShelfViewIconObserverTest, AddRemove) {
|
| AddRemoveWithMultipleDisplays
|
| #endif
|
| // Make sure creating/deleting an window on one displays notifies a
|
| -// launcher on external display as well as one on primary.
|
| +// shelf on external display as well as one on primary.
|
| TEST_F(ShelfViewIconObserverTest, MAYBE_AddRemoveWithMultipleDisplays) {
|
| UpdateDisplay("400x400,400x400");
|
| - TestShelfIconObserver second_observer(LauncherForSecondaryDisplay());
|
| + TestShelfIconObserver second_observer(ShelfForSecondaryDisplay());
|
|
|
| TestShelfDelegate* shelf_delegate = TestShelfDelegate::instance();
|
| ASSERT_TRUE(shelf_delegate);
|
| @@ -184,13 +184,12 @@ TEST_F(ShelfViewIconObserverTest, MAYBE_AddRemoveWithMultipleDisplays) {
|
| }
|
|
|
| TEST_F(ShelfViewIconObserverTest, BoundsChanged) {
|
| - ShelfWidget* shelf = Shell::GetPrimaryRootWindowController()->shelf();
|
| - Launcher* launcher = Launcher::ForPrimaryDisplay();
|
| - gfx::Size shelf_size =
|
| - shelf->GetWindowBoundsInScreen().size();
|
| + ShelfWidget* widget = Shell::GetPrimaryRootWindowController()->shelf();
|
| + Shelf* shelf = Shelf::ForPrimaryDisplay();
|
| + gfx::Size shelf_size = widget->GetWindowBoundsInScreen().size();
|
| shelf_size.set_width(shelf_size.width() / 2);
|
| ASSERT_GT(shelf_size.width(), 0);
|
| - launcher->SetShelfViewBounds(gfx::Rect(shelf_size));
|
| + shelf->SetShelfViewBounds(gfx::Rect(shelf_size));
|
| // No animation happens for ShelfView bounds change.
|
| EXPECT_TRUE(observer()->change_notified());
|
| observer()->Reset();
|
| @@ -208,9 +207,9 @@ class TestShelfDelegateForShelfView : public ShelfDelegate {
|
| virtual ~TestShelfDelegateForShelfView() {}
|
|
|
| // ShelfDelegate overrides:
|
| - virtual void OnLauncherCreated(Launcher* launcher) OVERRIDE {}
|
| + virtual void OnShelfCreated(Shelf* shelf) OVERRIDE {}
|
|
|
| - virtual void OnLauncherDestroyed(Launcher* launcher) OVERRIDE {}
|
| + virtual void OnShelfDestroyed(Shelf* shelf) OVERRIDE {}
|
|
|
| virtual LauncherID GetLauncherIDForAppID(const std::string& app_id) OVERRIDE {
|
| LauncherID id = 0;
|
| @@ -267,8 +266,8 @@ class ShelfViewTest : public AshTestBase {
|
| AshTestBase::SetUp();
|
| test::ShellTestApi test_api(Shell::GetInstance());
|
| model_ = test_api.shelf_model();
|
| - Launcher* launcher = Launcher::ForPrimaryDisplay();
|
| - shelf_view_ = test::LauncherTestAPI(launcher).shelf_view();
|
| + Shelf* shelf = Shelf::ForPrimaryDisplay();
|
| + shelf_view_ = ShelfTestAPI(shelf).shelf_view();
|
|
|
| // The bounds should be big enough for 4 buttons + overflow chevron.
|
| shelf_view_->SetBounds(0, 0, 500,
|
| @@ -280,7 +279,7 @@ class ShelfViewTest : public AshTestBase {
|
| item_manager_ = Shell::GetInstance()->shelf_item_delegate_manager();
|
| DCHECK(item_manager_);
|
|
|
| - // Add browser shortcut launcher item at index 0 for test.
|
| + // Add browser shortcut shelf item at index 0 for test.
|
| AddBrowserShortcut();
|
| }
|
|
|
| @@ -382,7 +381,7 @@ class ShelfViewTest : public AshTestBase {
|
| ASSERT_EQ(map_index, id_map.size());
|
| }
|
|
|
| - void VerifyLauncherItemBoundsAreValid() {
|
| + void VerifyShelfItemBoundsAreValid() {
|
| for (int i=0;i <= test_api_->GetLastVisibleIndex(); ++i) {
|
| if (test_api_->GetButton(i)) {
|
| gfx::Rect shelf_view_bounds = shelf_view_->GetLocalBounds();
|
| @@ -434,14 +433,14 @@ class ShelfViewTest : public AshTestBase {
|
|
|
| void SetupForDragTest(
|
| std::vector<std::pair<LauncherID, views::View*> >* id_map) {
|
| - // Initialize |id_map| with the automatically-created launcher buttons.
|
| + // Initialize |id_map| with the automatically-created shelf buttons.
|
| for (size_t i = 0; i < model_->items().size(); ++i) {
|
| internal::ShelfButton* button = test_api_->GetButton(i);
|
| id_map->push_back(std::make_pair(model_->items()[i].id, button));
|
| }
|
| ASSERT_NO_FATAL_FAILURE(CheckModelIDs(*id_map));
|
|
|
| - // Add 5 app launcher buttons for testing.
|
| + // Add 5 app shelf buttons for testing.
|
| for (int i = 0; i < 5; ++i) {
|
| LauncherID id = AddAppShortcut();
|
| // App Icon is located at index 0, and browser shortcut is located at
|
| @@ -940,7 +939,7 @@ TEST_F(ShelfViewTest, AddButtonQuickly) {
|
| }
|
| }
|
|
|
| -// Check that model changes are handled correctly while a launcher icon is being
|
| +// Check that model changes are handled correctly while a shelf icon is being
|
| // dragged.
|
| TEST_F(ShelfViewTest, ModelChangesWhileDragging) {
|
| internal::ShelfButtonHost* button_host = shelf_view_;
|
| @@ -986,7 +985,7 @@ TEST_F(ShelfViewTest, ModelChangesWhileDragging) {
|
| internal::ShelfButtonHost::MOUSE,
|
| false);
|
|
|
| - // Adding a launcher item cancels the drag and respects the order.
|
| + // Adding a shelf item cancels the drag and respects the order.
|
| dragged_button = SimulateDrag(internal::ShelfButtonHost::MOUSE, 1, 3);
|
| LauncherID new_id = AddAppShortcut();
|
| id_map.insert(id_map.begin() + 6,
|
| @@ -996,7 +995,7 @@ TEST_F(ShelfViewTest, ModelChangesWhileDragging) {
|
| internal::ShelfButtonHost::MOUSE,
|
| false);
|
|
|
| - // Adding a launcher item at the end (i.e. a panel) canels drag and respects
|
| + // Adding a shelf item at the end (i.e. a panel) canels drag and respects
|
| // the order.
|
| dragged_button = SimulateDrag(internal::ShelfButtonHost::MOUSE, 1, 3);
|
| new_id = AddPanel();
|
| @@ -1052,7 +1051,7 @@ TEST_F(ShelfViewLegacyShelfLayoutTest, ModelChangesWhileDragging) {
|
| internal::ShelfButtonHost::MOUSE,
|
| false);
|
|
|
| - // Adding a launcher item cancels the drag and respects the order.
|
| + // Adding a shelf item cancels the drag and respects the order.
|
| dragged_button = SimulateDrag(internal::ShelfButtonHost::MOUSE, 0, 2);
|
| LauncherID new_id = AddAppShortcut();
|
| id_map.insert(id_map.begin() + 5,
|
| @@ -1062,7 +1061,7 @@ TEST_F(ShelfViewLegacyShelfLayoutTest, ModelChangesWhileDragging) {
|
| internal::ShelfButtonHost::MOUSE,
|
| false);
|
|
|
| - // Adding a launcher item at the end (i.e. a panel) canels drag and respects
|
| + // Adding a shelf item at the end (i.e. a panel) canels drag and respects
|
| // the order.
|
| dragged_button = SimulateDrag(internal::ShelfButtonHost::MOUSE, 0, 2);
|
| new_id = AddPanel();
|
| @@ -1146,7 +1145,7 @@ TEST_F(ShelfViewTest, ClickOneDragAnother) {
|
| }
|
|
|
| // Confirm that item status changes are reflected in the buttons.
|
| -TEST_F(ShelfViewTest, LauncherItemStatus) {
|
| +TEST_F(ShelfViewTest, ShelfItemStatus) {
|
| ASSERT_EQ(test_api_->GetLastVisibleIndex() + 1,
|
| test_api_->GetButtonCount());
|
|
|
| @@ -1165,11 +1164,11 @@ TEST_F(ShelfViewTest, LauncherItemStatus) {
|
| }
|
|
|
| TEST_F(ShelfViewLegacyShelfLayoutTest,
|
| - LauncherItemPositionReflectedOnStateChanged) {
|
| + ShelfItemPositionReflectedOnStateChanged) {
|
| ASSERT_EQ(test_api_->GetLastVisibleIndex() + 1,
|
| test_api_->GetButtonCount());
|
|
|
| - // Add 2 items to the launcher.
|
| + // Add 2 items to the shelf.
|
| LauncherID item1_id = AddPlatformApp();
|
| LauncherID item2_id = AddPlatformAppNoWait();
|
| internal::ShelfButton* item1_button = GetButtonByID(item1_id);
|
| @@ -1199,7 +1198,7 @@ TEST_F(ShelfViewLegacyShelfLayoutTest,
|
|
|
| // Confirm that item status changes are reflected in the buttons
|
| // for platform apps.
|
| -TEST_F(ShelfViewTest, LauncherItemStatusPlatformApp) {
|
| +TEST_F(ShelfViewTest, ShelfItemStatusPlatformApp) {
|
| ASSERT_EQ(test_api_->GetLastVisibleIndex() + 1,
|
| test_api_->GetButtonCount());
|
|
|
| @@ -1217,24 +1216,24 @@ TEST_F(ShelfViewTest, LauncherItemStatusPlatformApp) {
|
| ASSERT_EQ(internal::ShelfButton::STATE_ATTENTION, button->state());
|
| }
|
|
|
| -// Confirm that launcher item bounds are correctly updated on shelf changes.
|
| -TEST_F(ShelfViewTest, LauncherItemBoundsCheck) {
|
| - VerifyLauncherItemBoundsAreValid();
|
| +// Confirm that shelf item bounds are correctly updated on shelf changes.
|
| +TEST_F(ShelfViewTest, ShelfItemBoundsCheck) {
|
| + VerifyShelfItemBoundsAreValid();
|
| shelf_view_->shelf_layout_manager()->SetAutoHideBehavior(
|
| SHELF_AUTO_HIDE_BEHAVIOR_ALWAYS);
|
| test_api_->RunMessageLoopUntilAnimationsDone();
|
| - VerifyLauncherItemBoundsAreValid();
|
| + VerifyShelfItemBoundsAreValid();
|
| shelf_view_->shelf_layout_manager()->SetAutoHideBehavior(
|
| SHELF_AUTO_HIDE_BEHAVIOR_NEVER);
|
| test_api_->RunMessageLoopUntilAnimationsDone();
|
| - VerifyLauncherItemBoundsAreValid();
|
| + VerifyShelfItemBoundsAreValid();
|
| }
|
|
|
| TEST_F(ShelfViewTest, ShelfTooltipTest) {
|
| ASSERT_EQ(test_api_->GetLastVisibleIndex() + 1,
|
| test_api_->GetButtonCount());
|
|
|
| - // Prepare some items to the launcher.
|
| + // Prepare some items to the shelf.
|
| LauncherID app_button_id = AddAppShortcut();
|
| LauncherID platform_button_id = AddPlatformApp();
|
|
|
| @@ -1276,14 +1275,14 @@ TEST_F(ShelfViewTest, ShelfTooltipTest) {
|
| EXPECT_EQ(platform_button, GetTooltipAnchorView());
|
| }
|
|
|
| -// Verify a fix for crash caused by a tooltip update for a deleted launcher
|
| +// Verify a fix for crash caused by a tooltip update for a deletedshelf
|
| // button, see crbug.com/288838.
|
| TEST_F(ShelfViewTest, RemovingItemClosesTooltip) {
|
| internal::ShelfButtonHost* button_host = shelf_view_;
|
| internal::ShelfTooltipManager* tooltip_manager =
|
| shelf_view_->tooltip_manager();
|
|
|
| - // Add an item to the launcher.
|
| + // Add an item to the shelf.
|
| LauncherID app_button_id = AddAppShortcut();
|
| internal::ShelfButton* app_button = GetButtonByID(app_button_id);
|
|
|
| @@ -1308,7 +1307,7 @@ TEST_F(ShelfViewTest, ShelfAlignmentClosesTooltip) {
|
| internal::ShelfTooltipManager* tooltip_manager =
|
| shelf_view_->tooltip_manager();
|
|
|
| - // Add an item to the launcher.
|
| + // Add an item to the shelf.
|
| LauncherID app_button_id = AddAppShortcut();
|
| internal::ShelfButton* app_button = GetButtonByID(app_button_id);
|
|
|
| @@ -1476,12 +1475,11 @@ TEST_F(ShelfViewTest, ResizeDuringOverflowAddAnimation) {
|
| // Checks the overflow bubble size when an item is ripped off and re-inserted.
|
| TEST_F(ShelfViewTest, OverflowBubbleSize) {
|
| // Replace ShelfDelegate.
|
| - test::ShellTestApi test_api(Shell::GetInstance());
|
| + ShellTestApi test_api(Shell::GetInstance());
|
| test_api.SetShelfDelegate(NULL);
|
| ShelfDelegate *delegate = new TestShelfDelegateForShelfView(model_);
|
| test_api.SetShelfDelegate(delegate);
|
| - test::LauncherTestAPI(
|
| - Launcher::ForPrimaryDisplay()).SetShelfDelegate(delegate);
|
| + ShelfTestAPI(Shelf::ForPrimaryDisplay()).SetShelfDelegate(delegate);
|
| test_api_->SetShelfDelegate(delegate);
|
|
|
| AddButtonsUntilOverflow();
|
| @@ -1608,10 +1606,9 @@ TEST_F(ShelfViewTest, CheckDragInsertBoundsWithMultiMonitor) {
|
| return;
|
|
|
| UpdateDisplay("800x600,800x600");
|
| - Launcher* secondary_launcher =
|
| - Launcher::ForWindow(Shell::GetAllRootWindows()[1]);
|
| + Shelf* secondary_shelf = Shelf::ForWindow(Shell::GetAllRootWindows()[1]);
|
| internal::ShelfView* shelf_view_for_secondary =
|
| - test::LauncherTestAPI(secondary_launcher).shelf_view();
|
| + ShelfTestAPI(secondary_shelf).shelf_view();
|
|
|
| // The bounds should be big enough for 4 buttons + overflow chevron.
|
| shelf_view_for_secondary->SetBounds(0, 0, 500,
|
| @@ -1692,9 +1689,9 @@ TEST_F(ShelfViewTest, CheckRipOffFromLeftShelfAlignmentWithMultiMonitor) {
|
| int browser_index = model_->GetItemIndexForType(TYPE_BROWSER_SHORTCUT);
|
| EXPECT_GT(browser_index, 0);
|
|
|
| - Launcher* secondary_launcher = Launcher::ForWindow(second_root);
|
| + Shelf* secondary_shelf = Shelf::ForWindow(second_root);
|
| internal::ShelfView* shelf_view_for_secondary =
|
| - test::LauncherTestAPI(secondary_launcher).shelf_view();
|
| + ShelfTestAPI(secondary_shelf).shelf_view();
|
|
|
| ShelfViewTestAPI test_api_for_secondary_shelf_view(shelf_view_for_secondary);
|
| internal::ShelfButton* button =
|
| @@ -1715,13 +1712,12 @@ TEST_F(ShelfViewTest, CheckRipOffFromLeftShelfAlignmentWithMultiMonitor) {
|
|
|
| // Checks various drag and drop operations from OverflowBubble to Shelf.
|
| TEST_F(ShelfViewTest, CheckDragAndDropFromOverflowBubbleToShelf) {
|
| - // Replace LauncherDelegate.
|
| - test::ShellTestApi test_api(Shell::GetInstance());
|
| + // Replace ShelfDelegate.
|
| + ShellTestApi test_api(Shell::GetInstance());
|
| test_api.SetShelfDelegate(NULL);
|
| ShelfDelegate *delegate = new TestShelfDelegateForShelfView(model_);
|
| test_api.SetShelfDelegate(delegate);
|
| - test::LauncherTestAPI(
|
| - Launcher::ForPrimaryDisplay()).SetShelfDelegate(delegate);
|
| + ShelfTestAPI(Shelf::ForPrimaryDisplay()).SetShelfDelegate(delegate);
|
| test_api_->SetShelfDelegate(delegate);
|
|
|
| AddButtonsUntilOverflow();
|
| @@ -1737,8 +1733,8 @@ class ShelfViewVisibleBoundsTest : public ShelfViewTest,
|
|
|
| void CheckAllItemsAreInBounds() {
|
| gfx::Rect visible_bounds = shelf_view_->GetVisibleItemsBoundsInScreen();
|
| - gfx::Rect launcher_bounds = shelf_view_->GetBoundsInScreen();
|
| - EXPECT_TRUE(launcher_bounds.Contains(visible_bounds));
|
| + gfx::Rect shelf_bounds = shelf_view_->GetBoundsInScreen();
|
| + EXPECT_TRUE(shelf_bounds.Contains(visible_bounds));
|
| for (int i = 0; i < test_api_->GetButtonCount(); ++i)
|
| if (internal::ShelfButton* button = test_api_->GetButton(i))
|
| EXPECT_TRUE(visible_bounds.Contains(button->GetBoundsInScreen()));
|
|
|