views: Get rid of View::SetParent() method.

views/view.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 "views/view.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 108 matching lines @@
       context_menu_controller_(NULL),
       drag_controller_(NULL) {
 }
 
 View::~View() {
   if (parent_)
     parent_->RemoveChildView(this);
 
   int c = static_cast<int>(children_.size());
   while (--c >= 0) {
-    children_[c]->SetParent(NULL);
+    children_[c]->parent_ = NULL;
     if (children_[c]->IsParentOwned())
       delete children_[c];
   }
 
 #if defined(OS_WIN)
   if (native_view_accessibility_win_.get())
     native_view_accessibility_win_->set_view(NULL);
 #endif
 }
 
@@ skipping 17 matching lines @@
 
   // Remove the view from its current parent if any.
   if (view->parent())
     view->parent()->RemoveChildView(view);
 
   // Sets the prev/next focus views.
   InitFocusSiblings(view, index);
 
   // Let's insert the view.
   children_.insert(children_.begin() + index, view);
-  view->SetParent(this);
+  view->parent_ = this;
 
   for (View* p = this; p; p = p->parent())
     p->ViewHierarchyChangedImpl(false, true, this, view);
 
   view->PropagateAddNotifications(this, view);
   UpdateTooltip();
   if (GetWidget())
     RegisterChildrenForVisibleBoundsNotification(view);
 
   if (layout_manager_.get())
@@ skipping 1158 matching lines @@
       if (prev_focusable)
         prev_focusable->next_focusable_view_ = next_focusable;
       if (next_focusable)
         next_focusable->previous_focusable_view_ = prev_focusable;
     }
 
     if (GetWidget())
       UnregisterChildrenForVisibleBoundsNotification(view);
     view->ResetTexture();
     view->PropagateRemoveNotifications(this);
-    view->SetParent(NULL);
+    view->parent_ = NULL;
 
     if (delete_removed_view && view->IsParentOwned())
       view_to_be_deleted.reset(view);
 
     children_.erase(i);
   }
 
   if (update_tool_tip)
     UpdateTooltip();
 
   if (layout_manager_.get())
     layout_manager_->ViewRemoved(this, view);
 }
 
-void View::SetParent(View* parent) {
-  if (parent != parent_)
-    parent_ = parent;
-}
-
 void View::PropagateRemoveNotifications(View* parent) {
   for (int i = 0, count = child_count(); i < count; ++i)
     GetChildViewAt(i)->PropagateRemoveNotifications(parent);
 
   for (View* v = this; v; v = v->parent())
     v->ViewHierarchyChangedImpl(true, false, parent, this);
 }
 
 void View::PropagateAddNotifications(View* parent, View* child) {
   for (int i = 0, count = child_count(); i < count; ++i)
@@ skipping 499 matching lines @@
     result.append(GetChildViewAt(i)->PrintViewGraph(false));
 
   if (first)
     result.append("}\n");
 
   return result;
 }
 #endif
 
 }  // namespace views