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

Unified Diff: chrome/browser/notifications/desktop_notification_service.h

Issue 10824265: Correctly remove file browser notifications from Ash notification tray. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 8 years, 4 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/notifications/desktop_notification_service.h
diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h
index f6b50ebcfd50baadc8840e4d53d8bf2839c1152d..5b14e600bbc7ba8d88123ab95eedb49b8fc5431d 100644
--- a/chrome/browser/notifications/desktop_notification_service.h
+++ b/chrome/browser/notifications/desktop_notification_service.h
@@ -103,16 +103,21 @@ class DesktopNotificationService : public content::NotificationObserver,
const string16& title,
const string16& message,
const GURL& icon_url,
+ const string16& replace_id,
NotificationDelegate* delegate,
Profile* profile);
// Same as above, but takes a gfx::ImageSkia for the icon instead.
static std::string AddIconNotification(const GURL& origin_url,
- const string16& title,
- const string16& message,
- const gfx::ImageSkia& icon,
- NotificationDelegate* delegate,
- Profile* profile);
+ const string16& title,
+ const string16& message,
+ const gfx::ImageSkia& icon,
+ const string16& replace_id,
+ NotificationDelegate* delegate,
+ Profile* profile);
+
+ // Remove any active notification corresponding to |notification_id|.
+ static void RemoveNotification(const std::string& notification_id);
// The default content setting determines how to handle origins that haven't
// been allowed or denied yet. If |provider_id| is not NULL, the id of the

Powered by Google App Engine
This is Rietveld 408576698