Remove ScopedVector from content/browser/ [1].

content/browser/frame_host/navigation_controller_impl_browsertest.cc

 // Copyright 2014 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 "content/browser/frame_host/navigation_controller_impl.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
 #include <vector>
@@ skipping 5970 matching lines @@
   EXPECT_EQ(1U, root->child_count());
   EXPECT_NE("foo-frame-name", root->child_at(0)->frame_name());
 
   // Add a nested frame with the previously used name.
   EXPECT_TRUE(ExecuteScript(root->child_at(0), kAddNamedFrameScript));
   EXPECT_EQ(1U, root->child_at(0)->child_count());
   EXPECT_EQ("foo-frame-name", root->child_at(0)->child_at(0)->frame_name());
 
   EXPECT_EQ(1U, nav_entry->root_node()->children.size());
 
-  NavigationEntryImpl::TreeNode* tree_node =
-      nav_entry->root_node()->children[0];
-  EXPECT_EQ(1U, tree_node->children.size());
+  EXPECT_EQ(1U, nav_entry->root_node()->children[0]->children.size());
 
-  tree_node = tree_node->children[0];
+  const auto& tree_node = nav_entry->root_node()->children[0]->children[0];
   EXPECT_EQ(0U, tree_node->children.size());
   EXPECT_EQ("foo-frame-name", tree_node->frame_entry->frame_unique_name());
 
   EXPECT_TRUE(ExecuteScript(root->child_at(0), kRemoveFrameScript));
   EXPECT_EQ(0U, root->child_at(0)->child_count());
 }
 
 // This test ensures that the comparison of tree position between a
 // FrameTreeNode and FrameNavigationEntry works correctly for matching
 // first-level frames.
 IN_PROC_BROWSER_TEST_F(NavigationControllerBrowserTest,
                        EnsureFirstLevelFrameNavigationEntriesMatch) {
   WebContentsImpl* web_contents =
       static_cast<WebContentsImpl*>(shell()->web_contents());
   FrameTreeNode* root = web_contents->GetFrameTree()->root();
-  NavigationEntryImpl::TreeNode* tree_node = nullptr;
 
   GURL start_url(embedded_test_server()->GetURL("/title1.html"));
   EXPECT_TRUE(NavigateToURL(shell(), start_url));
   NavigationEntryImpl* nav_entry =
       web_contents->GetController().GetLastCommittedEntry();
 
   // Add, then remove a named frame. It will create a FrameNavigationEntry
   // for the name and leave it around.
   EXPECT_TRUE(ExecuteScript(root, kAddNamedFrameScript));
   EXPECT_EQ(1U, root->child_count());
   EXPECT_EQ(1U, nav_entry->root_node()->children.size());
-  tree_node = nav_entry->root_node()->children[0];
+  auto* tree_node = nav_entry->root_node()->children[0].get();
 
   EXPECT_TRUE(ExecuteScript(root, kRemoveFrameScript));
   EXPECT_EQ(0U, root->child_count());
   EXPECT_EQ(1U, nav_entry->root_node()->children.size());
 
   // Add another frame with the same name as before. The matching logic
   // should consider them the same and result in the FrameNavigationEntry
   // being reused.
   EXPECT_TRUE(ExecuteScript(root, kAddNamedFrameScript));
   EXPECT_EQ(1U, root->child_count());
   EXPECT_EQ(1U, nav_entry->root_node()->children.size());
-  EXPECT_EQ(tree_node, nav_entry->root_node()->children[0]);
+  EXPECT_EQ(tree_node, nav_entry->root_node()->children[0].get());
 
   EXPECT_TRUE(ExecuteScript(root, kRemoveFrameScript));
   EXPECT_EQ(0U, root->child_count());
 }
 
 // Test that navigations classified as SAME_PAGE properly update all the
 // members of FrameNavigationEntry. If not, it is possible to get a mismatch
 // between the origin and URL of a document as seen in
 // https://crbug.com/630103.
 IN_PROC_BROWSER_TEST_F(NavigationControllerBrowserTest,
@@ skipping 745 matching lines @@
 
   // Do a 204 navigation.
   EXPECT_FALSE(NavigateToURL(shell(), kURL204));
 
   entry = controller.GetLastCommittedEntry();
   EXPECT_EQ(kURL, entry->GetURL());
   EXPECT_EQ(1, controller.GetEntryCount());
 }
 
 }  // namespace content