sync: Remove SyncManager::TestingMode in favour of InternalComponentsFactory.

sync/engine/sync_scheduler.h

Index: sync/engine/sync_scheduler.h
diff --git a/sync/engine/sync_scheduler.h b/sync/engine/sync_scheduler.h
index 0ca2f5db5a14be3d236213750e9489eda6a919f6..a6288efa7cf810a286db4f5910fed8ad1d8c5256 100644
--- a/sync/engine/sync_scheduler.h
+++ b/sync/engine/sync_scheduler.h
@@ -104,22 +104,22 @@ class SyncScheduler : public sessions::SyncSession::Delegate {
   // allow preempting ongoing sync cycles.
   // Invokes |callback| from the sync loop once syncer is idle and all tasks
   // are cancelled.
-  void RequestStop(const base::Closure& callback);
+  virtual void RequestStop(const base::Closure& callback);

[Nicolas Zea, 2012/07/18 21:10:12]

Do I dare wonder if this should be an interface?

 
   // The meat and potatoes. Both of these methods will post a delayed task
   // to attempt the actual nudge (see ScheduleNudgeImpl).
-  void ScheduleNudgeAsync(const base::TimeDelta& delay, NudgeSource source,
-                          syncer::ModelTypeSet types,
-                          const tracked_objects::Location& nudge_location);
-  void ScheduleNudgeWithPayloadsAsync(
+  virtual void ScheduleNudgeAsync(
+      const base::TimeDelta& delay,
+      NudgeSource source,
+      syncer::ModelTypeSet types,
+      const tracked_objects::Location& nudge_location);
+  virtual void ScheduleNudgeWithPayloadsAsync(
       const base::TimeDelta& delay, NudgeSource source,
       const syncer::ModelTypePayloadMap& types_with_payloads,
       const tracked_objects::Location& nudge_location);
 
-  void CleanupDisabledTypes();
-
   // Change status of notifications in the SyncSessionContext.
-  void set_notifications_enabled(bool notifications_enabled);
+  virtual void SetNotificationsEnabled(bool notifications_enabled);
 
   base::TimeDelta sessions_commit_delay() const;