Fix aura::Window::ContainsPointInRoot().

ui/aura/window.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/window.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
@@ skipping 469 matching lines @@
 void Window::RemoveObserver(WindowObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
 bool Window::ContainsPointInRoot(const gfx::Point& point_in_root) {
   Window* root_window = GetRootWindow();
   if (!root_window)
     return false;
   gfx::Point local_point(point_in_root);
   ConvertPointToWindow(root_window, this, &local_point);
-  return GetTargetBounds().Contains(local_point);
+  return gfx::Rect(gfx::Point(),
+                   GetTargetBounds().size()).Contains(local_point);

[oshima, 2012/07/21 13:25:04]

gfx::Rect(GetTargetBounds().size()).Contains(local_point);

can you fix ContainsPoint while we're at it?

[Yusuke Sato, 2012/07/23 16:39:42]

Done.

 }
 
 bool Window::ContainsPoint(const gfx::Point& local_point) {
   gfx::Rect local_bounds(gfx::Point(), bounds().size());
   return local_bounds.Contains(local_point);
 }
 
 bool Window::HitTest(const gfx::Point& local_point) {
   // Expand my bounds for hit testing (override is usually zero but it's
   // probably cheaper to do the math every time than to branch).
@@ skipping 446 matching lines @@
   for (Windows::const_reverse_iterator it = children_.rbegin(),
            rend = children_.rend();
        it != rend; ++it) {
     Window* child = *it;
     child->PrintWindowHierarchy(depth + 1);
   }
 }
 #endif
 
 }  // namespace aura