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

Unified Diff: chrome/browser/sync/test/integration/sync_exponential_backoff_test.cc

Issue 2258873003: [Sync] Move sessions/ to engine/cycle/ and rename things to match. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. Created 4 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/sync_exponential_backoff_test.cc
diff --git a/chrome/browser/sync/test/integration/sync_exponential_backoff_test.cc b/chrome/browser/sync/test/integration/sync_exponential_backoff_test.cc
index 19108118b4c294a07c3b2ff0e8acaf662b5e31d6..e61d734ef8c408245c099c9042604b340ca588ae 100644
--- a/chrome/browser/sync/test/integration/sync_exponential_backoff_test.cc
+++ b/chrome/browser/sync/test/integration/sync_exponential_backoff_test.cc
@@ -17,7 +17,7 @@ namespace {
using bookmarks_helper::AddFolder;
using bookmarks_helper::ModelMatchesVerifier;
-using syncer::sessions::SyncSessionSnapshot;
+using syncer::SyncCycleSnapshot;
using sync_integration_test_util::AwaitCommitActivityCompletion;
class SyncExponentialBackoffTest : public SyncTest {
@@ -35,7 +35,7 @@ class ExponentialBackoffChecker : public SingleClientStatusChangeChecker {
public:
explicit ExponentialBackoffChecker(ProfileSyncService* pss)
: SingleClientStatusChangeChecker(pss) {
- const SyncSessionSnapshot& snap = service()->GetLastSessionSnapshot();
+ const SyncCycleSnapshot& snap = service()->GetLastCycleSnapshot();
retry_verifier_.Initialize(snap);
}
@@ -44,7 +44,7 @@ class ExponentialBackoffChecker : public SingleClientStatusChangeChecker {
// Checks if backoff is complete. Called repeatedly each time PSS notifies
// observers of a state change.
bool IsExitConditionSatisfied() override {
- const SyncSessionSnapshot& snap = service()->GetLastSessionSnapshot();
+ const SyncCycleSnapshot& snap = service()->GetLastCycleSnapshot();
retry_verifier_.VerifyRetryInterval(snap);
return (retry_verifier_.done() && retry_verifier_.Succeeded());
}
@@ -97,7 +97,7 @@ IN_PROC_BROWSER_TEST_F(SyncExponentialBackoffTest, OfflineToOnline) {
// Verify that recovery time is short. Without canary job recovery time would
// be more than 5 seconds.
base::TimeDelta recovery_time =
- GetSyncService(0)->GetLastSessionSnapshot().sync_start_time() -
+ GetSyncService(0)->GetLastCycleSnapshot().sync_start_time() -
network_notification_time;
ASSERT_LE(recovery_time, base::TimeDelta::FromSeconds(2));
}

Powered by Google App Engine
This is Rietveld 408576698