Remove ScopedVector from NavigationController.

components/sessions/content/content_serialized_navigation_builder.cc

Index: components/sessions/content/content_serialized_navigation_builder.cc
diff --git a/components/sessions/content/content_serialized_navigation_builder.cc b/components/sessions/content/content_serialized_navigation_builder.cc
index d371994423f90f8b1ee3914acbf10ee4d1db4931..e98c2df6e8eff79af747cf14e1be9cff364174fe 100644
--- a/components/sessions/content/content_serialized_navigation_builder.cc
+++ b/components/sessions/content/content_serialized_navigation_builder.cc
@@ -85,19 +85,17 @@ ContentSerializedNavigationBuilder::ToNavigationEntry(
 }
 
 // static
-ScopedVector<content::NavigationEntry>
+std::vector<scoped_ptr<content::NavigationEntry>>
 ContentSerializedNavigationBuilder::ToNavigationEntries(
     const std::vector<SerializedNavigationEntry>& navigations,
     content::BrowserContext* browser_context) {
   int page_id = 0;
-  ScopedVector<content::NavigationEntry> entries;
-  for (std::vector<SerializedNavigationEntry>::const_iterator
-       it = navigations.begin(); it != navigations.end(); ++it) {
-    entries.push_back(
-        ToNavigationEntry(&(*it), page_id, browser_context).release());
+  std::vector<scoped_ptr<content::NavigationEntry>> entries;

[ncarter (slow), 2015/11/12 18:10:40]

Might as well:

entries.reserve(navigations.size());

[Avi (use Gerrit), 2015/11/12 19:13:04]

Done.

+  for (const auto& navigation : navigations) {
+    entries.push_back(ToNavigationEntry(&navigation, page_id, browser_context));
     ++page_id;
   }
-  return entries.Pass();
+  return entries;
 }
 
 }  // namespace sessions