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

Unified Diff: mojo/services/view_manager/focus_controller_unittest.cc

Issue 1092033003: Pulls display related changes from mojo to chrome (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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: mojo/services/view_manager/focus_controller_unittest.cc
diff --git a/mojo/services/view_manager/focus_controller_unittest.cc b/mojo/services/view_manager/focus_controller_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c25694c15705362f8650da6be417f2ad2977a6a3
--- /dev/null
+++ b/mojo/services/view_manager/focus_controller_unittest.cc
@@ -0,0 +1,106 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "mojo/services/view_manager/focus_controller.h"
+
+#include "mojo/services/view_manager/focus_controller_delegate.h"
+#include "mojo/services/view_manager/server_view.h"
+#include "mojo/services/view_manager/test_server_view_delegate.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace view_manager {
+namespace {
+
+class TestFocusControllerDelegate : public FocusControllerDelegate {
+ public:
+ TestFocusControllerDelegate()
+ : change_count_(0u),
+ old_focused_view_(nullptr),
+ new_focused_view_(nullptr) {}
+
+ void ClearAll() {
+ change_count_ = 0u;
+ old_focused_view_ = nullptr;
+ new_focused_view_ = nullptr;
+ }
+ size_t change_count() const { return change_count_; }
+ ServerView* old_focused_view() { return old_focused_view_; }
+ ServerView* new_focused_view() { return new_focused_view_; }
+
+ private:
+ // FocusControllerDelegate:
+ void OnFocusChanged(ServerView* old_focused_view,
+ ServerView* new_focused_view) override {
+ change_count_++;
+ old_focused_view_ = old_focused_view;
+ new_focused_view_ = new_focused_view;
+ }
+
+ size_t change_count_;
+ ServerView* old_focused_view_;
+ ServerView* new_focused_view_;
+
+ DISALLOW_COPY_AND_ASSIGN(TestFocusControllerDelegate);
+};
+
+} // namespace
+
+TEST(FocusControllerTest, Basic) {
+ TestServerViewDelegate server_view_delegate;
+ ServerView root(&server_view_delegate, ViewId());
+ root.SetVisible(true);
+ ServerView child(&server_view_delegate, ViewId());
+ child.SetVisible(true);
+ root.Add(&child);
+ ServerView child_child(&server_view_delegate, ViewId());
+ child_child.SetVisible(true);
+ child.Add(&child_child);
+
+ TestFocusControllerDelegate focus_delegate;
+ FocusController focus_controller(&focus_delegate, &root);
+
+ focus_controller.SetFocusedView(&child_child);
+ EXPECT_EQ(0u, focus_delegate.change_count());
+
+ // Remove the ancestor of the focused view, focus should go to the |root|.
+ root.Remove(&child);
+ EXPECT_EQ(1u, focus_delegate.change_count());
+ EXPECT_EQ(&root, focus_delegate.new_focused_view());
+ EXPECT_EQ(&child_child, focus_delegate.old_focused_view());
+ focus_delegate.ClearAll();
+
+ // Make the focused view invisible. Focus is lost in this case (as no one
+ // to give focus to).
+ root.SetVisible(false);
+ EXPECT_EQ(1u, focus_delegate.change_count());
+ EXPECT_EQ(nullptr, focus_delegate.new_focused_view());
+ EXPECT_EQ(&root, focus_delegate.old_focused_view());
+ focus_delegate.ClearAll();
+
+ // Go back to initial state and focus |child_child|.
+ root.SetVisible(true);
+ root.Add(&child);
+ focus_controller.SetFocusedView(&child_child);
+ EXPECT_EQ(0u, focus_delegate.change_count());
+
+ // Hide the focused view, focus should go to parent.
+ child_child.SetVisible(false);
+ EXPECT_EQ(1u, focus_delegate.change_count());
+ EXPECT_EQ(&child, focus_delegate.new_focused_view());
+ EXPECT_EQ(&child_child, focus_delegate.old_focused_view());
+ focus_delegate.ClearAll();
+
+ child_child.SetVisible(true);
+ focus_controller.SetFocusedView(&child_child);
+ EXPECT_EQ(0u, focus_delegate.change_count());
+
+ // Hide the parent of the focused view.
+ child.SetVisible(false);
+ EXPECT_EQ(1u, focus_delegate.change_count());
+ EXPECT_EQ(&root, focus_delegate.new_focused_view());
+ EXPECT_EQ(&child_child, focus_delegate.old_focused_view());
+ focus_delegate.ClearAll();
+}
+
+} // namespace view_manager
« no previous file with comments | « mojo/services/view_manager/focus_controller_delegate.h ('k') | mojo/services/view_manager/gesture_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698