Added Mac to list of supported platforms for chrome.notifications.

Added Mac to list of supported platforms for chrome.notifications.

BUG=227004
UPDATED DOC: https://chrome-apps-doc.appspot.com/_patch/23617008/apps/notifications.html

R=kalman@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220419

[not at google - send to devlin, 2013-08-28 21:43:11]

https://codereview.chromium.org/23617008/diff/1/chrome/common/extensions/docs...
File chrome/common/extensions/docs/templates/intros/notifications.html (right):

https://codereview.chromium.org/23617008/diff/1/chrome/common/extensions/docs...
chrome/common/extensions/docs/templates/intros/notifications.html:3: Currently
this API only works on ChromeOS, Windows, and Mac.
This just leaves Linux, right? Does it make sense to say "This API does not work
on Linux"?

[not at google - send to devlin, 2013-08-28 21:45:12]

https://codereview.chromium.org/23617008/diff/1/chrome/common/extensions/docs...
File chrome/common/extensions/docs/templates/intros/notifications.html (right):

https://codereview.chromium.org/23617008/diff/1/chrome/common/extensions/docs...
chrome/common/extensions/docs/templates/intros/notifications.html:3: Currently
this API only works on ChromeOS, Windows, and Mac.
On 2013/08/28 21:43:16, kalman wrote:
> This just leaves Linux, right? Does it make sense to say "This API does not
work
> on Linux"?

Maybe not; since it only recently works on mac. Maybe it should say "this API is
only available on ChromeOS, Windows, Mac (Chrome 31 and later)".

[somast1, 2013-08-28 22:25:27]

On 2013/08/28 21:45:12, kalman wrote:
>
https://codereview.chromium.org/23617008/diff/1/chrome/common/extensions/docs...
> File chrome/common/extensions/docs/templates/intros/notifications.html
(right):
> 
>
https://codereview.chromium.org/23617008/diff/1/chrome/common/extensions/docs...
> chrome/common/extensions/docs/templates/intros/notifications.html:3: Currently
> this API only works on ChromeOS, Windows, and Mac.
> On 2013/08/28 21:43:16, kalman wrote:
> > This just leaves Linux, right? Does it make sense to say "This API does not
> work
> > on Linux"?
> 
> Maybe not; since it only recently works on mac. Maybe it should say "this API
is
> only available on ChromeOS, Windows, Mac (Chrome 31 and later)".

That's not correct. It was enabled on Mac in M29 (went to stable last week:
http://chrome.blogspot.com/2013/08/smarter-omnibox-suggestions-for-all.html)

I'd just say 'This API is currently available on ChromeOS, Windows and Mac'.

[Andy, 2013-08-28 23:00:42]

On 2013/08/28 22:25:27, somast1 wrote:
> > I'd just say 'This API is currently available on ChromeOS, Windows and Mac'.

I'm ok with this language, but if we omit the word "only", we need to change the
type of disclaimer from a "Warning" to a "Note" or something like that.  It
would look odd to say:

"Warning: This API is currently available on ChromeOS, Windows, and Mac."

[Andy, 2013-08-28 23:09:40]

PTAL:
https://chrome-apps-doc.appspot.com/_patch/23617008/apps/notifications.html

[not at google - send to devlin, 2013-08-28 23:14:34]

lgtm

[Dmitry Titov, 2013-08-28 23:59:33]

lgtm, thanks!

[Andy, 2013-08-29 23:08:26]

Committed patchset #2 manually as r220419 (presubmit successful).