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

Unified Diff: sync/test/engine/mock_connection_manager.cc

Issue 56113003: Implement new invalidations ack tracking system (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Modify drive TODO comment + rebase Created 7 years, 1 month 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
« no previous file with comments | « sync/sync_tests.gypi ('k') | sync/tools/null_invalidation_state_tracker.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/test/engine/mock_connection_manager.cc
diff --git a/sync/test/engine/mock_connection_manager.cc b/sync/test/engine/mock_connection_manager.cc
index 434f6c36fc692b5047fe11442a8a50276bdf657e..ef3022e42299669b8c2faa9359cc3df7bd4c80ff 100644
--- a/sync/test/engine/mock_connection_manager.cc
+++ b/sync/test/engine/mock_connection_manager.cc
@@ -497,18 +497,6 @@ void MockConnectionManager::ProcessGetUpdates(
gu.caller_info().source());
}
- // Verify that the GetUpdates filter sent by the Syncer matches the test
- // expectation.
- ModelTypeSet protocol_types = ProtocolTypes();
- for (ModelTypeSet::Iterator iter = protocol_types.First(); iter.Good();
- iter.Inc()) {
- ModelType model_type = iter.Get();
- sync_pb::DataTypeProgressMarker const* progress_marker =
- GetProgressMarkerForType(gu.from_progress_marker(), model_type);
- EXPECT_EQ(expected_filter_.Has(model_type), (progress_marker != NULL))
- << "Syncer requested_types differs from test expectation.";
- }
-
// Verify that the items we're about to send back to the client are of
// the types requested by the client. If this fails, it probably indicates
// a test bug.
« no previous file with comments | « sync/sync_tests.gypi ('k') | sync/tools/null_invalidation_state_tracker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698