[Sync] Parse and save per-data type throttle data

chrome/browser/sync/sessions/sync_session_context.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sync/sessions/sync_session_context.h"
 
 #include "chrome/browser/sync/sessions/debug_info_getter.h"
 #include "chrome/browser/sync/sessions/session_state.h"
 #include "chrome/browser/sync/util/extensions_activity_monitor.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 15 matching lines @@
       registrar_(model_safe_worker_registrar),
       extensions_activity_monitor_(new ExtensionsActivityMonitor()),
       notifications_enabled_(false),
       max_commit_batch_size_(kDefaultMaxCommitBatchSize),
       debug_info_getter_(debug_info_getter) {
   std::vector<SyncEngineEventListener*>::const_iterator it;
   for (it = listeners.begin(); it != listeners.end(); ++it)
     listeners_.AddObserver(*it);
 }
 
+SyncSessionContext::SyncSessionContext()
+    : connection_manager_(NULL),
+      directory_manager_(NULL),
+      registrar_(NULL),
+      extensions_activity_monitor_(NULL),
+      debug_info_getter_(NULL) {
+}
+
 SyncSessionContext::~SyncSessionContext() {
   // In unittests, there may be no UI thread, so the above will fail.
   if (!BrowserThread::DeleteSoon(BrowserThread::UI, FROM_HERE,
                                 extensions_activity_monitor_)) {
     delete extensions_activity_monitor_;
   }
 }
 
+void SyncSessionContext::AddUnthrottleTime(const syncable::ModelTypeSet& types,
+    const base::TimeTicks& time) {

[akalin, 2011/11/21 20:04:26]

indent

[lipalani1, 2011/11/21 21:55:38]

Done.

+  for (syncable::ModelTypeSet::const_iterator it = types.begin();
+       it != types.end();
+       ++it) {
+    DCHECK(unthrottle_times_.count(*it) == 0);

[akalin, 2011/11/21 20:04:26]

DCHECK_EQ

[lipalani1, 2011/11/21 21:55:38]

Actually removed it. I thought about this further and it seems if the server
throttles a new type we should not have the restriction for it to send down only
the new types.
On 2011/11/21 20:04:26, akalin wrote:

+    unthrottle_times_[*it] = time;
+  }
+}
+
 }  // namespace sessions
 }  // namespace browser_sync