Limit number of notifications that can be received by the client.

chrome/browser/extensions/app_notification_manager.h

 // Copyright (c) 2011 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_EXTENSIONS_APP_NOTIFICATION_MANAGER_H_
 #define CHROME_BROWSER_EXTENSIONS_APP_NOTIFICATION_MANAGER_H_
 #pragma once
 
 #include <map>
 
 #include "base/compiler_specific.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/synchronization/waitable_event.h"
 #include "chrome/browser/extensions/app_notification.h"
 #include "chrome/browser/extensions/app_notification_storage.h"
 #include "chrome/browser/sync/api/sync_change.h"
 #include "chrome/browser/sync/api/syncable_service.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
+namespace notification_manager_constants {
+  static const unsigned int kMaxNotificationPerApp = 5;

[asargent_no_longer_on_chrome, 2011/11/17 23:40:14]

nit: We typically use int's instead of unsigned int's for constants, and we
often just put constants inside the class instead of a namespace, unless there
are a huge pile of them that deserve to be in their own separate file. 

So in the .h:

class AppNotificationManager ... {
 public:
  static const int kMaxNotificationPerApp;

  ...
};

and then define it inside the .cc file

const int AppNotificationManager::kMaxNotificationPerApp = 5;

[elvin, 2011/11/18 00:10:18]

Done, kept it unsigned as size() is unsigned?
On 2011/11/17 23:40:14, Antony Sargent wrote:

+}
+
 class Profile;
 
 // This class keeps track of notifications for installed apps.
 class AppNotificationManager
     : public base::RefCountedThreadSafe<AppNotificationManager>,
       public content::NotificationObserver,
       public SyncableService {
  public:
   explicit AppNotificationManager(Profile* profile);
   virtual ~AppNotificationManager();
@@ skipping 89 matching lines @@
   void Remove(const std::string& extension_id, const std::string& guid);
 
   // Returns the notification for the given |extension_id| and |guid|,
   // NULL if no such notification exists.
   const AppNotification* GetNotification(const std::string& extension_id,
                                          const std::string& guid);
 
   // Sends a change to syncer to add the given notification.
   void SyncAddChange(const AppNotification& notif);
 
+  // Sends a change to syncer to remove the given notification.
+  void SyncRemoveChange(const AppNotification& notif);
+
   // Sends changes to syncer to remove all notifications in the given list.
   void SyncClearAllChange(const AppNotificationList& list);
 
   // Converters from AppNotification to SyncData and vice versa.
   static SyncData CreateSyncDataFromNotification(
       const AppNotification& notification);
   static AppNotification* CreateNotificationFromSyncData(
       const SyncData& sync_data);
 
   Profile* profile_;
   content::NotificationRegistrar registrar_;
   scoped_ptr<NotificationMap> notifications_;
 
   // This should only be used on the FILE thread.
   scoped_ptr<AppNotificationStorage> storage_;
 
   // Sync change processor we use to push all our changes.
   SyncChangeProcessor* sync_processor_;
   // Whether the sync model is associated with the local model.
   // In other words, whether we are ready to apply sync changes.
   bool models_associated_;
   // Whether syncer changes are being processed right now.
   bool processing_syncer_changes_;
 
   DISALLOW_COPY_AND_ASSIGN(AppNotificationManager);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_APP_NOTIFICATION_MANAGER_H_