[fsp] Decouple file_service_provider::Service.

chrome/browser/chromeos/file_manager/volume_manager_unittest.cc

 // Copyright 2013 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/chromeos/file_manager/volume_manager.h"
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/chromeos/file_manager/fake_disk_mount_manager.h"
 #include "chrome/browser/chromeos/file_manager/volume_manager_observer.h"
+#include "chrome/browser/chromeos/file_system_provider/fake_provided_file_system.h"
 #include "chrome/browser/chromeos/file_system_provider/service.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_profile.h"
 #include "chromeos/dbus/fake_power_manager_client.h"
 #include "chromeos/disks/disk_mount_manager.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace file_manager {
 namespace {
@@ skipping 115 matching lines @@
 class VolumeManagerTest : public testing::Test {
  protected:
   // Helper class that contains per-profile objects.
   class ProfileEnvironment {
    public:
     ProfileEnvironment(chromeos::PowerManagerClient* power_manager_client,
                        chromeos::disks::DiskMountManager* disk_manager)
         : profile_(new TestingProfile),
           file_system_provider_service_(
               new chromeos::file_system_provider::Service(profile_.get())),
-          volume_manager_(new VolumeManager(
-              profile_.get(),
-              NULL,  // DriveIntegrationService
-              power_manager_client,
-              disk_manager,
-              file_system_provider_service_.get())) {
+          volume_manager_(
+              new VolumeManager(profile_.get(),
+                                NULL,  // DriveIntegrationService
+                                power_manager_client,
+                                disk_manager,
+                                file_system_provider_service_.get())) {
+      file_system_provider_service_->SetFileSystemFactoryForTests(base::Bind(
+          chromeos::file_system_provider::FakeProvidedFileSystem::Create));

[hashimoto, 2014/04/15 13:42:00]

nit: '&' can be omitted here, but we usually add '&'.

[mtomasz, 2014/04/16 01:04:04]

Done.

     }
 
     Profile* profile() const { return profile_.get(); }
     VolumeManager* volume_manager() const { return volume_manager_.get(); }
 
    private:
     scoped_ptr<TestingProfile> profile_;
     scoped_ptr<chromeos::file_system_provider::Service>
         file_system_provider_service_;
     scoped_ptr<VolumeManager> volume_manager_;
@@ skipping 635 matching lines @@
       chromeos::disks::DiskMountManager::MountPointInfo(
           "/other/profile/drive/folder/3.zip",
           "/archive/3",
           chromeos::MOUNT_TYPE_ARCHIVE,
           chromeos::disks::MOUNT_CONDITION_NONE));
   EXPECT_FALSE(volume_manager()->FindVolumeInfoById("archive:3", &volume_info));
   EXPECT_EQ(3u, observer.events().size());
 }
 
 }  // namespace file_manager