When an extension is uninstalled, close all desktop notifications from that e...

chrome/browser/notifications/balloon_collection_base.h

Index: chrome/browser/notifications/balloon_collection_base.h
===================================================================
--- chrome/browser/notifications/balloon_collection_base.h	(revision 0)
+++ chrome/browser/notifications/balloon_collection_base.h	(revision 0)
@@ -0,0 +1,61 @@
+// Copyright (c) 2010 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.
+
+// Handles the visible notification (or balloons).
+
+#ifndef CHROME_BROWSER_NOTIFICATIONS_BALLOON_COLLECTION_BASE_H_
+#define CHROME_BROWSER_NOTIFICATIONS_BALLOON_COLLECTION_BASE_H_
+#pragma once
+
+#include <deque>
+#include <string>
+
+#include "base/stl_util-inl.h"

[tfarina, 2010/11/11 12:56:58]

nit: do you need this include here? Can't it be included only in the .cc files?

[John Gregg, 2010/11/11 18:13:51]

Done.

+#include "chrome/browser/notifications/balloon.h"
+
+class GURL;
+
+// This class implements a basic BalloonCollection with the parts
+// shared between Chrome UI and ChromeOS UI.  Some methods are
+// still pure virtual which need to be implemented by the particular

[tfarina, 2010/11/11 12:56:58]

That is not true. Pure virtual means "virtual ... = 0;" but there is not such
method like this in this class.

[John Gregg, 2010/11/11 18:13:51]

yep, stale comment all around from when i was inheriting instead of composing.
fixed.

+// UI implementation.
+class BalloonCollectionBase {
+ public:
+  BalloonCollectionBase();
+  virtual ~BalloonCollectionBase();
+
+  typedef std::deque<Balloon*> Balloons;
+
+  // Adds a balloon to the collection. Takes ownership of pointer.
+  virtual void Add(Balloon* balloon);
+
+  // Removes a balloon from the collection (if present).  Frees
+  // the pointer after removal.
+  virtual void Remove(Balloon* balloon);
+
+  // Finds any balloon matching the given notification id, and
+  // calls CloseByScript on it.
+  virtual bool CloseById(const std::string& id);
+
+  // Removes all balloons matching the given notification source,
+  // and calls CloseByScript on them.
+  virtual bool CloseAllBySourceOrigin(const GURL& source_origin);
+
+  const Balloons& balloons() const { return balloons_; }
+
+  // Returns the balloon matching the given notification.
+  Balloon* FindBalloon(const Notification& notification);
+
+  // The number of balloons being displayed.
+  int count() const { return balloons_.size(); }

[tfarina, 2010/11/11 12:56:58]

nit: static_cast<int>

[John Gregg, 2010/11/11 18:13:51]

Done.

+
+ private:
+  // Queue of active balloons.  These pointers are non-owned, and
+  // must be managed by the caller of this class.
+  Balloons balloons_;
+
+  DISALLOW_COPY_AND_ASSIGN(BalloonCollectionBase);
+};
+
+#endif  // CHROME_BROWSER_NOTIFICATIONS_BALLOON_COLLECTION_BASE_H_