views: Make View::GetVisibleBounds() a const method.

Note: This was a TODO for beng.

BUG=72040
TEST=None

R=ben@chromium.org

Review URL: http://codereview.chromium.org/7083013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87312 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 5 insertions, 7 deletions

diff --git a/views/view.cc b/views/view.cc
index 8fa73c0..fd4749d 100644
--- a/views/view.cc
+++ b/views/view.cc
@@ -291,22 +291,21 @@ gfx::Insets View::GetInsets() const {
   return insets;
 }
 
-gfx::Rect View::GetVisibleBounds() {
+gfx::Rect View::GetVisibleBounds() const {
   if (!IsVisibleInRootView())
     return gfx::Rect();
   gfx::Rect vis_bounds(0, 0, width(), height());
   gfx::Rect ancestor_bounds;
-  View* view = this;
+  const View* view = this;
   int root_x = 0;
   int root_y = 0;
   while (view != NULL && !vis_bounds.IsEmpty()) {
     root_x += view->GetMirroredX();
     root_y += view->y();
     vis_bounds.Offset(view->GetMirroredX(), view->y());
-    View* ancestor = view->parent();
+    const View* ancestor = view->parent();
     if (ancestor != NULL) {
-      ancestor_bounds.SetRect(0, 0, ancestor->width(),
-        ancestor->height());
+      ancestor_bounds.SetRect(0, 0, ancestor->width(), ancestor->height());
       vis_bounds = vis_bounds.Intersect(ancestor_bounds);
     } else if (!view->GetWidget()) {
       // If the view has no Widget, we're not visible. Return an empty rect.
diff --git a/views/view.h b/views/view.h
index 4f6f0e3..2a145ab 100644
--- a/views/view.h
+++ b/views/view.h
@@ -278,8 +278,7 @@ class View : public AcceleratorTarget {
   // function takes into account the mirroring setting for each View and
   // therefore it will return the mirrored version of the visible bounds if
   // need be.
-  // TODO(beng): const.
-  gfx::Rect GetVisibleBounds();
+  gfx::Rect GetVisibleBounds() const;
 
   // Return the bounds of the View in screen coordinate system.
   gfx::Rect GetScreenBounds() const;