Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: include/AdblockPlus/FilterEngine.h

Issue 5797488346791936: Issue 1107 - Support notifications (Closed)
Patch Set: C++03 Created Jan. 23, 2015, 10:52 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: include/AdblockPlus/FilterEngine.h
diff --git a/include/AdblockPlus/FilterEngine.h b/include/AdblockPlus/FilterEngine.h
index b81433f5c9bf1b540cf837f6df1a1662bc623454..153b221fbc7b47c45e6388b290b777b2c4d31835 100644
--- a/include/AdblockPlus/FilterEngine.h
+++ b/include/AdblockPlus/FilterEngine.h
@@ -24,6 +24,7 @@
#include <vector>
#include <AdblockPlus/JsEngine.h>
#include <AdblockPlus/JsValue.h>
+#include <AdblockPlus/Notification.h>
#include "tr1_memory.h"
@@ -237,6 +238,14 @@ namespace AdblockPlus
std::vector<SubscriptionPtr> FetchAvailableSubscriptions() const;
/**
+ * Determines which notification is to be shown next.
+ * @param url URL to match notifications to (optional).
+ * @return Notification to be shown, or null if there is no any.
Felix Dahlke 2015/01/23 13:25:55 Can you add backticks around null? (`null`). That
+ */
+ std::tr1::shared_ptr<Notification> GetNextNotificationToShow(
Felix Dahlke 2015/01/23 13:25:55 Why not use NotificationPtr here?
+ const std::string& url = std::string());
+
+ /**
* Checks if any active filter matches the supplied URL.
* @param url URL to match.
* @param contentType Content type of the requested resource.

Powered by Google App Engine
This is Rietveld