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

Unified Diff: chrome/common/extensions/api/notifications.idl

Issue 20136004: Allow partial update for notification update API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add more tests per feedback Created 7 years, 5 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/common/extensions/api/notifications.idl
diff --git a/chrome/common/extensions/api/notifications.idl b/chrome/common/extensions/api/notifications.idl
index 5e24848e1187d8ef5bba01e38b531968efea09e9..5808bc466dec01e193664a9254bf0d35d6a86a05 100644
--- a/chrome/common/extensions/api/notifications.idl
+++ b/chrome/common/extensions/api/notifications.idl
@@ -41,17 +41,17 @@ namespace notifications {
dictionary NotificationOptions {
// Which type of notification to display.
- TemplateType type;
+ TemplateType? type;
// Sender's avatar, app icon, or a thumbnail for image notifications.
- DOMString iconUrl;
+ DOMString? iconUrl;
[nodoc] NotificationBitmap? iconBitmap;
// Title of the notification (e.g. sender name for email).
- DOMString title;
+ DOMString? title;
// Main notification content.
- DOMString message;
+ DOMString? message;
// Priority ranges from -2 to 2. -2 is lowest priority. 2 is highest. Zero
// is default.

Powered by Google App Engine
This is Rietveld 408576698