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

Unified Diff: ui/views/focus/focus_manager_test.cc

Issue 10562025: aura: Fix WidgetFocusChangeListener::OnNativeFocusChange(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge again Created 8 years, 6 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/views/focus/focus_manager_test.cc
diff --git a/ui/views/focus/focus_manager_test.cc b/ui/views/focus/focus_manager_test.cc
index da15b27af8e7a4a7d2227d6ce85764afbfe780fe..df9585ea403e1fe4061e5c63e913bedf3102ef74 100644
--- a/ui/views/focus/focus_manager_test.cc
+++ b/ui/views/focus/focus_manager_test.cc
@@ -14,7 +14,8 @@ namespace views {
FocusManagerTest::FocusManagerTest()
: contents_view_(new View),
- focus_change_listener_(NULL) {
+ focus_change_listener_(NULL),
+ widget_focus_change_listener_(NULL) {
}
FocusManagerTest::~FocusManagerTest() {
@@ -38,6 +39,10 @@ void FocusManagerTest::SetUp() {
void FocusManagerTest::TearDown() {
if (focus_change_listener_)
GetFocusManager()->RemoveFocusChangeListener(focus_change_listener_);
+ if (widget_focus_change_listener_) {
+ WidgetFocusManager::GetInstance()->RemoveFocusChangeListener(
+ widget_focus_change_listener_);
+ }
GetWidget()->Close();
// Flush the message loop to make application verifiers happy.
@@ -72,6 +77,13 @@ void FocusManagerTest::AddFocusChangeListener(FocusChangeListener* listener) {
GetFocusManager()->AddFocusChangeListener(listener);
}
+void FocusManagerTest::AddWidgetFocusChangeListener(
+ WidgetFocusChangeListener* listener) {
+ ASSERT_FALSE(widget_focus_change_listener_);
+ widget_focus_change_listener_ = listener;
+ WidgetFocusManager::GetInstance()->AddFocusChangeListener(listener);
+}
+
#if defined(OS_WIN) && !defined(USE_AURA)
void FocusManagerTest::SimulateActivateWindow() {
SendMessage(GetWidget()->GetNativeWindow(), WM_ACTIVATE, WA_ACTIVE, NULL);
@@ -111,4 +123,23 @@ void TestFocusChangeListener::ClearFocusChanges() {
focus_changes_.clear();
}
+////////////////////////////////////////////////////////////////////////////////
+// TestWidgetFocusChangeListener
+
+TestWidgetFocusChangeListener::TestWidgetFocusChangeListener() {
+}
+
+TestWidgetFocusChangeListener::~TestWidgetFocusChangeListener() {
+}
+
+void TestWidgetFocusChangeListener::ClearFocusChanges() {
+ focus_changes_.clear();
+}
+
+void TestWidgetFocusChangeListener::OnNativeFocusChange(
+ gfx::NativeView focused_before,
+ gfx::NativeView focused_now) {
+ focus_changes_.push_back(NativeViewPair(focused_before, focused_now));
+}
+
} // namespace views

Powered by Google App Engine
This is Rietveld 408576698