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

chrome/browser/extensions/app_notification_manager.h

Index: chrome/browser/extensions/app_notification_manager.h
===================================================================
--- chrome/browser/extensions/app_notification_manager.h	(revision 109957)
+++ chrome/browser/extensions/app_notification_manager.h	(working copy)
@@ -21,6 +21,10 @@
 #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.
@@ -130,6 +134,9 @@
   // 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);