Add appIconMask to notification API

chrome/common/extensions/api/notifications.idl

Index: chrome/common/extensions/api/notifications.idl
diff --git a/chrome/common/extensions/api/notifications.idl b/chrome/common/extensions/api/notifications.idl
index da95e62c95a0c86116d435fe282fdccc2045521c..412b2da43d02ddc38ee14dbe7453cb174213bdb7 100644
--- a/chrome/common/extensions/api/notifications.idl
+++ b/chrome/common/extensions/api/notifications.idl
@@ -53,11 +53,20 @@ namespace notifications {
     // <em>Required for $(ref:notifications.create)</em> method.
     TemplateType? type;
 
-    // Sender's avatar, app icon, or a thumbnail for image notifications.
+    // A URL to the sender's avatar, app icon, or a thumbnail for image
+    // notifications.

[not at google - send to devlin, 2014/07/21 20:50:15]

nit: separate these with a blank line so that they are detected as separate
paragraphs. That will look better in the generated documentation.

[liyanhou, 2014/07/22 00:05:22]

Done.

+    // URLs can be data URL, local URL, or blob URL.

[not at google - send to devlin, 2014/07/21 20:50:15]

Let me rephrase this sentence to make sure I understand:

This can be a data: URL, blob: URL, or a URL relative to a resource within this
extension's .crx file. It cannot be a URL to a remote resource.

(my main concern is the use of the word "local").

[liyanhou, 2014/07/22 00:05:22]

Done.

     // <em>Required for $(ref:notifications.create)</em> method.
     DOMString? iconUrl;
     [nodoc] NotificationBitmap? iconBitmap;
 
+    // A URL to the app icon mask. 
+    // URLs can be data URL, local URL, or blob URL.
+    // The app icon mask should be in alpha channel, as only the alpha channel
+    // of the image will be considered.

[not at google - send to devlin, 2014/07/21 20:50:15]

you could say here,

A URL to the app icon mask. URLs have the same restrictions as
$(ref:notifications.NotificationOptions.iconUrl iconUrl).

[liyanhou, 2014/07/22 00:05:22]

Done.

+    DOMString? appIconMaskUrl;
+    [nodoc] NotificationBitmap? appIconMaskBitmap;
+
     // Title of the notification (e.g. sender name for email).
     // <em>Required for $(ref:notifications.create)</em> method.
     DOMString? title;
@@ -83,7 +92,8 @@ namespace notifications {
     // Secondary notification content.
     [nodoc] DOMString? expandedMessage;
 
-    // Image thumbnail for image-type notifications.
+    // A URL to the image thumbnail for image-type notifications.
+    // URLs can be data URL, local URL, or blob URL.

[not at google - send to devlin, 2014/07/21 20:50:15]

likewise

[liyanhou, 2014/07/22 00:05:22]

Done.

     DOMString? imageUrl;
     [nodoc] NotificationBitmap? imageBitmap;