RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/app_notification_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/app_notification_manager.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/file_path.h"
 #include "base/location.h"
@@ skipping 56 matching lines @@
 
 AppNotificationManager::AppNotificationManager(Profile* profile)
     : profile_(profile),
       models_associated_(false),
       processing_syncer_changes_(false) {
   registrar_.Add(this,
                  chrome::NOTIFICATION_EXTENSION_UNINSTALLED,
                  content::Source<Profile>(profile_));
 }
 
-AppNotificationManager::~AppNotificationManager() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  // Post a task to delete our storage on the file thread.
-  BrowserThread::DeleteSoon(BrowserThread::FILE,
-                            FROM_HERE,
-                            storage_.release());
-}
-
 void AppNotificationManager::Init() {
   FilePath storage_path = profile_->GetPath().AppendASCII("App Notifications");
   load_timer_.reset(new PerfTimer());
   BrowserThread::PostTask(
       BrowserThread::FILE,
       FROM_HERE,
       base::Bind(&AppNotificationManager::LoadOnFileThread,
           this, storage_path));
 }
 
@@ skipping 40 matching lines @@
 
 const AppNotificationList* AppNotificationManager::GetAll(
     const std::string& extension_id) const {
   if (!loaded())
     return NULL;
   if (ContainsKey(*notifications_, extension_id))
     return &((*notifications_)[extension_id]);
   return NULL;
 }
 
-AppNotificationList& AppNotificationManager::GetAllInternal(
-    const std::string& extension_id) {
-  NotificationMap::iterator found = notifications_->find(extension_id);
-  if (found == notifications_->end()) {
-    (*notifications_)[extension_id] = AppNotificationList();
-    found = notifications_->find(extension_id);
-  }
-  CHECK(found != notifications_->end());
-  return found->second;
-}
-
 const AppNotification* AppNotificationManager::GetLast(
     const std::string& extension_id) {
   if (!loaded())
     return NULL;
   NotificationMap::iterator found = notifications_->find(extension_id);
   if (found == notifications_->end())
     return NULL;
   const AppNotificationList& list = found->second;
   if (list.empty())
     return NULL;
@@ skipping 24 matching lines @@
 }
 
 void AppNotificationManager::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   CHECK(type == chrome::NOTIFICATION_EXTENSION_UNINSTALLED);
   ClearAll(*content::Details<const std::string>(details).ptr());
 }
 
