Do not send mouse-move when target is destroyed due to synthesized mouse-exit in RootWindow

ui/aura/root_window_unittest.cc

 // Copyright (c) 2012 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 "ui/aura/root_window.h"
 
 #include <vector>
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/aura/client/event_client.h"
@@ skipping 1029 matching lines @@
   }
 
   // We expect two tap down events. One from the repost and the other one from
   // the scroll sequence posted above.
   EXPECT_EQ(tap_down_count, 2);
 
   EXPECT_EQ(kExpectedTargetEvents,
             EventTypesToString(repost_event_recorder->events()));
 }
 
+class OnMouseExitDeletingEventFilter : public EventFilterRecorder {
+ public:
+  OnMouseExitDeletingEventFilter() : window_to_delete_(NULL) {}
+  virtual ~OnMouseExitDeletingEventFilter() {}
+
+  void set_window_to_delete(Window* window_to_delete) {
+    window_to_delete_ = window_to_delete;
+  }
+
+ private:
+  // Overridden from ui::EventHandler:
+  virtual void OnMouseEvent(ui::MouseEvent* event) OVERRIDE {
+    EventFilterRecorder::OnMouseEvent(event);
+    if (window_to_delete_) {
+      delete window_to_delete_;
+      window_to_delete_ = NULL;
+    }
+  }
+
+  Window* window_to_delete_;
+
+  DISALLOW_COPY_AND_ASSIGN(OnMouseExitDeletingEventFilter);
+};
+
+// Tests that RootWindow drops mouse-moved event that is supposed to be sent to
+// a child, but the child is destroyed because of the synthesized mouse-exit
+// event generated on the previous mouse_moved_handler_.
+TEST_F(RootWindowTest, DeleteWindowDuringMouseMovedDispatch) {
+  // Create window 1 and set its event filter.
+  scoped_ptr<Window> w1(CreateNormalWindow(1, root_window(), NULL));
+  OnMouseExitDeletingEventFilter* w1_filter =

[sky, 2013/10/09 13:12:44]

Does this leak?

[mohsen, 2013/10/09 15:12:31]

SetEventFilter will pass the ownership of event filter to the window. Actually,
I wanted to add a note for it, but I forgot. Added now.

+      new OnMouseExitDeletingEventFilter();
+  w1->SetEventFilter(w1_filter);
+  w1->SetBounds(gfx::Rect(20, 20, 60, 60));
+  EXPECT_EQ(NULL, root_window()->mouse_moved_handler());
+
+  test::EventGenerator generator(root_window(), w1.get());
+
+  // Move mouse over window 1 to set it as the |mouse_moved_handler_| for the
+  // root window.
+  generator.MoveMouseTo(51, 51);
+  EXPECT_EQ(w1.get(), root_window()->mouse_moved_handler());
+
+  // Create window 2 under the mouse cursor and stack it above window 1.
+  Window* w2 = CreateNormalWindow(2, root_window(), NULL);
+  w2->SetBounds(gfx::Rect(30, 30, 40, 40));
+  root_window()->StackChildAbove(w2, w1.get());
+
+  // Set window 2 as the window that is to be deleted when a mouse-exited event
+  // happens on window 1.
+  w1_filter->set_window_to_delete(w2);
+
+  // Move mosue over window 2. This should generate a mouse-exited event for
+  // window 1 resulting in deletion of window 2. The original mouse-moved event
+  // that was targeted to window 2 should be dropped since window 2 is
+  // destroyed. This test passes if no crash happens.
+  generator.MoveMouseTo(52, 52);
+  EXPECT_EQ(NULL, root_window()->mouse_moved_handler());
+
+  // Check events received by window 1.
+  EXPECT_EQ("MOUSE_ENTERED MOUSE_MOVED MOUSE_EXITED",
+            EventTypesToString(w1_filter->events()));
+}
+
 }  // namespace aura