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

Side by Side Diff: services/window_manager/focus_controller_unittest.cc

Issue 988693005: Chromium roll (https://codereview.chromium.org/976353002) (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: fixed bad android build patch Created 5 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "services/window_manager/focus_controller.h" 5 #include "services/window_manager/focus_controller.h"
6 6
7 #include "mojo/converters/geometry/geometry_type_converters.h" 7 #include "mojo/converters/geometry/geometry_type_converters.h"
8 #include "services/window_manager/basic_focus_rules.h" 8 #include "services/window_manager/basic_focus_rules.h"
9 #include "services/window_manager/capture_controller.h" 9 #include "services/window_manager/capture_controller.h"
10 #include "services/window_manager/focus_controller_observer.h" 10 #include "services/window_manager/focus_controller_observer.h"
(...skipping 1053 matching lines...) Expand 10 before | Expand all | Expand 10 after
1064 FocusControllerRemovalTest(bool parent) 1064 FocusControllerRemovalTest(bool parent)
1065 : FocusControllerImplicitTestBase(parent), 1065 : FocusControllerImplicitTestBase(parent),
1066 window_to_destroy_(nullptr) {} 1066 window_to_destroy_(nullptr) {}
1067 1067
1068 // Overridden from FocusControllerImplicitTestBase: 1068 // Overridden from FocusControllerImplicitTestBase:
1069 void ChangeViewDisposition(View* view) override { 1069 void ChangeViewDisposition(View* view) override {
1070 View* disposition_view = GetDispositionView(view); 1070 View* disposition_view = GetDispositionView(view);
1071 disposition_view->parent()->RemoveChild(disposition_view); 1071 disposition_view->parent()->RemoveChild(disposition_view);
1072 window_to_destroy_ = disposition_view; 1072 window_to_destroy_ = disposition_view;
1073 } 1073 }
1074 virtual void TearDown() override { 1074 void TearDown() override {
1075 if (window_to_destroy_) 1075 if (window_to_destroy_)
1076 window_to_destroy_->Destroy(); 1076 window_to_destroy_->Destroy();
1077 1077
1078 FocusControllerImplicitTestBase::TearDown(); 1078 FocusControllerImplicitTestBase::TearDown();
1079 } 1079 }
1080 1080
1081 private: 1081 private:
1082 View* window_to_destroy_; 1082 View* window_to_destroy_;
1083 1083
1084 DISALLOW_COPY_AND_ASSIGN(FocusControllerRemovalTest); 1084 DISALLOW_COPY_AND_ASSIGN(FocusControllerRemovalTest);
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
1185 1185
1186 // See description above DontPassDestroyedView() for details. 1186 // See description above DontPassDestroyedView() for details.
1187 FOCUS_CONTROLLER_TEST(FocusControllerApiTest, DontPassDestroyedView); 1187 FOCUS_CONTROLLER_TEST(FocusControllerApiTest, DontPassDestroyedView);
1188 1188
1189 // TODO(erg): Add the TextInputClient tests here. 1189 // TODO(erg): Add the TextInputClient tests here.
1190 1190
1191 // If a mouse event was handled, it should not activate a view. 1191 // If a mouse event was handled, it should not activate a view.
1192 FOCUS_CONTROLLER_TEST(FocusControllerMouseEventTest, IgnoreHandledEvent); 1192 FOCUS_CONTROLLER_TEST(FocusControllerMouseEventTest, IgnoreHandledEvent);
1193 1193
1194 } // namespace window_manager 1194 } // namespace window_manager
OLDNEW
« no previous file with comments | « services/window_manager/focus_controller.h ('k') | services/window_manager/window_manager_apptest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698