The Firefox3XImporter test was incorrectly deleting a ref counted object caus...

chrome/browser/importer/importer_unittest.cc

 // Copyright (c) 2006-2008 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 "testing/gtest/include/gtest/gtest.h"
 
 #include <windows.h>
 #include <unknwn.h>
 #include <intshcut.h>
 #include <pstore.h>
@@ skipping 53 matching lines @@
     ASSERT_TRUE(file_util::CopyDirectory(data_path, profile_path_, false));
 
     std::wstring search_engine_path = app_path_;
     file_util::AppendToPath(&search_engine_path, L"searchplugins");
     CreateDirectory(search_engine_path.c_str(), NULL);
     if (import_search_plugins) {
       ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &data_path));
       file_util::AppendToPath(&data_path, L"firefox3_searchplugins");
       if (!file_util::PathExists(data_path)) {
         // TODO(maruel):  Create search test data that we can open source!
-        delete observer;
         LOG(ERROR) << L"Missing internal test data";
         return;
       }
       ASSERT_TRUE(file_util::CopyDirectory(data_path,
                                            search_engine_path, false));
     }
 
     MessageLoop* loop = MessageLoop::current();
     ProfileInfo profile_info;
     profile_info.browser_type = FIREFOX3;
@@ skipping 746 matching lines @@
   int bookmark_count_;
   int history_count_;
   int password_count_;
   int keyword_count_;
   bool import_search_engines_;
   std::wstring default_keyword_;
   std::wstring default_keyword_url_;
 };
 
 // This test is disabled, see bug 22884
-TEST_F(ImporterTest, DISABLED_Firefox30Importer) {
-  Firefox3Observer* observer = new Firefox3Observer();
-  Firefox3xImporterTest(L"firefox3_profile", observer, observer, true);
+TEST_F(ImporterTest, Firefox30Importer) {
+  scoped_refptr<Firefox3Observer> observer = new Firefox3Observer();
+  Firefox3xImporterTest(L"firefox3_profile", observer.get(), observer.get(),
+                        true);
 }
 
 TEST_F(ImporterTest, Firefox35Importer) {
   bool import_search_engines = false;
-  Firefox3Observer* observer = new Firefox3Observer(import_search_engines);
-  Firefox3xImporterTest(L"firefox35_profile", observer, observer,
+  scoped_refptr<Firefox3Observer> observer =
+      new Firefox3Observer(import_search_engines);
+  Firefox3xImporterTest(L"firefox35_profile", observer.get(), observer.get(),
                         import_search_engines);
 }