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

chrome/browser/ui/views/bookmark_menu_controller_views.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/bookmark_menu_controller_views.h"
 
 #include "base/stl_util-inl.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_node_data.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
@@ skipping 370 matching lines @@
   std::set<MenuItemView*> changed_parent_menus;
   for (std::vector<const BookmarkNode*>::const_iterator i = bookmarks.begin();
        i != bookmarks.end(); ++i) {
     NodeToMenuIDMap::iterator node_to_menu = node_to_menu_id_map_.find(*i);
     if (node_to_menu != node_to_menu_id_map_.end()) {
       MenuItemView* menu = GetMenuByID(node_to_menu->second);
       DCHECK(menu);  // If there an entry in node_to_menu_id_map_, there should
                      // be a menu.
       removed_menus->insert(menu);
       changed_parent_menus.insert(menu->GetParentMenuItem());
-      menu->GetParent()->RemoveChildView(menu);
+      menu->parent()->RemoveChildView(menu);
       node_to_menu_id_map_.erase(node_to_menu);
     }
   }
 
   // All the bookmarks in |bookmarks| should have the same parent. It's possible
   // to support different parents, but this would need to prune any nodes whose
   // parent has been removed. As all nodes currently have the same parent, there
   // is the DCHECK.
   DCHECK(changed_parent_menus.size() <= 1);
 
@@ skipping 13 matching lines @@
         break;
       }
     }
     if (ancestor_removed) {
       node_to_menu_id_map_.erase(i++);
     } else {
       ++i;
     }
   }
 }