sync: tiny cleanup in syncer.h/cc

Moves last bit of state from syncer to the sync session context, which is where we store state we want to carry over between sessions.

BUG=none
TEST=syncer_unittest

Review URL: http://codereview.chromium.org/6349030

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73355 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 13 insertions, 1 deletions

diff --git a/chrome/browser/sync/sessions/sync_session_context.cc b/chrome/browser/sync/sessions/sync_session_context.cc
index dbb4da6..6b0e91e 100644
--- a/chrome/browser/sync/sessions/sync_session_context.cc
+++ b/chrome/browser/sync/sessions/sync_session_context.cc
@@ -21,7 +21,8 @@ SyncSessionContext::SyncSessionContext(
       directory_manager_(directory_manager),
       registrar_(model_safe_worker_registrar),
       extensions_activity_monitor_(new ExtensionsActivityMonitor()),
-      notifications_enabled_(false) {
+      notifications_enabled_(false),
+      max_commit_batch_size_(kDefaultMaxCommitBatchSize) {
   std::vector<SyncEngineEventListener*>::const_iterator it;
   for (it = listeners.begin(); it != listeners.end(); ++it)
     listeners_.AddObserver(*it);
diff --git a/chrome/browser/sync/sessions/sync_session_context.h b/chrome/browser/sync/sessions/sync_session_context.h
index 2fe8832..3f402ae 100644
--- a/chrome/browser/sync/sessions/sync_session_context.h
+++ b/chrome/browser/sync/sessions/sync_session_context.h
@@ -36,6 +36,9 @@ class ExtensionsActivityMonitor;
 class ModelSafeWorkerRegistrar;
 class ServerConnectionManager;
 
+// Default number of items a client can commit in a single message.
+static const int kDefaultMaxCommitBatchSize = 25;
+
 namespace sessions {
 class ScopedSessionContextConflictResolver;
 struct SyncSessionSnapshot;
@@ -76,6 +79,11 @@ class SyncSessionContext {
   }
   const std::string& account_name() { return account_name_; }
 
+  void set_max_commit_batch_size(int batch_size) {
+    max_commit_batch_size_ = batch_size;
+  }
+  int32 max_commit_batch_size() const { return max_commit_batch_size_; }
+
   const ModelSafeRoutingInfo& previous_session_routing_info() const {
     return previous_session_routing_info_;
   }
@@ -126,6 +134,9 @@ class SyncSessionContext {
   // The name of the account being synced.
   std::string account_name_;
 
+  // The server limits the number of items a client can commit in one batch.
+  int max_commit_batch_size_;
+
   // Some routing info history to help us clean up types that get disabled
   // by the user.
   ModelSafeRoutingInfo previous_session_routing_info_;