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

Unified Diff: sync/engine/sync_scheduler_unittest.cc

Issue 146113003: sync: GU retry with less explicit TimeTicks logic (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix logic in GU retry fail edge case Created 6 years, 11 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/engine/sync_scheduler_unittest.cc
diff --git a/sync/engine/sync_scheduler_unittest.cc b/sync/engine/sync_scheduler_unittest.cc
index 7b0a0d35c03720d0f8723e9b9b18354c21f54b1f..b056717479b8e24e5e81dbf57faf2372d094a595 100644
--- a/sync/engine/sync_scheduler_unittest.cc
+++ b/sync/engine/sync_scheduler_unittest.cc
@@ -1369,21 +1369,21 @@ TEST_F(SyncSchedulerTest, ReceiveNewRetryDelay) {
scheduler()->ScheduleLocalRefreshRequest(zero(), ModelTypeSet(BOOKMARKS),
FROM_HERE);
scheduler()->OnReceivedGuRetryDelay(delay1);
+ EXPECT_EQ(delay1, GetRetryTimerDelay());
EXPECT_CALL(*syncer(), NormalSyncShare(_,_,_))
.WillOnce(DoAll(
WithoutArgs(VerifyRetryTimerDelay(this, delay1)),
WithArg<2>(sessions::test_util::SimulateGuRetryDelayCommand(delay2)),
- WithoutArgs(VerifyRetryTimerDelay(this, delay2)),
RecordSyncShare(&times)));
// Run nudge GU.
RunLoop();
+ EXPECT_EQ(delay2, GetRetryTimerDelay());
EXPECT_CALL(*syncer(), RetrySyncShare(_,_))
- .WillOnce(
- DoAll(Invoke(sessions::test_util::SimulatePollRetrySuccess),
- RecordSyncShare(&times)));
+ .WillOnce(DoAll(Invoke(sessions::test_util::SimulatePollRetrySuccess),
+ RecordSyncShare(&times)));
// Run to wait for retrying.
RunLoop();

Powered by Google App Engine
This is Rietveld 408576698