Avoid conversion of index to BookmarkNode pointer unnacessarily.

chrome/browser/ui/cocoa/bookmarks/bookmark_editor_base_controller_unittest.mm

 // Copyright (c) 2012 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.
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_editor_controller.h"
@@ skipping 187 matching lines @@
   [controller_ createNewFolders];
   [controller_ cancel:nil];
 }
 
 TEST_F(BookmarkEditorBaseControllerTest, SelectedFolderDeleted) {
   BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile());
   [controller_ selectTestNodeInBrowser:folder_b_3_];
   EXPECT_EQ(folder_b_3_, [controller_ selectedNode]);
 
   // Delete the selected node, and verify it's no longer selected:
-  model->Remove(folder_b_, 3);
+  model->Remove(folder_b_, folder_b_->GetChild(3));
   EXPECT_NE(folder_b_3_, [controller_ selectedNode]);
 
   [controller_ cancel:nil];
 }
 
 TEST_F(BookmarkEditorBaseControllerTest, SelectedFoldersParentDeleted) {
   BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile());
   const BookmarkNode* root = model->bookmark_bar_node();
   [controller_ selectTestNodeInBrowser:folder_b_3_];
   EXPECT_EQ(folder_b_3_, [controller_ selectedNode]);
 
   // Delete the selected node's parent, and verify it's no longer selected:
-  model->Remove(root, 1);
+  model->Remove(root, root->GetChild(1));
   EXPECT_NE(folder_b_3_, [controller_ selectedNode]);
 
   [controller_ cancel:nil];
 }
 
 TEST_F(BookmarkEditorBaseControllerTest, FolderAdded) {
   BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile());
   const BookmarkNode* root = model->bookmark_bar_node();
 
   // Add a folder node to the model, and verify it can be selected in the tree:
@@ skipping 38 matching lines @@
       reinterpret_cast<const BookmarkNode*>(&children);
   BookmarkFolderInfo* info =
     [BookmarkFolderInfo bookmarkFolderInfoWithFolderName:@"name"
                                               folderNode:fakeNode
                                                 children:children];
   EXPECT_TRUE(info);
   EXPECT_EQ([info folderName], @"name");
   EXPECT_EQ([info children], children);
   EXPECT_EQ([info folderNode], fakeNode);
 }