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

Unified Diff: ui/app_list/app_list_model_unittest.cc

Issue 63283003: Move AppListModel::Users to AppListViewDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Mac fixes 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
« no previous file with comments | « ui/app_list/app_list_model_observer.h ('k') | ui/app_list/app_list_view_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/app_list/app_list_model_unittest.cc
diff --git a/ui/app_list/app_list_model_unittest.cc b/ui/app_list/app_list_model_unittest.cc
index 12486a6d5e2f60d1d0e6e585116e4a87f9b7b442..944b71a386ca084aab0aca84b4399d6d0c7f5488 100644
--- a/ui/app_list/app_list_model_unittest.cc
+++ b/ui/app_list/app_list_model_unittest.cc
@@ -23,7 +23,6 @@ class TestObserver : public AppListModelObserver,
public:
TestObserver()
: status_changed_count_(0),
- users_changed_count_(0),
signin_changed_count_(0),
items_added_(0),
items_removed_(0),
@@ -37,10 +36,6 @@ class TestObserver : public AppListModelObserver,
++status_changed_count_;
}
- virtual void OnAppListModelUsersChanged() OVERRIDE {
- ++users_changed_count_;
- }
-
virtual void OnAppListModelSigninStatusChanged() OVERRIDE {
++signin_changed_count_;
}
@@ -62,7 +57,6 @@ class TestObserver : public AppListModelObserver,
}
int status_changed_count() const { return status_changed_count_; }
- int users_changed_count() const { return users_changed_count_; }
int signin_changed_count() const { return signin_changed_count_; }
size_t items_added() { return items_added_; }
size_t items_removed() { return items_removed_; }
@@ -70,7 +64,6 @@ class TestObserver : public AppListModelObserver,
void ResetCounts() {
status_changed_count_ = 0;
- users_changed_count_ = 0;
signin_changed_count_ = 0;
items_added_ = 0;
items_removed_ = 0;
@@ -79,7 +72,6 @@ class TestObserver : public AppListModelObserver,
private:
int status_changed_count_;
- int users_changed_count_;
int signin_changed_count_;
size_t items_added_;
size_t items_removed_;
@@ -130,17 +122,6 @@ TEST_F(AppListModelTest, SetStatus) {
EXPECT_EQ(2, observer_.status_changed_count());
}
-TEST_F(AppListModelTest, SetUsers) {
- EXPECT_EQ(0u, model_.users().size());
- AppListModel::Users users;
- users.push_back(AppListModel::User());
- users[0].name = UTF8ToUTF16("test");
- model_.SetUsers(users);
- EXPECT_EQ(1, observer_.users_changed_count());
- ASSERT_EQ(1u, model_.users().size());
- EXPECT_EQ(UTF8ToUTF16("test"), model_.users()[0].name);
-}
-
TEST_F(AppListModelTest, SetSignedIn) {
EXPECT_TRUE(model_.signed_in());
model_.SetSignedIn(false);
« no previous file with comments | « ui/app_list/app_list_model_observer.h ('k') | ui/app_list/app_list_view_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698