Cleanup adding BoundsInWidget to View to convert callers...

chrome/views/view.h

 // Copyright (c) 2006-2008 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.
 
 #ifndef CHROME_VIEWS_VIEW_H_
 #define CHROME_VIEWS_VIEW_H_
 
 #include <atlbase.h>     
 #include <atlapp.h>      
 #include <atlmisc.h>     
@@ skipping 637 matching lines @@
   // ValidateFloatingViewForID if that view has not yet been attached.
   virtual View* RetrieveFloatingViewForID(int id);
 
   // Restores the focus to the previously selected floating view.
   virtual void RestoreFloatingViewFocus();
 
   // Goes up the parent hierarchy of this view and returns the first floating
   // view found.  Returns NULL if none were found.
   View* RetrieveFloatingViewParent();
 
+  // Convert a point from the coordinate system of the current View to that of 
+  // the current widget bounds on the screen.
+  gfx::Rect BoundsInWidget();

[Ben Goodger (Google), 2008/12/08 20:40:08]

Yeah I agree with Peter now after some reflection, I think this would be better
as

gfx::Point PositionInWidget();

+
   // Utility functions
 
   // Note that the utility coordinate conversions functions always operate on
   // the mirrored position of the child Views if the parent View uses a
   // right-to-left UI layout.
 
   // Convert a point from source coordinate system to dst coordinate system.
   //
   // source is a parent or a child of dst, directly or transitively.
   // If source and dst are not in the same View hierarchy, the result is
@@ skipping 635 matching lines @@
   // right-to-left locales for this View.
   bool flip_canvas_on_paint_for_rtl_ui_;
 
   DISALLOW_COPY_AND_ASSIGN(View);
 };
 
 }  // namespace views
 
 #endif  // CHROME_VIEWS_VIEW_H_