Add appIconMask to notification API

chrome/common/extensions/api/notifications.idl

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Use the <code>chrome.notifications</code> API to create rich notifications
 // using templates and show these notifications to users in the system tray.
 namespace notifications {
   [noinline_doc] enum TemplateType {
     // icon, title, message, expandedMessage, up to two buttons
     basic,
@@ skipping 40 matching lines @@
   dictionary NotificationOptions {
     // Which type of notification to display.
     // <em>Required for $(ref:notifications.create)</em> method.
     TemplateType? type;
 
     // Sender's avatar, app icon, or a thumbnail for image notifications.
     // <em>Required for $(ref:notifications.create)</em> method.
     DOMString? iconUrl;
     [nodoc] NotificationBitmap? iconBitmap;
 
+    // The alpha channel of app icon mask will be masked with white
+    // foreground and grey background, displayed on the bottom right corner
+    // of a notification

[not at google - send to devlin, 2014/07/15 15:17:54]

please phrase this comment in terms of the property, which is a noun. see
comments on all of these other properties as examples.

i'd also like to see what this URL actually is. a remote URL? a local URL? data
URL? same goes for all of the other comments on URL images (the iconUrls,
imageUrl). the comments of those could be better really; rather than "Sender's
avatar..." it should be "A URL to the sender's avatar...", with a second
paragraph "URLs can be ...".

and the comment should end with a period.

[liyanhou, 2014/07/21 16:51:34]

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;
 
     // Main notification content.
     // <em>Required for $(ref:notifications.create)</em> method.
     DOMString? message;
 
     // Alternate notification content with a lower-weight font.
     DOMString? contextMessage;
@@ skipping 86 matching lines @@
     static void onButtonClicked(DOMString notificationId, long buttonIndex);
 
     // The user changes the permission level.
     static void onPermissionLevelChanged(PermissionLevel level);
 
     // The user clicked on a link for the app's notification settings.
     static void onShowSettings();
   };
 
 };