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

Unified Diff: services/ui/ws/window_finder_unittest.cc

Issue 2118383002: mus: Disregard windows that explicitly set can_accept_events to be false when sending events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Check if can_accept_events setting has changed; change names Created 4 years, 5 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: services/ui/ws/window_finder_unittest.cc
diff --git a/services/ui/ws/window_finder_unittest.cc b/services/ui/ws/window_finder_unittest.cc
index af96b0694a1d641849437e6f8fba387c0577466a..3cb3103a80da9488139ec059d1b858c8ee35eaee 100644
--- a/services/ui/ws/window_finder_unittest.cc
+++ b/services/ui/ws/window_finder_unittest.cc
@@ -41,6 +41,11 @@ TEST(WindowFinderTest, FindDeepestVisibleWindow) {
EXPECT_EQ(&child1, FindDeepestVisibleWindowForEvents(&root, &local_point));
EXPECT_EQ(gfx::Point(3, 4), local_point);
+ local_point.SetPoint(13, 14);
+ child1.set_can_accept_events(false);
+ EXPECT_EQ(&root, FindDeepestVisibleWindowForEvents(&root, &local_point));
+ EXPECT_EQ(gfx::Point(13, 14), local_point);
+
child2.set_extended_hit_test_region(gfx::Insets(10, 10, 10, 10));
local_point.SetPoint(13, 14);
EXPECT_EQ(&child2, FindDeepestVisibleWindowForEvents(&root, &local_point));

Powered by Google App Engine
This is Rietveld 408576698