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

Unified Diff: chrome/browser/sync/test/integration/migration_watcher.h

Issue 490643004: sync: Refactor migration integration tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments 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: chrome/browser/sync/test/integration/migration_watcher.h
diff --git a/chrome/browser/sync/test/integration/migration_watcher.h b/chrome/browser/sync/test/integration/migration_watcher.h
new file mode 100644
index 0000000000000000000000000000000000000000..42e340bf37f549bd4534d2c5558fc2944e811b2a
--- /dev/null
+++ b/chrome/browser/sync/test/integration/migration_watcher.h
@@ -0,0 +1,53 @@
+// Copyright 2014 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.
+
+#ifndef CHROME_BROWSER_SYNC_TEST_INTEGRATION_MIGRATION_WATCHER_H_
+#define CHROME_BROWSER_SYNC_TEST_INTEGRATION_MIGRATION_WATCHER_H_
+
+#include "base/macros.h"
+#include "chrome/browser/sync/backend_migrator.h"
+#include "sync/internal_api/public/base/model_type.h"
+
+class ProfileSyncServiceHarness;
+class MigrationWaiter;
+
+// Helper class to observe and record migration state.
+class MigrationWatcher : public browser_sync::MigrationObserver {
+ public:
+ explicit MigrationWatcher(ProfileSyncServiceHarness* harness);
+ virtual ~MigrationWatcher();
+
+ // Returns true if the observed profile has a migration in progress.
+ bool HasPendingBackendMigration() const;
+
+ // Returns the set of types this class has observed being migrated.
+ syncer::ModelTypeSet GetMigratedTypes() const;
+
+ // Implementation of browser_sync::MigrationObserver.
+ virtual void OnMigrationStateChange() OVERRIDE;
+
+ // Registers the |waiter| to receive callbacks on migration state change.
+ void set_migration_waiter(MigrationWaiter* waiter);
+
+ // Unregisters the current MigrationWaiter, if any.
+ void clear_migration_waiter();
+
+ private:
+ // The ProfileSyncServiceHarness to watch.
+ ProfileSyncServiceHarness* const harness_;
+
+ // The set of data types currently undergoing migration.
+ syncer::ModelTypeSet pending_types_;
+
+ // The set of data types for which migration is complete. Accumulated by
+ // successive calls to OnMigrationStateChanged.
+ syncer::ModelTypeSet migrated_types_;
+
+ // The MigrationWatier that is waiting for this migration to complete.
+ MigrationWaiter* migration_waiter_;
+
+ DISALLOW_COPY_AND_ASSIGN(MigrationWatcher);
+};
+
+#endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_MIGRATION_WATCHER_H_

Powered by Google App Engine
This is Rietveld 408576698