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

Unified Diff: chrome/browser/sync/profile_sync_service_unittest.cc

Issue 11474036: Remove initial_sync_ended bits (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Another rebase Created 8 years 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/profile_sync_service_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index 74040bdc951dce5c2ad8887bfae8a7915de8b6ef..a125be1475c4eb1209316b06eb39d0bf5263259b 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -421,6 +421,18 @@ TEST_F(ProfileSyncServiceTest, FailToOpenDatabase) {
EXPECT_FALSE(harness_.service->sync_initialized());
}
+// This setup will allow the database to exist, but leave it empty. The attempt
+// to download control types will silently fail (no downloads have any effect in
+// these tests). The sync_backend_host will notice this and inform the profile
+// sync service of the failure to initialize the backed.
+TEST_F(ProfileSyncServiceTest, FailToDownloadControlTypes) {
+ harness_.StartSyncServiceAndSetInitialSyncEnded(false, true, true, true,
+ syncer::STORAGE_IN_MEMORY);
+
+ // The backend is not ready. Ensure the PSS knows this.
+ EXPECT_FALSE(harness_.service->sync_initialized());
+}
+
// Register a handler with the ProfileSyncService, and disable and
// reenable sync. The handler should get notified of the state
// changes.
« no previous file with comments | « chrome/browser/sync/glue/sync_backend_host_unittest.cc ('k') | chrome/browser/sync/test_profile_sync_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698