-void AppNotificationManager::LoadOnFileThread(const FilePath& storage_path) {
-  PerfTimer timer;
-  CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  DCHECK(!loaded());
-
-  storage_.reset(AppNotificationStorage::Create(storage_path));
-  if (!storage_.get())
-    return;
-  scoped_ptr<NotificationMap> result(new NotificationMap());
-  std::set<std::string> ids;
-  if (!storage_->GetExtensionIds(&ids))
-    return;
-  std::set<std::string>::const_iterator i;
-  for (i = ids.begin(); i != ids.end(); ++i) {
-    const std::string& id = *i;
-    AppNotificationList& list = (*result)[id];
-    if (!storage_->Get(id, &list))
-      result->erase(id);
-  }
-
-  BrowserThread::PostTask(
-      BrowserThread::UI,
-      FROM_HERE,
-      base::Bind(&AppNotificationManager::HandleLoadResults,
-          this, result.release()));
-
-  UMA_HISTOGRAM_LONG_TIMES("AppNotification.MgrFileThreadLoadTime",
-                           timer.Elapsed());
-}
-
-void AppNotificationManager::HandleLoadResults(NotificationMap* map) {
-  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  DCHECK(map);
-  DCHECK(!loaded());
-  notifications_.reset(map);
-  UMA_HISTOGRAM_LONG_TIMES("AppNotification.MgrLoadDelay",
-                           load_timer_->Elapsed());
-  load_timer_.reset();
-
-  // Generate STATE_CHANGED notifications for extensions that have at
-  // least one notification loaded.
-  int app_count = 0;
-  int notification_count = 0;
-  NotificationMap::const_iterator i;
-  for (i = map->begin(); i != map->end(); ++i) {
-    const std::string& id = i->first;
-    if (i->second.empty())
-      continue;
-    app_count++;
-    notification_count += i->second.size();
-    content::NotificationService::current()->Notify(
-        chrome::NOTIFICATION_APP_NOTIFICATION_STATE_CHANGED,
-        content::Source<Profile>(profile_),
-        content::Details<const std::string>(&id));
-  }
-  UMA_HISTOGRAM_COUNTS("AppNotification.MgrLoadAppCount", app_count);
-  UMA_HISTOGRAM_COUNTS("AppNotification.MgrLoadTotalCount",
-                       notification_count);
-
-  // Generate MANAGER_LOADED notification.
-  content::NotificationService::current()->Notify(
-      chrome::NOTIFICATION_APP_NOTIFICATION_MANAGER_LOADED,
-      content::Source<AppNotificationManager>(this),
-      content::NotificationService::NoDetails());
-}
-
-void AppNotificationManager::SaveOnFileThread(const std::string& extension_id,
-                                              AppNotificationList* list) {
-  // Own the |list|.
-  scoped_ptr<AppNotificationList> scoped_list(list);
-  CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  storage_->Set(extension_id, *scoped_list);
-}
-
-void AppNotificationManager::DeleteOnFileThread(
-    const std::string& extension_id) {
-  CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  storage_->Delete(extension_id);
-}
-
 SyncDataList AppNotificationManager::GetAllSyncData(
     syncable::ModelType type) const {
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(loaded());
   DCHECK_EQ(syncable::APP_NOTIFICATIONS, type);
   SyncDataList data;
   for (NotificationMap::const_iterator iter = notifications_->begin();
       iter != notifications_->end(); ++iter) {
 
     // Skip local notifications since they should not be synced.
@@ skipping 156 matching lines @@
   return error;
 }
 
 void AppNotificationManager::StopSyncing(syncable::ModelType type) {
   DCHECK_EQ(type, syncable::APP_NOTIFICATIONS);
   models_associated_ = false;
   sync_processor_.reset();
   sync_error_factory_.reset();
 }
 
+AppNotificationManager::~AppNotificationManager() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  // Post a task to delete our storage on the file thread.
+  BrowserThread::DeleteSoon(BrowserThread::FILE,
+                            FROM_HERE,
+                            storage_.release());
+}
+
+void AppNotificationManager::LoadOnFileThread(const FilePath& storage_path) {
+  PerfTimer timer;
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+  DCHECK(!loaded());
+
+  storage_.reset(AppNotificationStorage::Create(storage_path));
+  if (!storage_.get())
+    return;
+  scoped_ptr<NotificationMap> result(new NotificationMap());
+  std::set<std::string> ids;
+  if (!storage_->GetExtensionIds(&ids))
+    return;
+  std::set<std::string>::const_iterator i;
+  for (i = ids.begin(); i != ids.end(); ++i) {
+    const std::string& id = *i;
+    AppNotificationList& list = (*result)[id];
+    if (!storage_->Get(id, &list))
+      result->erase(id);
+  }
+
+  BrowserThread::PostTask(
+      BrowserThread::UI,
+      FROM_HERE,
+      base::Bind(&AppNotificationManager::HandleLoadResults,
+          this, result.release()));
+
+  UMA_HISTOGRAM_LONG_TIMES("AppNotification.MgrFileThreadLoadTime",
+                           timer.Elapsed());
+}
+
+void AppNotificationManager::HandleLoadResults(NotificationMap* map) {
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  DCHECK(map);
+  DCHECK(!loaded());
+  notifications_.reset(map);
+  UMA_HISTOGRAM_LONG_TIMES("AppNotification.MgrLoadDelay",
+                           load_timer_->Elapsed());
+  load_timer_.reset();
+
+  // Generate STATE_CHANGED notifications for extensions that have at
+  // least one notification loaded.
+  int app_count = 0;
+  int notification_count = 0;
+  NotificationMap::const_iterator i;
+  for (i = map->begin(); i != map->end(); ++i) {
+    const std::string& id = i->first;
+    if (i->second.empty())
+      continue;
+    app_count++;
+    notification_count += i->second.size();
+    content::NotificationService::current()->Notify(
+        chrome::NOTIFICATION_APP_NOTIFICATION_STATE_CHANGED,
+        content::Source<Profile>(profile_),
+        content::Details<const std::string>(&id));
+  }
+  UMA_HISTOGRAM_COUNTS("AppNotification.MgrLoadAppCount", app_count);
+  UMA_HISTOGRAM_COUNTS("AppNotification.MgrLoadTotalCount",
+                       notification_count);
+
+  // Generate MANAGER_LOADED notification.
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_APP_NOTIFICATION_MANAGER_LOADED,
+      content::Source<AppNotificationManager>(this),
+      content::NotificationService::NoDetails());
+}
+
+void AppNotificationManager::SaveOnFileThread(const std::string& extension_id,
+                                              AppNotificationList* list) {
+  // Own the |list|.
+  scoped_ptr<AppNotificationList> scoped_list(list);
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+  storage_->Set(extension_id, *scoped_list);
+}
+
+void AppNotificationManager::DeleteOnFileThread(
+    const std::string& extension_id) {
+  CHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+  storage_->Delete(extension_id);
+}
+
+AppNotificationList& AppNotificationManager::GetAllInternal(
+    const std::string& extension_id) {
+  NotificationMap::iterator found = notifications_->find(extension_id);
+  if (found == notifications_->end()) {
+    (*notifications_)[extension_id] = AppNotificationList();
+    found = notifications_->find(extension_id);
+  }
+  CHECK(found != notifications_->end());
+  return found->second;
+}
+
+void AppNotificationManager::Remove(const std::string& extension_id,
+                                    const std::string& guid) {
+  DCHECK(loaded());
+  AppNotificationList& list = GetAllInternal(extension_id);
+  RemoveByGuid(&list, guid);
+
+  if (storage_.get()) {
+    BrowserThread::PostTask(
+        BrowserThread::FILE,
+        FROM_HERE,
+        base::Bind(&AppNotificationManager::SaveOnFileThread,
+            this, extension_id, CopyAppNotificationList(list)));
+  }
+
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_APP_NOTIFICATION_STATE_CHANGED,
+      content::Source<Profile>(profile_),
+      content::Details<const std::string>(&extension_id));
+}
+
+const AppNotification* AppNotificationManager::GetNotification(
+    const std::string& extension_id, const std::string& guid) {
+  DCHECK(loaded());
+  const AppNotificationList& list = GetAllInternal(extension_id);
+  return FindByGuid(list, guid);
+}
+
 void AppNotificationManager::SyncAddChange(const AppNotification& notif) {
   // Skip if either:
   // - Notification is marked as local.
   // - Sync is not enabled by user.
   // - Change is generated from within the manager.
   if (notif.is_local() || !models_associated_ || processing_syncer_changes_)
     return;
 
   // TODO(munjal): crbug.com/10059. Work with Lingesh/Antony to resolve.
 
@@ skipping 32 matching lines @@
     // Skip notifications marked as local.
     if (notif.is_local())
       continue;
     changes.push_back(SyncChange(
         SyncChange::ACTION_DELETE,
         CreateSyncDataFromNotification(notif)));
   }
   sync_processor_->ProcessSyncChanges(FROM_HERE, changes);
 }
 
