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

views/window/non_client_view.cc

Index: views/window/non_client_view.cc
===================================================================
--- views/window/non_client_view.cc	(revision 74231)
+++ views/window/non_client_view.cc	(working copy)
@@ -44,8 +44,8 @@
   if (frame_view_.get())
     RemoveChildView(frame_view_.get());
   frame_view_.reset(frame_view);
-  if (GetParent())
-    AddChildView(kFrameViewIndex, frame_view_.get());
+  if (parent())
+    AddChildViewAt(frame_view_.get(), kFrameViewIndex);
 }
 
 bool NonClientView::CanClose() {
@@ -153,8 +153,8 @@
   // are subsequently resized all the parent-child relationships are
   // established.
   if (is_add && GetWidget() && child == this) {
-    AddChildView(kFrameViewIndex, frame_view_.get());
-    AddChildView(kClientViewIndex, client_view_);
+    AddChildViewAt(frame_view_.get(), kFrameViewIndex);
+    AddChildViewAt(client_view_, kClientViewIndex);
   }
 }