[Sync] Refactor non-frontend DTC to handle new API properly.

chrome/browser/sync/api/syncable_service.h

Index: chrome/browser/sync/api/syncable_service.h
diff --git a/chrome/browser/sync/api/syncable_service.h b/chrome/browser/sync/api/syncable_service.h
index 9d7ae621b7b8cb534a7e2d7b7660a5c99784d81e..5af835c347a9a64ccd798dc9c1c04d097ca7d4f7 100644
--- a/chrome/browser/sync/api/syncable_service.h
+++ b/chrome/browser/sync/api/syncable_service.h
@@ -9,6 +9,7 @@
 #include <vector>
 
 #include "base/compiler_specific.h"
+#include "base/memory/weak_ptr.h"
 #include "chrome/browser/sync/syncable/model_type.h"
 #include "chrome/browser/sync/api/sync_change_processor.h"
 #include "chrome/browser/sync/api/sync_data.h"
@@ -18,7 +19,8 @@ class SyncData;
 
 typedef std::vector<SyncData> SyncDataList;
 
-class SyncableService : public SyncChangeProcessor {
+class SyncableService : public SyncChangeProcessor,
+                        public base::SupportsWeakPtr<SyncableService> {

[akalin, 2011/10/11 22:41:08]

I don't like making this interface a non-interface (since SupportsWeakPtr adds
methods), but we can debate it later.  Can you make
AutofillProfileSyncableService inherit from SupportsWeakPtr for now?  I think
with my other comments, it's doable.

[Nicolas Zea, 2011/10/12 04:24:19]

Done, and changed all implementations of SyncableService to additionally
implement SupportsWeakPtr.

  public:
   // Informs the service to begin syncing the specified synced datatype |type|.
   // The service should then merge |initial_sync_data| into it's local data,