[Sync] Add tests for invalid specifics field number handling

sync/engine/store_timestamps_command.h

Index: sync/engine/store_timestamps_command.h
diff --git a/sync/engine/store_timestamps_command.h b/sync/engine/store_timestamps_command.h
index a73f30936e27efd826916e3e0fe04e321dee77b9..a6426a8937bec8cb1f75d8ebe915cc7114372203 100644
--- a/sync/engine/store_timestamps_command.h
+++ b/sync/engine/store_timestamps_command.h
@@ -6,11 +6,27 @@
 #define SYNC_ENGINE_STORE_TIMESTAMPS_COMMAND_H_
 
 #include "base/compiler_specific.h"
+#include "sync/base/sync_export.h"
 #include "sync/engine/syncer_command.h"
 #include "sync/engine/syncer_types.h"
+#include "sync/internal_api/public/base/model_type.h"
+
+namespace sync_pb {
+class GetUpdatesResponse;
+}  // namespace sync_pb
 
 namespace syncer {
 
+namespace syncable {
+class Directory;
+}  // namespace syncable
+
+// Sets |dir|'s progress markers from the data in |response|.  Returns
+// the set of model types with new progress markers.
+SYNC_EXPORT_PRIVATE ModelTypeSet ProcessNewProgressMarkers(

[Nicolas Zea, 2012/12/13 01:48:21]

Is this now necessary for any methods called in tests?

[akalin, 2012/12/13 01:50:30]

Not yet, but very soon!

+    const sync_pb::GetUpdatesResponse& response,
+    syncable::Directory* dir);
+
 // A syncer command that extracts the changelog timestamp information from
 // a GetUpdatesResponse (fetched in DownloadUpdatesCommand) and stores
 // it in the directory.  This is meant to run immediately after