Set DragDropTracker's capture window's size to zero, otherwise, it gets

ash/drag_drop/drag_drop_tracker_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 "ash/drag_drop/drag_drop_tracker.h"
 
 #include "ash/shell.h"
 #include "ash/shell_window_ids.h"
 #include "ash/test/ash_test_base.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 14 matching lines @@
   static aura::Window* CreateTestWindow(const gfx::Rect& bounds,
                                         aura::Window* parent) {
     static int window_id = 0;
     return aura::test::CreateTestWindowWithDelegate(
         aura::test::TestWindowDelegate::CreateSelfDestroyingDelegate(),
         window_id++,
         bounds,
         parent);
   }
 
   static aura::Window* GetTarget(aura::RootWindow* root_window,

[mazda, 2012/10/01 19:33:24]

Could you delete this |root_window| too?

                           const gfx::Point& location) {
     scoped_ptr<internal::DragDropTracker> tracker(
-        new internal::DragDropTracker(root_window));
+        new internal::DragDropTracker);
     ui::MouseEvent e(ui::ET_MOUSE_DRAGGED,
                      location,
                      location,
                      ui::EF_NONE);
     aura::Window* target = tracker->GetTarget(e);
     return target;
   }
 
   static ui::MouseEvent* ConvertMouseEvent(aura::RootWindow* root_window,

[mazda, 2012/10/01 19:33:24]

Could you delete this one as well?

                                            aura::Window* target,
                                            const ui::MouseEvent& event) {
     scoped_ptr<internal::DragDropTracker> tracker(
-        new internal::DragDropTracker(root_window));
+        new internal::DragDropTracker);
     ui::MouseEvent* converted = tracker->ConvertMouseEvent(target, event);
     return converted;
   }
 };
 
 // TODO(mazda): Remove this once ash/wm/coordinate_conversion.h supports
 // non-X11 platforms.
 #if defined(USE_X11)
 #define MAYBE_GetTarget GetTarget
 #else
 #define MAYBE_GetTarget DISABLED_GetTarget
 #endif
 
 TEST_F(DragDropTrackerTest, MAYBE_GetTarget) {
   Shell::RootWindowList root_windows = Shell::GetAllRootWindows();
   EXPECT_EQ(2U, root_windows.size());
 
   Shell::GetInstance()->set_active_root_window(root_windows[0]);
 
   scoped_ptr<aura::Window> window0(
       CreateTestWindow(gfx::Rect(0, 0, 100, 100), NULL));
   window0->Show();
 
   Shell::GetInstance()->set_active_root_window(root_windows[1]);
 
   scoped_ptr<aura::Window> window1(
       CreateTestWindow(gfx::Rect(100, 100, 100, 100), NULL));
   window1->Show();
 

[mazda, 2012/10/01 19:33:24]

I guess the active root window needs to be set to pass the test.

Shell::GetInstance()->set_active_root_window(root_windows[0]);

   // Start tracking from the RootWindow0 and check the point on RootWindow0 that
   // |window0| covers.
   EXPECT_EQ(window0.get(), GetTarget(root_windows[0], gfx::Point(50, 50)));
 
   // Start tracking from the RootWindow0 and check the point on RootWindow0 that
   // neither |window0| nor |window1| covers.
   EXPECT_NE(window0.get(), GetTarget(root_windows[0], gfx::Point(150, 150)));
   EXPECT_NE(window1.get(), GetTarget(root_windows[0], gfx::Point(150, 150)));
 
   // Start tracking from the RootWindow0 and check the point on RootWindow1 that
   // |window1| covers.
   EXPECT_EQ(window1.get(), GetTarget(root_windows[0], gfx::Point(350, 150)));
 
   // Start tracking from the RootWindow0 and check the point on RootWindow1 that
   // neither |window0| nor |window1| covers.
   EXPECT_NE(window0.get(), GetTarget(root_windows[0], gfx::Point(50, 250)));
   EXPECT_NE(window1.get(), GetTarget(root_windows[0], gfx::Point(50, 250)));
 

[mazda, 2012/10/01 19:33:24]

Here too.

Shell::GetInstance()->set_active_root_window(root_windows[1]);

   // Start tracking from the RootWindow1 and check the point on RootWindow0 that
   // |window0| covers.
   EXPECT_EQ(window0.get(), GetTarget(root_windows[1], gfx::Point(-150, 50)));
 
   // Start tracking from the RootWindow1 and check the point on RootWindow0 that
   // neither |window0| nor |window1| covers.
   EXPECT_NE(window0.get(), GetTarget(root_windows[1], gfx::Point(150, -50)));
   EXPECT_NE(window1.get(), GetTarget(root_windows[1], gfx::Point(150, -50)));
 
   // Start tracking from the RootWindow1 and check the point on RootWindow1 that
@@ skipping 20 matching lines @@
 
   Shell::GetInstance()->set_active_root_window(root_windows[0]);
   scoped_ptr<aura::Window> window0(
       CreateTestWindow(gfx::Rect(0, 0, 100, 100), NULL));
   window0->Show();
 
   Shell::GetInstance()->set_active_root_window(root_windows[1]);
   scoped_ptr<aura::Window> window1(
       CreateTestWindow(gfx::Rect(100, 100, 100, 100), NULL));
   window1->Show();
 

[mazda, 2012/10/01 19:33:24]

Shell::GetInstance()->set_active_root_window(root_windows[0]);

   // Start tracking from the RootWindow0 and converts the mouse event into
   // |window0|'s coodinates.
   ui::MouseEvent original00(ui::ET_MOUSE_DRAGGED,
                             gfx::Point(50, 50),
                             gfx::Point(50, 50),
                             ui::EF_NONE);
   scoped_ptr<ui::MouseEvent> converted00(ConvertMouseEvent(root_windows[0],
                                                            window0.get(),
                                                            original00));
   EXPECT_EQ(original00.type(), converted00->type());
   EXPECT_EQ("50,50", converted00->location().ToString());
   EXPECT_EQ("50,50", converted00->root_location().ToString());
   EXPECT_EQ(original00.flags(), converted00->flags());
 
   // Start tracking from the RootWindow0 and converts the mouse event into
   // |window1|'s coodinates.
   ui::MouseEvent original01(ui::ET_MOUSE_DRAGGED,
                             gfx::Point(350, 150),
                             gfx::Point(350, 150),
                             ui::EF_NONE);
   scoped_ptr<ui::MouseEvent> converted01(ConvertMouseEvent(root_windows[0],
                                                            window1.get(),
                                                            original01));
   EXPECT_EQ(original01.type(), converted01->type());
   EXPECT_EQ("50,50", converted01->location().ToString());
   EXPECT_EQ("150,150", converted01->root_location().ToString());
   EXPECT_EQ(original01.flags(), converted01->flags());
 

[mazda, 2012/10/01 19:33:24]

Shell::GetInstance()->set_active_root_window(root_windows[1]);

   // Start tracking from the RootWindow1 and converts the mouse event into
   // |window0|'s coodinates.
   ui::MouseEvent original10(ui::ET_MOUSE_DRAGGED,
                             gfx::Point(-150, 50),
                             gfx::Point(-150, 50),
                             ui::EF_NONE);
   scoped_ptr<ui::MouseEvent> converted10(ConvertMouseEvent(root_windows[1],
                                                            window0.get(),
                                                            original10));
   EXPECT_EQ(original10.type(), converted10->type());
@@ skipping 11 matching lines @@
                                                            window1.get(),
                                                            original11));
   EXPECT_EQ(original11.type(), converted11->type());
   EXPECT_EQ("50,50", converted11->location().ToString());
   EXPECT_EQ("150,150", converted11->root_location().ToString());
   EXPECT_EQ(original11.flags(), converted11->flags());
 }
 
 }  // namespace test
 }  // namespace aura