Simplify ScopedVector<scoped_ptr<T>> to ScopedVector<T>.

tools/gn/loader.cc

Index: tools/gn/loader.cc
diff --git a/tools/gn/loader.cc b/tools/gn/loader.cc
index be5f5cc62ab4f0c46909c36bf48be9bffedf67fc..628a08e719745f63176c8030c2aa0257c5b0cd17 100644
--- a/tools/gn/loader.cc
+++ b/tools/gn/loader.cc
@@ -265,8 +265,10 @@ void LoaderImpl::BackgroundLoadFile(const Settings* settings,
     g_scheduler->FailWithError(err);
 
   // Pass all of the items that were defined off to the builder.
-  for (const auto& item : collected_items)
-    settings->build_settings()->ItemDefined(item->Pass());
+  for (auto& item : collected_items) {
+    settings->build_settings()->ItemDefined(make_scoped_ptr(item));

[dcheng, 2014/10/15 20:26:09]

This is a bit awkward, but I don't know of a better solution without
std::unique_ptr. Maybe cc's ScopedPtrVector will help here.

+    item = nullptr;
+  }
 
   trace.Done();