Change View::HitTest to View::HitTestRect

chrome/browser/ui/views/location_bar/content_setting_image_view.cc

Index: chrome/browser/ui/views/location_bar/content_setting_image_view.cc
diff --git a/chrome/browser/ui/views/location_bar/content_setting_image_view.cc b/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
index 0aaf76d8af8e94e232d9e932613cbfb44244f862..be3d111f4488430e63b965b67675cf488892a118 100644
--- a/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
+++ b/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
@@ -165,7 +165,7 @@ bool ContentSettingImageView::OnMousePressed(const views::MouseEvent& event) {
 }
 
 void ContentSettingImageView::OnMouseReleased(const views::MouseEvent& event) {
-  if (!HitTest(event.location()))
+  if (!HitTest(gfx::Rect(event.location(), gfx::Size(0, 0))))

[sky, 2012/08/07 20:11:43]

Based on how many calls of this we have I think its worth adding two:

bool HitTestPoint(const gfx::Point& point) {
  return HitTestRect(gfx::Rect(point, whatever size you decide);
}

The key thing is the first is non-virtual.

virtual bool HitRectRect(...);

[tdanderson, 2012/08/08 23:47:51]

Done.

     return;
 
   TabContents* tab_contents = parent_->GetTabContents();