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

Unified Diff: chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc

Issue 97173003: ash: Rename more Launcher classes to Shelf*. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: similarity 30 Created 7 years, 1 month 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/ash/launcher/chrome_launcher_controller_unittest.cc
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc
index 38c24fe85fca381ab2da6c8eadd5deba4d8b01ca..249a22d69cf44a03f4020a2ce15c42637d70c7e3 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc
@@ -9,11 +9,11 @@
#include <vector>
#include "ash/ash_switches.h"
-#include "ash/launcher/launcher_item_delegate_manager.h"
+#include "ash/shelf/shelf_item_delegate_manager.h"
#include "ash/shelf/shelf_model.h"
#include "ash/shelf/shelf_model_observer.h"
#include "ash/shell.h"
-#include "ash/test/launcher_item_delegate_manager_test_api.h"
+#include "ash/test/shelf_item_delegate_manager_test_api.h"
#include "base/command_line.h"
#include "base/compiler_specific.h"
#include "base/files/file_path.h"
@@ -274,10 +274,10 @@ class ChromeLauncherControllerTest : public BrowserWithTestWindowTest {
if (ash::Shell::HasInstance()) {
item_delegate_manager_ =
- ash::Shell::GetInstance()->launcher_item_delegate_manager();
+ ash::Shell::GetInstance()->shelf_item_delegate_manager();
} else {
item_delegate_manager_ =
- new ash::LauncherItemDelegateManager(model_.get());
+ new ash::ShelfItemDelegateManager(model_.get());
}
DictionaryValue manifest;
@@ -426,7 +426,7 @@ class ChromeLauncherControllerTest : public BrowserWithTestWindowTest {
launcher_controller_.reset(
new ChromeLauncherController(profile(), model_.get()));
if (!ash::Shell::HasInstance())
- SetLauncherItemDelegateManager(item_delegate_manager_);
+ SetShelfItemDelegateManager(item_delegate_manager_);
launcher_controller_->Init();
}
@@ -444,9 +444,8 @@ class ChromeLauncherControllerTest : public BrowserWithTestWindowTest {
launcher_controller_->SetAppTabHelperForTest(helper);
}
- void SetLauncherItemDelegateManager(
- ash::LauncherItemDelegateManager* manager) {
- launcher_controller_->SetLauncherItemDelegateManagerForTest(manager);
+ void SetShelfItemDelegateManager(ash::ShelfItemDelegateManager* manager) {
+ launcher_controller_->SetShelfItemDelegateManagerForTest(manager);
}
void InsertPrefValue(base::ListValue* pref_value,
@@ -595,7 +594,7 @@ class ChromeLauncherControllerTest : public BrowserWithTestWindowTest {
ExtensionService* extension_service_;
- ash::LauncherItemDelegateManager* item_delegate_manager_;
+ ash::ShelfItemDelegateManager* item_delegate_manager_;
DISALLOW_COPY_AND_ASSIGN(ChromeLauncherControllerTest);
};
@@ -2640,9 +2639,9 @@ TEST_F(ChromeLauncherControllerTest, PersistLauncherItemPositions) {
if (!ash::Shell::HasInstance()) {
delete item_delegate_manager_;
} else {
- // Clear already registered LauncherItemDelegate.
- ash::test::LauncherItemDelegateManagerTestAPI test(item_delegate_manager_);
- test.RemoveAllLauncherItemDelegateForTest();
+ // Clear already registered ShelfItemDelegate.
+ ash::test::ShelfItemDelegateManagerTestAPI test(item_delegate_manager_);
+ test.RemoveAllShelfItemDelegateForTest();
}
model_.reset(new ash::ShelfModel);
@@ -2654,9 +2653,8 @@ TEST_F(ChromeLauncherControllerTest, PersistLauncherItemPositions) {
app_tab_helper->SetAppID(tab_strip_model->GetWebContentsAt(1), "2");
SetAppTabHelper(app_tab_helper);
if (!ash::Shell::HasInstance()) {
- item_delegate_manager_ =
- new ash::LauncherItemDelegateManager(model_.get());
- SetLauncherItemDelegateManager(item_delegate_manager_);
+ item_delegate_manager_ = new ash::ShelfItemDelegateManager(model_.get());
+ SetShelfItemDelegateManager(item_delegate_manager_);
}
launcher_controller_->Init();
@@ -2696,9 +2694,9 @@ TEST_F(ChromeLauncherControllerTest, PersistPinned) {
if (!ash::Shell::HasInstance()) {
delete item_delegate_manager_;
} else {
- // Clear already registered LauncherItemDelegate.
- ash::test::LauncherItemDelegateManagerTestAPI test(item_delegate_manager_);
- test.RemoveAllLauncherItemDelegateForTest();
+ // Clear already registered ShelfItemDelegate.
+ ash::test::ShelfItemDelegateManagerTestAPI test(item_delegate_manager_);
+ test.RemoveAllShelfItemDelegateForTest();
}
model_.reset(new ash::ShelfModel);
@@ -2711,9 +2709,8 @@ TEST_F(ChromeLauncherControllerTest, PersistPinned) {
app_icon_loader = new TestAppIconLoaderImpl;
SetAppIconLoader(app_icon_loader);
if (!ash::Shell::HasInstance()) {
- item_delegate_manager_ =
- new ash::LauncherItemDelegateManager(model_.get());
- SetLauncherItemDelegateManager(item_delegate_manager_);
+ item_delegate_manager_ = new ash::ShelfItemDelegateManager(model_.get());
+ SetShelfItemDelegateManager(item_delegate_manager_);
}
launcher_controller_->Init();

Powered by Google App Engine
This is Rietveld 408576698