Fix remaining leaks in StorageMonitorLinux.

chrome/test/base/testing_browser_process.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/test/base/testing_browser_process.h"
 
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_util.h"
 #include "build/build_config.h"
 #include "chrome/browser/background/background_mode_manager.h"
@@ skipping 311 matching lines @@
 #else
   return bookmark_prompt_controller_.get();
 #endif
 }
 
 chrome::StorageMonitor* TestingBrowserProcess::storage_monitor() {
 #if defined(OS_IOS) || defined(OS_ANDROID)
   NOTIMPLEMENTED();
   return NULL;
 #else
-  if (!storage_monitor_.get()) {
-    chrome::test::TestStorageMonitor* monitor =

[vandebo (ex-Chrome), 2013/07/24 14:42:59]

Why get rid of this, this is how a lot of things in TestingBrowserProcess works.

-        new chrome::test::TestStorageMonitor();
-    monitor->Init();
-    monitor->MarkInitialized();
-    storage_monitor_.reset(monitor);
-  }
   return storage_monitor_.get();
 #endif
 }
 
 chrome::MediaFileSystemRegistry*
 TestingBrowserProcess::media_file_system_registry() {
 #if defined(OS_IOS) || defined(OS_ANDROID)
   NOTIMPLEMENTED();
   return NULL;
 #else
@@ skipping 66 matching lines @@
     SafeBrowsingService* sb_service) {
 #if !defined(OS_IOS)
   NOTIMPLEMENTED();
   sb_service_ = sb_service;
 #endif
 }
 
 void TestingBrowserProcess::SetStorageMonitor(
     scoped_ptr<chrome::StorageMonitor> storage_monitor) {
 #if !defined(OS_IOS) && !defined(OS_ANDROID)
-  storage_monitor_.reset(storage_monitor.release());
+  storage_monitor_ = storage_monitor.Pass();
 #endif
 }