Valgrind: Clean up leaks in callers to extension_function_test_utils::RunFunction.

chrome/browser/extensions/api/browsing_data/browsing_data_test.cc

 // 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.
 
 #include "chrome/browser/extensions/api/browsing_data/browsing_data_api.h"
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 57 matching lines @@
     // We're not taking ownership of the details object, but storing a copy of
     // it locally.
     called_with_details_.reset(new BrowsingDataRemover::NotificationDetails(
         *content::Details<BrowsingDataRemover::NotificationDetails>(
             details).ptr()));
   }
 
   void RunRemoveBrowsingDataFunctionAndCompareRemovalMask(
       const std::string& key,
       int expected_mask) {
+    scoped_refptr<RemoveBrowsingDataFunction> function =
+        new RemoveBrowsingDataFunction();
     SCOPED_TRACE(key);
     EXPECT_EQ(NULL, RunFunctionAndReturnSingleResult(
-        new RemoveBrowsingDataFunction(),
+        function.get(),
         std::string("[{\"since\": 1}, {\"") + key + "\": true}]",
         browser()));
     EXPECT_EQ(expected_mask, GetRemovalMask());
     EXPECT_EQ(BrowsingDataHelper::UNPROTECTED_WEB, GetOriginSetMask());
   }
 
   void RunRemoveBrowsingDataFunctionAndCompareOriginSetMask(
       const std::string& protectedStr,
       int expected_mask) {
+    scoped_refptr<RemoveBrowsingDataFunction> function =
+        new RemoveBrowsingDataFunction();
     SCOPED_TRACE(protectedStr);
     EXPECT_EQ(NULL, RunFunctionAndReturnSingleResult(
-        new RemoveBrowsingDataFunction(),
+        function.get(),
         "[{\"originType\": " + protectedStr + "}, {\"cookies\": true}]",
         browser()));
     EXPECT_EQ(expected_mask, GetOriginSetMask());
   }
 
  private:
   scoped_ptr<BrowsingDataRemover::NotificationDetails> called_with_details_;
   content::NotificationRegistrar registrar_;
 };
 
 }  // namespace
 
 IN_PROC_BROWSER_TEST_F(ExtensionBrowsingDataTest, OneAtATime) {
   BrowsingDataRemover::set_removing(true);
+  scoped_refptr<RemoveBrowsingDataFunction> function =
+      new RemoveBrowsingDataFunction();
   EXPECT_TRUE(MatchPattern(
-      RunFunctionAndReturnError(
-          new RemoveBrowsingDataFunction(),
-          kRemoveEverythingArguments,
-          browser()),
+      RunFunctionAndReturnError(function,
+                                kRemoveEverythingArguments,
+                                browser()),
       extension_browsing_data_api_constants::kOneAtATimeError));
   BrowsingDataRemover::set_removing(false);
 
   EXPECT_EQ(base::Time(), GetBeginTime());
   EXPECT_EQ(-1, GetRemovalMask());
 }
 
 // Use-after-free, see http://crbug.com/116522
 IN_PROC_BROWSER_TEST_F(ExtensionBrowsingDataTest,
                        DISABLED_RemoveBrowsingDataAll) {
-  EXPECT_EQ(NULL, RunFunctionAndReturnSingleResult(
-      new RemoveBrowsingDataFunction(),
-      kRemoveEverythingArguments,
-      browser()));
+  scoped_refptr<RemoveBrowsingDataFunction> function =
+      new RemoveBrowsingDataFunction();
+  EXPECT_EQ(NULL, RunFunctionAndReturnSingleResult(function.get(),
+                                                   kRemoveEverythingArguments,
+                                                   browser()));
 
   EXPECT_EQ(base::Time::FromDoubleT(1.0), GetBeginTime());
   EXPECT_EQ((BrowsingDataRemover::REMOVE_SITE_DATA |
       BrowsingDataRemover::REMOVE_CACHE |
       BrowsingDataRemover::REMOVE_DOWNLOADS |
       BrowsingDataRemover::REMOVE_FORM_DATA |
       BrowsingDataRemover::REMOVE_HISTORY |
       BrowsingDataRemover::REMOVE_PASSWORDS) &
       // We can't remove plugin data inside a test profile.
       ~BrowsingDataRemover::REMOVE_PLUGIN_DATA, GetRemovalMask());
@@ skipping 46 matching lines @@
   RunRemoveBrowsingDataFunctionAndCompareRemovalMask(
       "localStorage", BrowsingDataRemover::REMOVE_LOCAL_STORAGE);
   RunRemoveBrowsingDataFunctionAndCompareRemovalMask(
       "serverBoundCerts", BrowsingDataRemover::REMOVE_SERVER_BOUND_CERTS);
   RunRemoveBrowsingDataFunctionAndCompareRemovalMask(
       "passwords", BrowsingDataRemover::REMOVE_PASSWORDS);
   // We can't remove plugin data inside a test profile.
   RunRemoveBrowsingDataFunctionAndCompareRemovalMask(
       "webSQL", BrowsingDataRemover::REMOVE_WEBSQL);
 }