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

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

Issue 23754021: Invalidation trickles mega-patch (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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
« no previous file with comments | « sync/sync_tests.gypi ('k') | sync/tools/sync_listen_notifications.cc » ('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 5bf2b4b0fb3ff80a431b48f26dd742849fca3879..5f1a1aae79af9aa56ba9a8f807d977e66a318484 100644
--- a/sync/test/engine/mock_connection_manager.cc
+++ b/sync/test/engine/mock_connection_manager.cc
@@ -492,18 +492,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/sync_listen_notifications.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698