Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(360)

Unified Diff: sync/internal_api/public/test/test_internal_components_factory.h

Issue 455023003: Let SyncBackupManager keep backup data in memory until shutdown. Only persist (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix sync manager test Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sync/internal_api/public/test/test_internal_components_factory.h
diff --git a/sync/internal_api/public/test/test_internal_components_factory.h b/sync/internal_api/public/test/test_internal_components_factory.h
index 3b7d3c5a87741c140bca9121fce6d32cd231bf90..0c6b9095120b0000eac05c0d0fd3d659551cb0a4 100644
--- a/sync/internal_api/public/test/test_internal_components_factory.h
+++ b/sync/internal_api/public/test/test_internal_components_factory.h
@@ -9,20 +9,11 @@
namespace syncer {
-enum StorageOption {
- // BuildDirectoryBackingStore should not use persistent on-disk storage.
- STORAGE_IN_MEMORY,
- // Use this if you want BuildDirectoryBackingStore to create a real
- // on disk store.
- STORAGE_ON_DISK,
- // Use this to test the case where a directory fails to load.
- STORAGE_INVALID
-};
-
class TestInternalComponentsFactory : public InternalComponentsFactory {
public:
explicit TestInternalComponentsFactory(const Switches& switches,
- StorageOption option);
+ StorageOption option,
+ StorageOption* storage_used);
virtual ~TestInternalComponentsFactory();
virtual scoped_ptr<SyncScheduler> BuildScheduler(
@@ -41,6 +32,7 @@ class TestInternalComponentsFactory : public InternalComponentsFactory {
virtual scoped_ptr<syncable::DirectoryBackingStore>
BuildDirectoryBackingStore(
+ StorageOption storage,
const std::string& dir_name,
const base::FilePath& backing_filepath) OVERRIDE;
@@ -48,7 +40,8 @@ class TestInternalComponentsFactory : public InternalComponentsFactory {
private:
const Switches switches_;
- const StorageOption storage_option_;
+ const StorageOption storage_override_;
+ StorageOption* storage_used_;
DISALLOW_COPY_AND_ASSIGN(TestInternalComponentsFactory);
};
« no previous file with comments | « sync/internal_api/public/internal_components_factory_impl.h ('k') | sync/internal_api/sync_backup_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698