Rework tree APIs to reflect Google style and more const-correctness....

chrome/browser/ui/views/frame/browser_view_layout.cc

 // Copyright (c) 2011 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 "chrome/browser/ui/views/frame/browser_view_layout.h"
 
 #include "chrome/browser/sidebar/sidebar_manager.h"
 #include "chrome/browser/ui/find_bar/find_bar.h"
 #include "chrome/browser/ui/find_bar/find_bar_controller.h"
 #include "chrome/browser/ui/view_ids.h"
@@ skipping 117 matching lines @@
   return tabstrip_->IsPositionInWindowCaption(tabstrip_point);
 }
 
 int BrowserViewLayout::NonClientHitTest(
     const gfx::Point& point) {
   // Since the TabStrip only renders in some parts of the top of the window,
   // the un-obscured area is considered to be part of the non-client caption
   // area of the window. So we need to treat hit-tests in these regions as
   // hit-tests of the titlebar.
 
-  views::View* parent = browser_view_->GetParent();
+  views::View* parent = browser_view_->parent();
 
   gfx::Point point_in_browser_view_coords(point);
   views::View::ConvertPointToView(
       parent, browser_view_, &point_in_browser_view_coords);
 
   // Determine if the TabStrip exists and is capable of being clicked on. We
   // might be a popup window without a TabStrip.
   if (browser_view_->IsTabStripVisible()) {
     // See if the mouse pointer is within the bounds of the TabStrip.
     gfx::Point point_in_tabstrip_coords(point);
@@ skipping 143 matching lines @@
 int BrowserViewLayout::LayoutTabStrip() {
   if (!browser_view_->IsTabStripVisible()) {
     tabstrip_->SetVisible(false);
     tabstrip_->SetBounds(0, 0, 0, 0);
     return 0;
   }
 
   gfx::Rect tabstrip_bounds(
       browser_view_->frame()->GetBoundsForTabStrip(tabstrip_));
   gfx::Point tabstrip_origin(tabstrip_bounds.origin());
-  views::View::ConvertPointToView(browser_view_->GetParent(), browser_view_,
+  views::View::ConvertPointToView(browser_view_->parent(), browser_view_,
                                   &tabstrip_origin);
   tabstrip_bounds.set_origin(tabstrip_origin);
 
   if (browser_view_->UseVerticalTabs())
     vertical_layout_rect_.Inset(tabstrip_bounds.width(), 0, 0, 0);
 
   tabstrip_->SetVisible(true);
   tabstrip_->SetBoundsRect(tabstrip_bounds);
   return browser_view_->UseVerticalTabs() ?
       tabstrip_bounds.y() : tabstrip_bounds.bottom();
@@ skipping 63 matching lines @@
 // Note that |future_parent_offset| is relative to browser_view_, not to
 // the parent view.
 void BrowserViewLayout::UpdateReservedContentsRect(
     const gfx::Rect& browser_reserved_rect,
     TabContentsContainer* source,
     const gfx::Rect& future_source_bounds,
     const gfx::Point& future_parent_offset) {
   gfx::Point resize_corner_origin(browser_reserved_rect.origin());
   // Convert |resize_corner_origin| from browser_view_ to source's parent
   // coordinates.
-  views::View::ConvertPointToView(browser_view_, source->GetParent(),
+  views::View::ConvertPointToView(browser_view_, source->parent(),
                                   &resize_corner_origin);
   // Create |reserved_rect| in source's parent coordinates.
   gfx::Rect reserved_rect(resize_corner_origin, browser_reserved_rect.size());
   // Apply source's parent future offset to it.
   reserved_rect.Offset(-future_parent_offset.x(), -future_parent_offset.y());
   if (future_source_bounds.Intersects(reserved_rect)) {
     // |source| is not properly positioned yet to use ConvertPointToView,
     // so convert it into |source|'s coordinates manually.
     reserved_rect.Offset(-future_source_bounds.x(), -future_source_bounds.y());
   } else {
@@ skipping 118 matching lines @@
     bottom -= height;
   }
   return bottom;
 }
 
 bool BrowserViewLayout::InfobarVisible() const {
   // NOTE: Can't check if the size IsEmpty() since it's always 0-width.
   return browser()->SupportsWindowFeature(Browser::FEATURE_INFOBAR) &&
       (infobar_container_->GetPreferredSize().height() != 0);
 }