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

Unified Diff: chrome/test/data/extensions/api_test/notification/galore/app/model.js

Issue 12212037: Update notifications API after review. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge conflict resolved Created 7 years, 10 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/test/data/extensions/api_test/notification/galore/app/model.js
diff --git a/chrome/test/data/extensions/api_test/notification/galore/app/model.js b/chrome/test/data/extensions/api_test/notification/galore/app/model.js
index 56289ff2caccf1b1defeedf08f852edcbbe15371..2c07f027610ae92b33d7422d6016210bb4888c1c 100644
--- a/chrome/test/data/extensions/api_test/notification/galore/app/model.js
+++ b/chrome/test/data/extensions/api_test/notification/galore/app/model.js
@@ -7,7 +7,7 @@ var Galore = Galore || {};
Galore.NOTIFICATIONS = [
{
name: 'Simple Notifications',
- type: 'simple',
+ templateType: 'simple',
notifications: [
{
iconUrl: '$@/images/man1-$%x$%.jpg',
@@ -33,16 +33,16 @@ Galore.NOTIFICATIONS = [
},
{
name: 'Basic Notifications',
- type: 'base',
+ templateType: 'basic',
notifications: [
{
iconUrl: '$@/images/man1-$%x$%.jpg',
title: 'Althe Frazon',
message: 'Lorem ipsum',
- buttonOneIconUrl: '$@/images/call-16x16.jpg',
- buttonOneTitle: 'Call',
- buttonTwoIconUrl: '$@/images/send-16x16.jpg',
- buttonTwoTitle: 'Send Email'
+ buttons: [
+ {title: 'Call', iconUrl: '$@/images/call-16x16.jpg'},
+ {title: 'Send Email', iconUrl: '$@/images/send-16x16.jpg'},
+ ]
},
{
iconUrl: '$@/images/woman1-$%x$%.jpg',
@@ -93,7 +93,7 @@ Galore.NOTIFICATIONS = [
},
{
name: 'Image Notifications',
- type: 'image',
+ templateType: 'image',
notifications: [
{
iconUrl: '$@/images/woman3-$%x$%.jpg',
@@ -106,8 +106,10 @@ Galore.NOTIFICATIONS = [
title: 'Notification #$#: Daffodils!',
message: 'Narcissus',
imageUrl: '$@/images/image1-300x225.jpg',
- buttonOneTitle: 'Lorem Ipsum Dolor Sit Amet Consectetur Adipisicing',
- buttonTwoTitle: 'Elit Sed Do'
+ buttons: [
+ {title: 'Lorem Ipsum Dolor Sit Amet Consectetur Adipisicing'},
+ {title: 'Elit Sed Do'},
+ ]
},
{
iconUrl: '$@/images/flower2-$%x$%.jpg',
@@ -131,7 +133,7 @@ Galore.NOTIFICATIONS = [
},
{
name: 'List Notifications',
- type: 'multiple',
+ templateType: 'list',
notifications: [
{
iconUrl: '$@/images/inbox-00-$%x$%.png',
@@ -144,8 +146,9 @@ Galore.NOTIFICATIONS = [
{title: 'Kelly Seiken', message: 'Ut labore et dolore magna aliqua'},
{title: 'Maricia Rilari', message: 'Ut enim ad minim veniam'}
],
- buttonOneIconUrl: '$@/images/send-16x16.png',
- buttonOneTitle: 'Send Message'
+ buttons: [
+ {title: 'Send Message', iconUrl: '$@/images/send-16x16.png'}
+ ]
},
{
iconUrl: '$@/images/inbox-01-$%x$%.png',
« no previous file with comments | « chrome/test/data/extensions/api_test/notification/galore/app/controller.js ('k') | ui/notifications/notification_types.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698