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

chrome/browser/ui/views/info_bubble.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/info_bubble.h"
 
 #include <vector>
 
 #include "chrome/browser/ui/window_sizer.h"
 #include "chrome/common/notification_service.h"
@@ skipping 420 matching lines @@
       views::Background::CreateSolidBackground(kBackgroundColor));
 #else
   // Create a view to paint the border and background.
   border_contents_ = CreateBorderContents();
   border_contents_->Init();
   gfx::Rect contents_bounds;
   border_contents_->SizeAndGetBounds(position_relative_to,
       arrow_location, false, contents->GetPreferredSize(),
       &contents_bounds, &window_bounds);
   // This new view must be added before |contents| so it will paint under it.
-  contents_view->AddChildView(0, border_contents_);
+  contents_view->AddChildViewAt(border_contents_, 0);
 
   // |contents_view| has no layout manager, so we have to explicitly position
   // its children.
   border_contents_->SetBoundsRect(
       gfx::Rect(gfx::Point(), window_bounds.size()));
   contents->SetBoundsRect(contents_bounds);
 #endif
   SetBounds(window_bounds);
 
   // Register the Escape accelerator for closing.
@@ skipping 41 matching lines @@
 #endif
   SetBounds(window_bounds);
 }
 
 #if defined(OS_WIN)
 void InfoBubble::OnActivate(UINT action, BOOL minimized, HWND window) {
   // The popup should close when it is deactivated.
   if (action == WA_INACTIVE) {
     Close();
   } else if (action == WA_ACTIVE) {
-    DCHECK_GT(GetRootView()->GetChildViewCount(), 0);
+    DCHECK(GetRootView()->has_children());
     GetRootView()->GetChildViewAt(0)->RequestFocus();
   }
 }
 #elif defined(OS_LINUX)
 void InfoBubble::IsActiveChanged() {
   if (!IsActive())
     Close();
 }
 #endif
 
@@ skipping 44 matching lines @@
     animation_->Hide();
 }
 
 bool InfoBubble::AcceleratorPressed(const views::Accelerator& accelerator) {
   if (!delegate_ || delegate_->CloseOnEscape()) {
     DoClose(true);
     return true;
   }
   return false;
 }