Implement a bag of chips for sync.

sync/engine/process_commit_response_command.cc

Index: sync/engine/process_commit_response_command.cc
diff --git a/sync/engine/process_commit_response_command.cc b/sync/engine/process_commit_response_command.cc
index 8a00608f925bd143a18a35006565e753447bc870..0062ed4004c030ea5666a56f5f6fda4ca6389f24 100644
--- a/sync/engine/process_commit_response_command.cc
+++ b/sync/engine/process_commit_response_command.cc
@@ -62,6 +62,17 @@ ProcessCommitResponseCommand::ProcessCommitResponseCommand(
 
 ProcessCommitResponseCommand::~ProcessCommitResponseCommand() {}
 
+SyncerError ProcessCommitResponseCommand::ExecuteImpl(SyncSession* session) {

[tim (not reviewing), 2012/09/10 17:52:55]

mm, I understand how you got here, but lets not override this :)

Maybe lets leave ProcessCommitResponseCommand to do the "model changing work",
and move this meta-work to commit.cc, right before invoking
ProcessCommitResponseCommand?

+  syncable::Directory* dir = session->context()->directory();
+  const sync_pb::ClientToServerResponse& response =
+      session->status_controller().updates_response();
+
+  // Persist a bag of chips if it has been sent by the server.
+  SyncerProtoUtil::PersistBagOfChips(dir, response);
+
+  return ModelChangingSyncerCommand::ExecuteImpl(session);
+}
+
 std::set<ModelSafeGroup> ProcessCommitResponseCommand::GetGroupsToChange(
     const sessions::SyncSession& session) const {
   std::set<ModelSafeGroup> groups_with_commits;