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

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

Issue 71653003: ash: Rename LauncherModel to ShelfModel. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: chrome changes 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 ef63b769a42d936d665f731480c007837b2f0037..9e1e164950118fdbaa5585e10d8733a87e7a8e5c 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc
@@ -10,7 +10,7 @@
#include "ash/ash_switches.h"
#include "ash/launcher/launcher_item_delegate_manager.h"
-#include "ash/launcher/launcher_model.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"
@@ -272,7 +272,7 @@ class ChromeLauncherControllerTest : public BrowserWithTestWindowTest {
virtual void SetUp() OVERRIDE {
BrowserWithTestWindowTest::SetUp();
- model_.reset(new ash::LauncherModel);
+ model_.reset(new ash::ShelfModel);
model_observer_.reset(new TestShelfModelObserver);
model_->AddObserver(model_observer_.get());
@@ -592,7 +592,7 @@ class ChromeLauncherControllerTest : public BrowserWithTestWindowTest {
scoped_refptr<Extension> extension8_;
scoped_ptr<ChromeLauncherController> launcher_controller_;
scoped_ptr<TestShelfModelObserver> model_observer_;
- scoped_ptr<ash::LauncherModel> model_;
+ scoped_ptr<ash::ShelfModel> model_;
// |item_delegate_manager_| owns |test_controller_|.
LauncherItemController* test_controller_;
@@ -1598,7 +1598,7 @@ TEST_F(ChromeLauncherControllerTest, RestoreDefaultAndLockedAppsResyncOrder) {
// Going back to a status where there is no requirement for app 2 to be pinned
// should convert it back to locked but not pinned and state. The position
- // is determined by the |LauncherModel|'s weight system. Since at the moment
+ // is determined by the |ShelfModel|'s weight system. Since at the moment
// the weight of a running app has the same as a shortcut, it will remain
// where it is. Surely note ideal, but since the sync process can shift around
// locations - as well as many other edge cases - this gets nearly impossible
@@ -1655,7 +1655,7 @@ TEST_F(ChromeLauncherControllerTest,
// Going back to a status where there is no requirement for app 2 to be pinned
// should convert it back to running V2 app. Since the position is determined
- // by the |LauncherModel|'s weight system, it will be after last pinned item.
+ // by the |ShelfModel|'s weight system, it will be after last pinned item.
base::ListValue policy_value2;
InsertPrefValue(&policy_value2, 0, extension3_->id());
InsertPrefValue(&policy_value2, 1, extension1_->id());
@@ -2517,7 +2517,7 @@ TEST_F(ChromeLauncherControllerTest, PersistLauncherItemPositions) {
ash::test::LauncherItemDelegateManagerTestAPI test(item_delegate_manager_);
test.RemoveAllLauncherItemDelegateForTest();
}
- model_.reset(new ash::LauncherModel);
+ model_.reset(new ash::ShelfModel);
AddAppListLauncherItem();
launcher_controller_.reset(
@@ -2573,7 +2573,7 @@ TEST_F(ChromeLauncherControllerTest, PersistPinned) {
ash::test::LauncherItemDelegateManagerTestAPI test(item_delegate_manager_);
test.RemoveAllLauncherItemDelegateForTest();
}
- model_.reset(new ash::LauncherModel);
+ model_.reset(new ash::ShelfModel);
AddAppListLauncherItem();
launcher_controller_.reset(

Powered by Google App Engine
This is Rietveld 408576698