| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "sync/sessions/nudge_tracker.h" | 5 #include "components/sync/sessions_impl/nudge_tracker.h" |
| 6 | 6 |
| 7 #include <stddef.h> | 7 #include <stddef.h> |
| 8 #include <stdint.h> | 8 #include <stdint.h> |
| 9 |
| 9 #include <string> | 10 #include <string> |
| 10 #include <utility> | 11 #include <utility> |
| 11 #include <vector> | 12 #include <vector> |
| 12 | 13 |
| 13 #include "base/message_loop/message_loop.h" | 14 #include "base/message_loop/message_loop.h" |
| 14 #include "base/run_loop.h" | 15 #include "base/run_loop.h" |
| 15 #include "sync/internal_api/public/base/model_type_test_util.h" | 16 #include "components/sync/base/model_type_test_util.h" |
| 16 #include "sync/test/mock_invalidation.h" | 17 #include "components/sync/test/mock_invalidation.h" |
| 17 #include "sync/test/mock_invalidation_tracker.h" | 18 #include "components/sync/test/mock_invalidation_tracker.h" |
| 18 #include "sync/test/trackable_mock_invalidation.h" | 19 #include "components/sync/test/trackable_mock_invalidation.h" |
| 19 #include "testing/gtest/include/gtest/gtest.h" | 20 #include "testing/gtest/include/gtest/gtest.h" |
| 20 | 21 |
| 21 namespace syncer { | 22 namespace syncer { |
| 22 | 23 |
| 23 namespace { | 24 namespace { |
| 24 | 25 |
| 25 testing::AssertionResult ModelTypeSetEquals(ModelTypeSet a, ModelTypeSet b) { | 26 testing::AssertionResult ModelTypeSetEquals(ModelTypeSet a, ModelTypeSet b) { |
| 26 if (a == b) { | 27 if (a == b) { |
| 27 return testing::AssertionSuccess(); | 28 return testing::AssertionSuccess(); |
| 28 } else { | 29 } else { |
| 29 return testing::AssertionFailure() | 30 return testing::AssertionFailure() |
| 30 << "Left side " << ModelTypeSetToString(a) | 31 << "Left side " << ModelTypeSetToString(a) |
| 31 << ", does not match rigth side: " << ModelTypeSetToString(b); | 32 << ", does not match rigth side: " << ModelTypeSetToString(b); |
| 32 } | 33 } |
| 33 } | 34 } |
| 34 | 35 |
| 35 } // namespace | 36 } // namespace |
| 36 | 37 |
| 37 namespace sessions { | 38 namespace sessions { |
| 38 | 39 |
| 39 class NudgeTrackerTest : public ::testing::Test { | 40 class NudgeTrackerTest : public ::testing::Test { |
| 40 public: | 41 public: |
| 41 NudgeTrackerTest() { | 42 NudgeTrackerTest() { SetInvalidationsInSync(); } |
| 42 SetInvalidationsInSync(); | |
| 43 } | |
| 44 | 43 |
| 45 static size_t GetHintBufferSize() { | 44 static size_t GetHintBufferSize() { |
| 46 // Assumes that no test has adjusted this size. | 45 // Assumes that no test has adjusted this size. |
| 47 return NudgeTracker::kDefaultMaxPayloadsPerType; | 46 return NudgeTracker::kDefaultMaxPayloadsPerType; |
| 48 } | 47 } |
| 49 | 48 |
| 50 bool InvalidationsOutOfSync() const { | 49 bool InvalidationsOutOfSync() const { |
| 51 // We don't currently track invalidations out of sync on a per-type basis. | 50 // We don't currently track invalidations out of sync on a per-type basis. |
| 52 sync_pb::GetUpdateTriggers gu_trigger; | 51 sync_pb::GetUpdateTriggers gu_trigger; |
| 53 nudge_tracker_.FillProtoMessage(BOOKMARKS, &gu_trigger); | 52 nudge_tracker_.FillProtoMessage(BOOKMARKS, &gu_trigger); |
| (...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 235 BuildInvalidation(1000, "new_hint")); | 234 BuildInvalidation(1000, "new_hint")); |
| 236 | 235 |
| 237 { | 236 { |
| 238 sync_pb::GetUpdateTriggers gu_trigger; | 237 sync_pb::GetUpdateTriggers gu_trigger; |
| 239 nudge_tracker_.FillProtoMessage(BOOKMARKS, &gu_trigger); | 238 nudge_tracker_.FillProtoMessage(BOOKMARKS, &gu_trigger); |
| 240 EXPECT_TRUE(gu_trigger.client_dropped_hints()); | 239 EXPECT_TRUE(gu_trigger.client_dropped_hints()); |
| 241 ASSERT_EQ(GetHintBufferSize(), | 240 ASSERT_EQ(GetHintBufferSize(), |
| 242 static_cast<size_t>(gu_trigger.notification_hint_size())); | 241 static_cast<size_t>(gu_trigger.notification_hint_size())); |
| 243 | 242 |
| 244 // Verify the newest hint was not dropped and is the last in the list. | 243 // Verify the newest hint was not dropped and is the last in the list. |
| 245 EXPECT_EQ("new_hint", gu_trigger.notification_hint(GetHintBufferSize()-1)); | 244 EXPECT_EQ("new_hint", |
| 245 gu_trigger.notification_hint(GetHintBufferSize() - 1)); |
| 246 | 246 |
| 247 // Verify the oldest hint, too. | 247 // Verify the oldest hint, too. |
| 248 EXPECT_EQ("hint", gu_trigger.notification_hint(0)); | 248 EXPECT_EQ("hint", gu_trigger.notification_hint(0)); |
| 249 } | 249 } |
| 250 } | 250 } |
| 251 | 251 |
| 252 // Tests the receipt of 'unknown version' invalidations. | 252 // Tests the receipt of 'unknown version' invalidations. |
| 253 TEST_F(NudgeTrackerTest, DropHintsAtServer_Alone) { | 253 TEST_F(NudgeTrackerTest, DropHintsAtServer_Alone) { |
| 254 // Record the unknown version invalidation. | 254 // Record the unknown version invalidation. |
| 255 nudge_tracker_.RecordRemoteInvalidation(BOOKMARKS, | 255 nudge_tracker_.RecordRemoteInvalidation(BOOKMARKS, |
| (...skipping 183 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 439 const base::TimeDelta throttle_length = base::TimeDelta::FromMinutes(10); | 439 const base::TimeDelta throttle_length = base::TimeDelta::FromMinutes(10); |
| 440 const base::TimeTicks t1 = t0 + throttle_length; | 440 const base::TimeTicks t1 = t0 + throttle_length; |
| 441 | 441 |
| 442 EXPECT_FALSE(nudge_tracker_.IsSyncRequired()); | 442 EXPECT_FALSE(nudge_tracker_.IsSyncRequired()); |
| 443 | 443 |
| 444 // A local change to sessions enables the flag. | 444 // A local change to sessions enables the flag. |
| 445 nudge_tracker_.RecordLocalChange(ModelTypeSet(SESSIONS)); | 445 nudge_tracker_.RecordLocalChange(ModelTypeSet(SESSIONS)); |
| 446 EXPECT_TRUE(nudge_tracker_.IsSyncRequired()); | 446 EXPECT_TRUE(nudge_tracker_.IsSyncRequired()); |
| 447 | 447 |
| 448 // But the throttling of sessions unsets it. | 448 // But the throttling of sessions unsets it. |
| 449 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS), | 449 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS), throttle_length, |
| 450 throttle_length, | 450 t0); |
| 451 t0); | |
| 452 EXPECT_FALSE(nudge_tracker_.IsSyncRequired()); | 451 EXPECT_FALSE(nudge_tracker_.IsSyncRequired()); |
| 453 | 452 |
| 454 // A refresh request for bookmarks means we have reason to sync again. | 453 // A refresh request for bookmarks means we have reason to sync again. |
| 455 nudge_tracker_.RecordLocalRefreshRequest(ModelTypeSet(BOOKMARKS)); | 454 nudge_tracker_.RecordLocalRefreshRequest(ModelTypeSet(BOOKMARKS)); |
| 456 EXPECT_TRUE(nudge_tracker_.IsSyncRequired()); | 455 EXPECT_TRUE(nudge_tracker_.IsSyncRequired()); |
| 457 | 456 |
| 458 // A successful sync cycle means we took care of bookmarks. | 457 // A successful sync cycle means we took care of bookmarks. |
| 459 nudge_tracker_.RecordSuccessfulSyncCycle(); | 458 nudge_tracker_.RecordSuccessfulSyncCycle(); |
| 460 EXPECT_FALSE(nudge_tracker_.IsSyncRequired()); | 459 EXPECT_FALSE(nudge_tracker_.IsSyncRequired()); |
| 461 | 460 |
| (...skipping 11 matching lines...) Expand all Loading... |
| 473 const base::TimeDelta throttle_length = base::TimeDelta::FromMinutes(10); | 472 const base::TimeDelta throttle_length = base::TimeDelta::FromMinutes(10); |
| 474 const base::TimeTicks t1 = t0 + throttle_length; | 473 const base::TimeTicks t1 = t0 + throttle_length; |
| 475 | 474 |
| 476 EXPECT_FALSE(nudge_tracker_.IsGetUpdatesRequired()); | 475 EXPECT_FALSE(nudge_tracker_.IsGetUpdatesRequired()); |
| 477 | 476 |
| 478 // A refresh request to sessions enables the flag. | 477 // A refresh request to sessions enables the flag. |
| 479 nudge_tracker_.RecordLocalRefreshRequest(ModelTypeSet(SESSIONS)); | 478 nudge_tracker_.RecordLocalRefreshRequest(ModelTypeSet(SESSIONS)); |
| 480 EXPECT_TRUE(nudge_tracker_.IsGetUpdatesRequired()); | 479 EXPECT_TRUE(nudge_tracker_.IsGetUpdatesRequired()); |
| 481 | 480 |
| 482 // But the throttling of sessions unsets it. | 481 // But the throttling of sessions unsets it. |
| 483 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS), | 482 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS), throttle_length, |
| 484 throttle_length, | 483 t0); |
| 485 t0); | |
| 486 EXPECT_FALSE(nudge_tracker_.IsGetUpdatesRequired()); | 484 EXPECT_FALSE(nudge_tracker_.IsGetUpdatesRequired()); |
| 487 | 485 |
| 488 // A refresh request for bookmarks means we have reason to sync again. | 486 // A refresh request for bookmarks means we have reason to sync again. |
| 489 nudge_tracker_.RecordLocalRefreshRequest(ModelTypeSet(BOOKMARKS)); | 487 nudge_tracker_.RecordLocalRefreshRequest(ModelTypeSet(BOOKMARKS)); |
| 490 EXPECT_TRUE(nudge_tracker_.IsGetUpdatesRequired()); | 488 EXPECT_TRUE(nudge_tracker_.IsGetUpdatesRequired()); |
| 491 | 489 |
| 492 // A successful sync cycle means we took care of bookmarks. | 490 // A successful sync cycle means we took care of bookmarks. |
| 493 nudge_tracker_.RecordSuccessfulSyncCycle(); | 491 nudge_tracker_.RecordSuccessfulSyncCycle(); |
| 494 EXPECT_FALSE(nudge_tracker_.IsGetUpdatesRequired()); | 492 EXPECT_FALSE(nudge_tracker_.IsGetUpdatesRequired()); |
| 495 | 493 |
| (...skipping 12 matching lines...) Expand all Loading... |
| 508 EXPECT_TRUE(nudge_tracker_.GetThrottledTypes().Empty()); | 506 EXPECT_TRUE(nudge_tracker_.GetThrottledTypes().Empty()); |
| 509 } | 507 } |
| 510 | 508 |
| 511 // Tests throttling-related getter functions when some types are throttled. | 509 // Tests throttling-related getter functions when some types are throttled. |
| 512 TEST_F(NudgeTrackerTest, ThrottleAndUnthrottle) { | 510 TEST_F(NudgeTrackerTest, ThrottleAndUnthrottle) { |
| 513 const base::TimeTicks t0 = base::TimeTicks::FromInternalValue(1234); | 511 const base::TimeTicks t0 = base::TimeTicks::FromInternalValue(1234); |
| 514 const base::TimeDelta throttle_length = base::TimeDelta::FromMinutes(10); | 512 const base::TimeDelta throttle_length = base::TimeDelta::FromMinutes(10); |
| 515 const base::TimeTicks t1 = t0 + throttle_length; | 513 const base::TimeTicks t1 = t0 + throttle_length; |
| 516 | 514 |
| 517 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS, PREFERENCES), | 515 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS, PREFERENCES), |
| 518 throttle_length, | 516 throttle_length, t0); |
| 519 t0); | |
| 520 | 517 |
| 521 EXPECT_TRUE(nudge_tracker_.IsAnyTypeThrottled()); | 518 EXPECT_TRUE(nudge_tracker_.IsAnyTypeThrottled()); |
| 522 EXPECT_TRUE(nudge_tracker_.IsTypeThrottled(SESSIONS)); | 519 EXPECT_TRUE(nudge_tracker_.IsTypeThrottled(SESSIONS)); |
| 523 EXPECT_TRUE(nudge_tracker_.IsTypeThrottled(PREFERENCES)); | 520 EXPECT_TRUE(nudge_tracker_.IsTypeThrottled(PREFERENCES)); |
| 524 EXPECT_FALSE(nudge_tracker_.GetThrottledTypes().Empty()); | 521 EXPECT_FALSE(nudge_tracker_.GetThrottledTypes().Empty()); |
| 525 EXPECT_EQ(throttle_length, nudge_tracker_.GetTimeUntilNextUnthrottle(t0)); | 522 EXPECT_EQ(throttle_length, nudge_tracker_.GetTimeUntilNextUnthrottle(t0)); |
| 526 | 523 |
| 527 nudge_tracker_.UpdateTypeThrottlingState(t1); | 524 nudge_tracker_.UpdateTypeThrottlingState(t1); |
| 528 | 525 |
| 529 EXPECT_FALSE(nudge_tracker_.IsAnyTypeThrottled()); | 526 EXPECT_FALSE(nudge_tracker_.IsAnyTypeThrottled()); |
| 530 EXPECT_FALSE(nudge_tracker_.IsTypeThrottled(SESSIONS)); | 527 EXPECT_FALSE(nudge_tracker_.IsTypeThrottled(SESSIONS)); |
| 531 EXPECT_TRUE(nudge_tracker_.GetThrottledTypes().Empty()); | 528 EXPECT_TRUE(nudge_tracker_.GetThrottledTypes().Empty()); |
| 532 } | 529 } |
| 533 | 530 |
| 534 TEST_F(NudgeTrackerTest, OverlappingThrottleIntervals) { | 531 TEST_F(NudgeTrackerTest, OverlappingThrottleIntervals) { |
| 535 const base::TimeTicks t0 = base::TimeTicks::FromInternalValue(1234); | 532 const base::TimeTicks t0 = base::TimeTicks::FromInternalValue(1234); |
| 536 const base::TimeDelta throttle1_length = base::TimeDelta::FromMinutes(10); | 533 const base::TimeDelta throttle1_length = base::TimeDelta::FromMinutes(10); |
| 537 const base::TimeDelta throttle2_length = base::TimeDelta::FromMinutes(20); | 534 const base::TimeDelta throttle2_length = base::TimeDelta::FromMinutes(20); |
| 538 const base::TimeTicks t1 = t0 + throttle1_length; | 535 const base::TimeTicks t1 = t0 + throttle1_length; |
| 539 const base::TimeTicks t2 = t0 + throttle2_length; | 536 const base::TimeTicks t2 = t0 + throttle2_length; |
| 540 | 537 |
| 541 // Setup the longer of two intervals. | 538 // Setup the longer of two intervals. |
| 542 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS, PREFERENCES), | 539 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS, PREFERENCES), |
| 543 throttle2_length, | 540 throttle2_length, t0); |
| 544 t0); | 541 EXPECT_TRUE(ModelTypeSetEquals(ModelTypeSet(SESSIONS, PREFERENCES), |
| 545 EXPECT_TRUE(ModelTypeSetEquals( | 542 nudge_tracker_.GetThrottledTypes())); |
| 546 ModelTypeSet(SESSIONS, PREFERENCES), | 543 EXPECT_EQ(throttle2_length, nudge_tracker_.GetTimeUntilNextUnthrottle(t0)); |
| 547 nudge_tracker_.GetThrottledTypes())); | |
| 548 EXPECT_EQ(throttle2_length, | |
| 549 nudge_tracker_.GetTimeUntilNextUnthrottle(t0)); | |
| 550 | 544 |
| 551 // Setup the shorter interval. | 545 // Setup the shorter interval. |
| 552 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS, BOOKMARKS), | 546 nudge_tracker_.SetTypesThrottledUntil(ModelTypeSet(SESSIONS, BOOKMARKS), |
| 553 throttle1_length, | 547 throttle1_length, t0); |
| 554 t0); | 548 EXPECT_TRUE(ModelTypeSetEquals(ModelTypeSet(SESSIONS, PREFERENCES, BOOKMARKS), |
| 555 EXPECT_TRUE(ModelTypeSetEquals( | 549 nudge_tracker_.GetThrottledTypes())); |
| 556 ModelTypeSet(SESSIONS, PREFERENCES, BOOKMARKS), | 550 EXPECT_EQ(throttle1_length, nudge_tracker_.GetTimeUntilNextUnthrottle(t0)); |
| 557 nudge_tracker_.GetThrottledTypes())); | |
| 558 EXPECT_EQ(throttle1_length, | |
| 559 nudge_tracker_.GetTimeUntilNextUnthrottle(t0)); | |
| 560 | 551 |
| 561 // Expire the first interval. | 552 // Expire the first interval. |
| 562 nudge_tracker_.UpdateTypeThrottlingState(t1); | 553 nudge_tracker_.UpdateTypeThrottlingState(t1); |
| 563 | 554 |
| 564 // SESSIONS appeared in both intervals. We expect it will be throttled for | 555 // SESSIONS appeared in both intervals. We expect it will be throttled for |
| 565 // the longer of the two, so it's still throttled at time t1. | 556 // the longer of the two, so it's still throttled at time t1. |
| 566 EXPECT_TRUE(ModelTypeSetEquals( | 557 EXPECT_TRUE(ModelTypeSetEquals(ModelTypeSet(SESSIONS, PREFERENCES), |
| 567 ModelTypeSet(SESSIONS, PREFERENCES), | 558 nudge_tracker_.GetThrottledTypes())); |
| 568 nudge_tracker_.GetThrottledTypes())); | |
| 569 EXPECT_EQ(throttle2_length - throttle1_length, | 559 EXPECT_EQ(throttle2_length - throttle1_length, |
| 570 nudge_tracker_.GetTimeUntilNextUnthrottle(t1)); | 560 nudge_tracker_.GetTimeUntilNextUnthrottle(t1)); |
| 571 | 561 |
| 572 // Expire the second interval. | 562 // Expire the second interval. |
| 573 nudge_tracker_.UpdateTypeThrottlingState(t2); | 563 nudge_tracker_.UpdateTypeThrottlingState(t2); |
| 574 EXPECT_TRUE(nudge_tracker_.GetThrottledTypes().Empty()); | 564 EXPECT_TRUE(nudge_tracker_.GetThrottledTypes().Empty()); |
| 575 } | 565 } |
| 576 | 566 |
| 577 TEST_F(NudgeTrackerTest, Retry) { | 567 TEST_F(NudgeTrackerTest, Retry) { |
| 578 const base::TimeTicks t0 = base::TimeTicks::FromInternalValue(12345); | 568 const base::TimeTicks t0 = base::TimeTicks::FromInternalValue(12345); |
| (...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 906 // Test dropping when the buffer overflows and subsequent drop recovery. | 896 // Test dropping when the buffer overflows and subsequent drop recovery. |
| 907 TEST_F(NudgeTrackerAckTrackingTest, OverflowAndRecover) { | 897 TEST_F(NudgeTrackerAckTrackingTest, OverflowAndRecover) { |
| 908 std::vector<int> invalidation_ids; | 898 std::vector<int> invalidation_ids; |
| 909 | 899 |
| 910 int inv10_id = SendInvalidation(BOOKMARKS, 10, "hint"); | 900 int inv10_id = SendInvalidation(BOOKMARKS, 10, "hint"); |
| 911 for (size_t i = 1; i < GetHintBufferSize(); ++i) { | 901 for (size_t i = 1; i < GetHintBufferSize(); ++i) { |
| 912 invalidation_ids.push_back(SendInvalidation(BOOKMARKS, i + 10, "hint")); | 902 invalidation_ids.push_back(SendInvalidation(BOOKMARKS, i + 10, "hint")); |
| 913 } | 903 } |
| 914 | 904 |
| 915 for (std::vector<int>::iterator it = invalidation_ids.begin(); | 905 for (std::vector<int>::iterator it = invalidation_ids.begin(); |
| 916 it != invalidation_ids.end(); | 906 it != invalidation_ids.end(); ++it) { |
| 917 ++it) { | |
| 918 EXPECT_TRUE(IsInvalidationUnacknowledged(*it)); | 907 EXPECT_TRUE(IsInvalidationUnacknowledged(*it)); |
| 919 } | 908 } |
| 920 | 909 |
| 921 // This invalidation, though arriving the most recently, has the oldest | 910 // This invalidation, though arriving the most recently, has the oldest |
| 922 // version number so it should be dropped first. | 911 // version number so it should be dropped first. |
| 923 int inv5_id = SendInvalidation(BOOKMARKS, 5, "old_hint"); | 912 int inv5_id = SendInvalidation(BOOKMARKS, 5, "old_hint"); |
| 924 EXPECT_TRUE(IsInvalidationDropped(inv5_id)); | 913 EXPECT_TRUE(IsInvalidationDropped(inv5_id)); |
| 925 | 914 |
| 926 // This invalidation has a larger version number, so it will force a | 915 // This invalidation has a larger version number, so it will force a |
| 927 // previously delivered invalidation to be dropped. | 916 // previously delivered invalidation to be dropped. |
| 928 int inv100_id = SendInvalidation(BOOKMARKS, 100, "new_hint"); | 917 int inv100_id = SendInvalidation(BOOKMARKS, 100, "new_hint"); |
| 929 EXPECT_TRUE(IsInvalidationDropped(inv10_id)); | 918 EXPECT_TRUE(IsInvalidationDropped(inv10_id)); |
| 930 | 919 |
| 931 // This should recover from the drop and bring us back into sync. | 920 // This should recover from the drop and bring us back into sync. |
| 932 RecordSuccessfulSyncCycle(); | 921 RecordSuccessfulSyncCycle(); |
| 933 | 922 |
| 934 for (std::vector<int>::iterator it = invalidation_ids.begin(); | 923 for (std::vector<int>::iterator it = invalidation_ids.begin(); |
| 935 it != invalidation_ids.end(); | 924 it != invalidation_ids.end(); ++it) { |
| 936 ++it) { | |
| 937 EXPECT_TRUE(IsInvalidationAcknowledged(*it)); | 925 EXPECT_TRUE(IsInvalidationAcknowledged(*it)); |
| 938 } | 926 } |
| 939 EXPECT_TRUE(IsInvalidationAcknowledged(inv100_id)); | 927 EXPECT_TRUE(IsInvalidationAcknowledged(inv100_id)); |
| 940 | 928 |
| 941 EXPECT_TRUE(AllInvalidationsAccountedFor()); | 929 EXPECT_TRUE(AllInvalidationsAccountedFor()); |
| 942 } | 930 } |
| 943 | 931 |
| 944 // Test receipt of an unknown version invalidation from the server. | 932 // Test receipt of an unknown version invalidation from the server. |
| 945 TEST_F(NudgeTrackerAckTrackingTest, UnknownVersionFromServer_Simple) { | 933 TEST_F(NudgeTrackerAckTrackingTest, UnknownVersionFromServer_Simple) { |
| 946 int inv_id = SendUnknownVersionInvalidation(BOOKMARKS); | 934 int inv_id = SendUnknownVersionInvalidation(BOOKMARKS); |
| (...skipping 26 matching lines...) Expand all Loading... |
| 973 EXPECT_TRUE(IsInvalidationAcknowledged(inv2_id)); | 961 EXPECT_TRUE(IsInvalidationAcknowledged(inv2_id)); |
| 974 EXPECT_TRUE(IsInvalidationAcknowledged(inv3_id)); | 962 EXPECT_TRUE(IsInvalidationAcknowledged(inv3_id)); |
| 975 EXPECT_TRUE(IsInvalidationAcknowledged(inv4_id)); | 963 EXPECT_TRUE(IsInvalidationAcknowledged(inv4_id)); |
| 976 EXPECT_TRUE(IsInvalidationAcknowledged(inv5_id)); | 964 EXPECT_TRUE(IsInvalidationAcknowledged(inv5_id)); |
| 977 | 965 |
| 978 EXPECT_TRUE(AllInvalidationsAccountedFor()); | 966 EXPECT_TRUE(AllInvalidationsAccountedFor()); |
| 979 } | 967 } |
| 980 | 968 |
| 981 } // namespace sessions | 969 } // namespace sessions |
| 982 } // namespace syncer | 970 } // namespace syncer |
| OLD | NEW |