sync: Finish the SyncScheduler refactor

sync/engine/nudge_tracker.cc

+// Copyright (c) 2013 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 "sync/engine/nudge_tracker.h"
+#include "sync/protocol/sync.pb.h"
+
+namespace syncer {
+
+NudgeTracker::NudgeTracker() { }
+
+NudgeTracker::~NudgeTracker() { }
+
+void NudgeTracker::CoalesceSources(const sessions::SyncSourceInfo& source) {
+  CoalesceStates(source.types, &source_info_.types);
+  source_info_.updates_source = source.updates_source;
+}
+
+bool NudgeTracker::IsEmpty() {
+  return source_info_.types.empty();
+}
+
+void NudgeTracker::Reset() {
+  source_info_ = sessions::SyncSourceInfo();
+}
+
+// TODO(rlarocque): This function often reports incorrect results.  However, it

[tim (not reviewing), 2013/04/15 22:37:49]

Is there a bug you can reference that details what we mean by 'incorrect', and
that tracks ongoing work here?

[rlarocque, 2013/04/16 01:30:26]

I created crbug.com/231693, which is sort of related.  We should be able to fix
this TODO when we fix that bug, assuming we want this function to return correct
results.

+// is compatible with the old behaviour.  We would need to make the nudge
+// tracker stop overwriting its own information if we wanted to track this
+// properly and return correct results.
+ModelTypeSet NudgeTracker::GetLocallyModifiedTypes() const {
+  ModelTypeSet locally_modified;
+
+  if (source_info_.updates_source != sync_pb::GetUpdatesCallerInfo::LOCAL) {
+    return locally_modified;
+  }
+
+  for (ModelTypeInvalidationMap::const_iterator i = source_info_.types.begin();
+       i != source_info().types.end(); ++i) {
+    locally_modified.Put(i->first);
+  }
+  return locally_modified;
+}
+
+}  // namespace syncer