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

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: 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
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 624e02218822a9c2fc9509f10280222fbabb5915..ea85933e74bffff0bdf243ac423c7ee018711e7a 100644
--- a/sync/test/engine/mock_connection_manager.cc
+++ b/sync/test/engine/mock_connection_manager.cc
@@ -496,18 +496,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.

Powered by Google App Engine
This is Rietveld 408576698