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

Unified Diff: chrome/browser/sync/glue/app_notification_data_type_controller.cc

Issue 10071033: RefCounted types should not have public destructors, chrome/browser/ part 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Implementation ordering fixes as well Created 8 years, 8 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/app_notification_data_type_controller.cc
diff --git a/chrome/browser/sync/glue/app_notification_data_type_controller.cc b/chrome/browser/sync/glue/app_notification_data_type_controller.cc
index 4a94fdba647a6251849415c77a84a31eabcfc0cf..c9071105bfc7f9764cae81e47090bb1fc0c2a7f9 100644
--- a/chrome/browser/sync/glue/app_notification_data_type_controller.cc
+++ b/chrome/browser/sync/glue/app_notification_data_type_controller.cc
@@ -29,9 +29,6 @@ AppNotificationDataTypeController::AppNotificationDataTypeController(
sync_service) {
}
-AppNotificationDataTypeController::~AppNotificationDataTypeController() {
-}
-
void AppNotificationDataTypeController::Observe(
int type,
const content::NotificationSource& source,
@@ -44,6 +41,14 @@ void AppNotificationDataTypeController::Observe(
Associate();
}
+AppNotificationDataTypeController::~AppNotificationDataTypeController() {
+}
+
+AppNotificationManager*
+AppNotificationDataTypeController::GetAppNotificationManager() {
+ return profile_->GetExtensionService()->app_notification_manager();
+}
+
// We want to start the AppNotificationManager before we begin associating.
bool AppNotificationDataTypeController::StartModels() {
AppNotificationManager* manager = GetAppNotificationManager();
@@ -62,9 +67,4 @@ void AppNotificationDataTypeController::StopModels() {
registrar_.RemoveAll();
}
-AppNotificationManager*
-AppNotificationDataTypeController::GetAppNotificationManager() {
- return profile_->GetExtensionService()->app_notification_manager();
-}
-
} // namespace browser_sync

Powered by Google App Engine
This is Rietveld 408576698