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

Unified Diff: chrome/browser/notifier/chrome_notifier_service.h

Issue 11745024: Synced Notification Sync Change Processor (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Synced Notifications - CR changes per Zea and DCheng Created 7 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/notifier/chrome_notifier_service.h
diff --git a/chrome/browser/notifier/chrome_notifier_service.h b/chrome/browser/notifier/chrome_notifier_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..cce04bfb3d4b03fe7446f9f106e35777eab3e013
--- /dev/null
+++ b/chrome/browser/notifier/chrome_notifier_service.h
@@ -0,0 +1,78 @@
+// 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.
+
+#ifndef CHROME_BROWSER_NOTIFIER_CHROME_NOTIFIER_SERVICE_H_
+#define CHROME_BROWSER_NOTIFIER_CHROME_NOTIFIER_SERVICE_H_
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_vector.h"
+#include "base/threading/non_thread_safe.h"
+#include "chrome/browser/notifier/synced_notification.h"
+#include "chrome/browser/profiles/profile_keyed_service.h"
+#include "sync/api/syncable_service.h"
+
+class Profile;
+class NotificationUIManager;
+
+namespace notifier {
+
+// The ChromeNotifierService holds notifications which
+// represent the state of delivered notifications for chrome.
+// These are obtained from the sync service and kept up to date.
+class ChromeNotifierService : public syncer::SyncableService,
+ public ProfileKeyedService,
+ public base::NonThreadSafe {
+
+ public:
+ ChromeNotifierService(Profile* profile, NotificationUIManager* manager);
+ virtual ~ChromeNotifierService();
+
+ // Methods from ProfileKeyedService.
+ virtual void Shutdown() OVERRIDE;
+
+ // syncer::SyncableService implementation.
+ virtual syncer::SyncMergeResult MergeDataAndStartSyncing(
+ syncer::ModelType type,
+ const syncer::SyncDataList& initial_sync_data,
+ scoped_ptr<syncer::SyncChangeProcessor> sync_processor,
+ scoped_ptr<syncer::SyncErrorFactory> error_handler) OVERRIDE;
+ virtual void StopSyncing(syncer::ModelType type) OVERRIDE;
+ virtual syncer::SyncDataList GetAllSyncData(
+ syncer::ModelType type) const OVERRIDE;
+ virtual syncer::SyncError ProcessSyncChanges(
+ const tracked_objects::Location& from_here,
+ const syncer::SyncChangeList& change_list) OVERRIDE;
+
+ // Convert from internal representation to SyncData representation.
+ static syncer::SyncData CreateSyncDataFromNotification(
+ SyncedNotification& notification);
+
+ // Convert from SyncData representation to internal representation.
+ static SyncedNotification* CreateNotificationFromSyncData(
+ const syncer::SyncData& sync_data);
+
+ // Add a notification to our list. This takes ownership of the pointer.
+ void Add(scoped_ptr<notifier::SyncedNotification> notification);
+
+ void Show(notifier::SyncedNotification* notification);
+
+ // Get a pointer to a notification. ChromeNotifierService owns this pointer.
+ // The caller must not free the it.
+ notifier::SyncedNotification* FindNotificationById(const std::string& id);
+
+ private:
+ // Backpointer to the owning profile, do not free at shutdown.
+ Profile* profile_;
+ NotificationUIManager* notification_manager_;
+
+ // TODO(petewil): consider whether a map would better suit our data.
+ // If there are many entries, lookup time may trump locality of reference.
+ ScopedVector<notifier::SyncedNotification> notification_data_;
+
+ DISALLOW_COPY_AND_ASSIGN(ChromeNotifierService);
+};
+
+} // namespace notifier
+
+#endif // CHROME_BROWSER_NOTIFIER_CHROME_NOTIFIER_SERVICE_H_

Powered by Google App Engine
This is Rietveld 408576698