Sync: Add DeviceInfo's ChangeProcessor

This is the long-awaited change to enable DeviceInfo tracking.

Long ago (r161496) we committed code to support the DeviceInfo type, but
we had to leave this code disabled until the server was ready to support
the new type.  That support has now been added.

This change includes a very special kind of ChangeProcessor named
SyncedDeviceTracker.  It is the only ChangeProcessor that is owned by
the sync thread.  The SyncBackendHost creates and initializes it during
backend intialization and deletes it before the UserShare it references
is destroyed.  As part of its initialization, the SyncedDeviceTracker
will update the DeviceInfo entry for the current device.

The SyncedDeviceTracker could support sending notifications to any
interested listeners when new or update DeviceInfo information arrives,
but this functionality has not been implemented yet.

Also included are:
- Tests for the SyncedDeviceTracker
- A MockTransactionObserver to support these tests
- A cache_guid() accessor on the SyncManager, used to initialize the
  SyncedDeviceTracker
- Remove support for default constructing a DeviceInfo object

BUG=122825


Review URL: https://chromiumcodereview.appspot.com/11360259

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

1 files changed, 1 insertions, 5 deletions

diff --git a/sync/internal_api/sync_manager_impl.h b/sync/internal_api/sync_manager_impl.h
index 6accdab..de242be 100644
--- a/sync/internal_api/sync_manager_impl.h
+++ b/sync/internal_api/sync_manager_impl.h
@@ -68,7 +68,6 @@ class SyncManagerImpl :
       const std::string& sync_server_and_path,
       int sync_server_port,
       bool use_ssl,
-      const scoped_refptr<base::TaskRunner>& blocking_task_runner,
       scoped_ptr<HttpPostProviderFactory> post_factory,
       const std::vector<ModelSafeWorker*>& workers,
       ExtensionsActivityMonitor* extensions_activity_monitor,
@@ -111,6 +110,7 @@ class SyncManagerImpl :
   virtual void StopSyncingForShutdown(const base::Closure& callback) OVERRIDE;
   virtual void ShutdownOnSyncThread() OVERRIDE;
   virtual UserShare* GetUserShare() OVERRIDE;
+  virtual const std::string cache_guid() OVERRIDE;
   virtual bool ReceivedExperiment(Experiments* experiments) OVERRIDE;
   virtual bool HasUnsyncedItems() OVERRIDE;
   virtual SyncEncryptionHandler* GetEncryptionHandler() OVERRIDE;
@@ -302,10 +302,6 @@ class SyncManagerImpl :
   // WeakHandle when we construct it.
   WeakHandle<SyncManagerImpl> weak_handle_this_;
 
-  // |blocking_task_runner| is a TaskRunner to be used for tasks that
-  // may block on disk I/O.
-  scoped_refptr<base::TaskRunner> blocking_task_runner_;
-
   // We give a handle to share_ to clients of the API for use when constructing
   // any transaction type.
   UserShare share_;