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

Unified Diff: ui/app_list/views/search_box_view_unittest.cc

Issue 185623002: Add Hide and Show to the app list SearchBoxView. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ares_contents_switcher
Patch Set: Created 6 years, 10 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
Index: ui/app_list/views/search_box_view_unittest.cc
diff --git a/ui/app_list/views/search_box_view_unittest.cc b/ui/app_list/views/search_box_view_unittest.cc
index 3d3ca350a2a10aaa28ce724c3e966edffc94f8db..df81850ef2d304745f085a5ca65e5a7cf4e070ad 100644
--- a/ui/app_list/views/search_box_view_unittest.cc
+++ b/ui/app_list/views/search_box_view_unittest.cc
@@ -44,7 +44,7 @@ class KeyPressCounterView : public views::View {
class SearchBoxViewTest : public views::test::WidgetTest,
public SearchBoxViewDelegate {
public:
- SearchBoxViewTest() : query_changed_count_(0) {}
+ SearchBoxViewTest() : query_changed_count_(0), visibility_changed_count_(0) {}
virtual ~SearchBoxViewTest() {}
// Overridden from testing::Test:
@@ -105,6 +105,12 @@ class SearchBoxViewTest : public views::test::WidgetTest,
return result;
}
+ int GetVisibilityChangedCountAndReset() {
+ int result = visibility_changed_count_;
+ visibility_changed_count_ = 0;
+ return result;
+ }
+
private:
// Overridden from SearchBoxViewDelegate:
virtual void QueryChanged(SearchBoxView* sender) OVERRIDE {
@@ -112,12 +118,17 @@ class SearchBoxViewTest : public views::test::WidgetTest,
last_query_ = sender->search_box()->text();
}
+ virtual void SearchBoxVisibilityChanged(SearchBoxView* sender) OVERRIDE {
+ ++visibility_changed_count_;
+ }
+
AppListTestViewDelegate view_delegate_;
views::Widget* widget_;
SearchBoxView* view_;
KeyPressCounterView* counter_view_;
base::string16 last_query_;
int query_changed_count_;
+ int visibility_changed_count_;
DISALLOW_COPY_AND_ASSIGN(SearchBoxViewTest);
};
@@ -159,5 +170,13 @@ TEST_F(SearchBoxViewTest, CancelAutoLaunch) {
EXPECT_EQ(base::TimeDelta(), GetAutoLaunchTimeout());
}
+TEST_F(SearchBoxViewTest, VisibilityChange) {
+ view()->Show();
+ EXPECT_EQ(1, GetVisibilityChangedCountAndReset());
+
+ view()->Hide();
+ EXPECT_EQ(1, GetVisibilityChangedCountAndReset());
+}
+
} // namespace test
} // namespace app_list
« ui/app_list/views/app_list_view.cc ('K') | « ui/app_list/views/search_box_view_delegate.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698