Take advantage of the new Pass() machinery on scoped_ptr{,_malloc}.

media/base/composite_data_source_factory.cc

Index: media/base/composite_data_source_factory.cc
diff --git a/media/base/composite_data_source_factory.cc b/media/base/composite_data_source_factory.cc
index e75150593f2123f38e415f2d42d5051d399056f4..9a5e550a1184c829ede924023c539f443e5b8288 100644
--- a/media/base/composite_data_source_factory.cc
+++ b/media/base/composite_data_source_factory.cc
@@ -26,7 +26,7 @@ class CompositeDataSourceFactory::BuildRequest
   void CallNextFactory();
   void OnBuildDone(PipelineStatus status, DataSource* data_source);
 
-  FactoryList factories_;
+  FactoryList factories_;  // Not owned by this class.
 };
 
 CompositeDataSourceFactory::CompositeDataSourceFactory() {}
@@ -35,21 +35,25 @@ CompositeDataSourceFactory::~CompositeDataSourceFactory() {
   STLDeleteElements(&factories_);
 }
 
-void CompositeDataSourceFactory::AddFactory(DataSourceFactory* factory) {
-  DCHECK(factory);
-  factories_.push_back(factory);
+void CompositeDataSourceFactory::AddFactory(
+    scoped_ptr<DataSourceFactory> factory) {
+  DCHECK(factory.get());
+  factories_.push_back(factory.release());
 }
 
-DataSourceFactory* CompositeDataSourceFactory::Clone() const {
-  CompositeDataSourceFactory* new_factory = new CompositeDataSourceFactory();
+scoped_ptr<DataSourceFactory> CompositeDataSourceFactory::Clone() const {
+  scoped_ptr<CompositeDataSourceFactory> new_factory(
+      new CompositeDataSourceFactory());
 
   for (FactoryList::const_iterator itr = factories_.begin();
        itr != factories_.end();
        ++itr) {
-    new_factory->AddFactory((*itr)->Clone());
+    new_factory->AddFactory((*itr)->Clone().Pass());
   }
 
-  return new_factory;
+  // TODO(fischman): replace the extra scoped_ptr+release() with Pass() when
+  // http://crbug.com/109026 is fixed.
+  return scoped_ptr<DataSourceFactory>(new_factory.release());
 }
 
 bool CompositeDataSourceFactory::AllowRequests() const {