Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5223)

Unified Diff: chrome/browser/sync/glue/new_non_frontend_data_type_controller.h

Issue 9264062: [Sync] Consolidate DataTypeController methods that post on the backend thread (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add suppressions Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/glue/new_non_frontend_data_type_controller.h
diff --git a/chrome/browser/sync/glue/new_non_frontend_data_type_controller.h b/chrome/browser/sync/glue/new_non_frontend_data_type_controller.h
index 915181ef5d394fcba0c5afb062ed163dae59263e..64e65d6537af4460a74cbf0bfbe1a213d92f972e 100644
--- a/chrome/browser/sync/glue/new_non_frontend_data_type_controller.h
+++ b/chrome/browser/sync/glue/new_non_frontend_data_type_controller.h
@@ -38,20 +38,20 @@ class NewNonFrontendDataTypeController : public NonFrontendDataTypeController {
DataTypeController::State new_state,
const SyncError& error) OVERRIDE;
- // Calls local_service_->StopSyncing() and releases our references to it and
- // |shared_change_processor_|.
- virtual void StopLocalService();
- // Posts StopLocalService() to the datatype's thread.
- virtual void StopLocalServiceAsync() = 0;
-
// Extract/create the syncable service from the profile and return a
// WeakHandle to it.
virtual base::WeakPtr<SyncableService> GetWeakPtrToSyncableService()
const = 0;
private:
+ // Posts StopLocalService() to the datatype's thread.
+ void StopLocalServiceAsync();
+
+ // Calls local_service_->StopSyncing() and releases our references to it and
+ // |shared_change_processor_|.
+ void StopLocalService();
+
// Deprecated.
- virtual bool StopAssociationAsync() OVERRIDE;
virtual void CreateSyncComponents() OVERRIDE;
// A weak pointer to the actual local syncable service, which performs all the

Powered by Google App Engine
This is Rietveld 408576698