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

Unified Diff: lib/api.js

Issue 5797488346791936: Issue 1107 - Support notifications (Closed)
Patch Set: move MarkAsShown into Notification class and get rid of locale arg in Notification::GetTexts Created Jan. 22, 2015, 10:02 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: lib/api.js
diff --git a/lib/api.js b/lib/api.js
index b294274d51ce5053485fa0eddd109e51d50cb802..d957761121457e63791db8a0e06c75ede6ae9e38 100644
--- a/lib/api.js
+++ b/lib/api.js
@@ -26,6 +26,7 @@ var API = (function()
var Synchronizer = require("synchronizer").Synchronizer;
var Prefs = require("prefs").Prefs;
var checkForUpdates = require("updater").checkForUpdates;
+ var Notification = require("notification").Notification;
return {
getFilterFromText: function(text)
@@ -136,6 +137,30 @@ var API = (function()
return result;
},
+ getNextNotificationToShow: function(url)
+ {
+ return Notification.getNextToShow(url);
+ },
+
+ getNotificationTexts: function(notification)
+ {
+ return Notification.getLocalizedTexts(notification);
+ },
+
+ addNotification: function(notification)
+ {
+ return Notification.addNotification(notification);
+ },
+
+ removeNotification: function(notification)
+ {
+ return Notification.removeNotification(notification);
+ },
+
+ markNotificationAsShown: function(id)
+ {
+ Notification.markAsShown(id);
+ },
checkFilterMatch: function(url, contentType, documentUrl)
{
var requestHost = extractHostFromURL(url);

Powered by Google App Engine
This is Rietveld