-const AppNotification* AppNotificationManager::GetNotification(
-    const std::string& extension_id, const std::string& guid) {
-  DCHECK(loaded());
-  const AppNotificationList& list = GetAllInternal(extension_id);
-  return FindByGuid(list, guid);
-}
-
-void AppNotificationManager::Remove(const std::string& extension_id,
-                                    const std::string& guid) {
-  DCHECK(loaded());
-  AppNotificationList& list = GetAllInternal(extension_id);
-  RemoveByGuid(&list, guid);
-
-  if (storage_.get()) {
-    BrowserThread::PostTask(
-        BrowserThread::FILE,
-        FROM_HERE,
-        base::Bind(&AppNotificationManager::SaveOnFileThread,
-            this, extension_id, CopyAppNotificationList(list)));
-  }
-
-  content::NotificationService::current()->Notify(
-      chrome::NOTIFICATION_APP_NOTIFICATION_STATE_CHANGED,
-      content::Source<Profile>(profile_),
-      content::Details<const std::string>(&extension_id));
-}
-
 // static
 SyncData AppNotificationManager::CreateSyncDataFromNotification(
     const AppNotification& notification) {
   DCHECK(!notification.is_local());
   sync_pb::EntitySpecifics specifics;
   sync_pb::AppNotification* notif_specifics =
       specifics.mutable_app_notification();
   notif_specifics->set_app_id(notification.extension_id());
   notif_specifics->set_creation_timestamp_ms(
       notification.creation_time().ToInternalValue());
@@ skipping 22 matching lines @@
   AppNotification* notification = new AppNotification(
       false, base::Time::FromInternalValue(specifics.creation_timestamp_ms()),
       specifics.guid(), specifics.app_id(),
       specifics.title(), specifics.body_text());
   if (specifics.has_link_text())
     notification->set_link_text(specifics.link_text());
   if (specifics.has_link_url())
     notification->set_link_url(GURL(specifics.link_url()));
   return notification;
 }