Source/modules/notifications/Notification.idl - Issue 1042513002: Add the vibrate attribute to the Notification object

Side by Side Diff: Source/modules/notifications/Notification.idl

Issue 1042513002: Add the vibrate attribute to the Notification object (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: WebNotificationVibrationData is renamed WebNotificationVibratePattern Created 5 years, 8 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 unified diff | Download patch

« Source/modules/notifications/Notification.cpp ('K') | « Source/modules/notifications/Notification.cpp ('k') | Source/modules/notifications/NotificationOptions.idl » ('j') | Source/modules/notifications/NotificationOptions.idl » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 /*	1 /*

2 * Copyright (C) 2009 Google Inc. All rights reserved.	2 * Copyright (C) 2009 Google Inc. All rights reserved.

3 * Copyright (C) 2011, 2012 Apple Inc. All rights reserved.	3 * Copyright (C) 2011, 2012 Apple Inc. All rights reserved.

4 *	4 *

5 * Redistribution and use in source and binary forms, with or without	5 * Redistribution and use in source and binary forms, with or without

6 * modification, are permitted provided that the following conditions are	6 * modification, are permitted provided that the following conditions are

7 * met:	7 * met:

8 *	8 *

9 * * Redistributions of source code must retain the above copyright	9 * * Redistributions of source code must retain the above copyright

10 * notice, this list of conditions and the following disclaimer.	10 * notice, this list of conditions and the following disclaimer.

(...skipping 37 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
48 [MeasureAs=NotificationShowEvent] attribute EventHandler onshow;	48 [MeasureAs=NotificationShowEvent] attribute EventHandler onshow;

49 attribute EventHandler onerror;	49 attribute EventHandler onerror;

50 [MeasureAs=NotificationCloseEvent] attribute EventHandler onclose;	50 [MeasureAs=NotificationCloseEvent] attribute EventHandler onclose;

51	51

52 readonly attribute DOMString title;	52 readonly attribute DOMString title;

53 readonly attribute DOMString dir;	53 readonly attribute DOMString dir;

54 readonly attribute DOMString lang;	54 readonly attribute DOMString lang;

55 readonly attribute DOMString body;	55 readonly attribute DOMString body;

56 readonly attribute DOMString tag;	56 readonly attribute DOMString tag;

57 readonly attribute DOMString icon;	57 readonly attribute DOMString icon;

	58

	59 // FIXME: The Union type VibrationPattern is not supported in NavigatorVibra tion.

	60 // If VibrationPattern is supported, we need to change with this.

	61 [RuntimeEnabled=NotificationExperimental] readonly attribute (unsigned long or sequence<unsigned long>) vibrate;
	Peter Beverloo 2015/04/07 11:37:31 See my other comment - let's not maintain the type See my other comment - let's not maintain the type of argument passed throughout the stack, at least until a decision has been made on how to support this in the specification. Sanghyun Park 2015/04/08 08:00:03 Done. Show quoted text On 2015/04/07 11:37:31, Peter Beverloo wrote: > See my other comment - let's not maintain the type of argument passed throughout > the stack, at least until a decision has been made on how to support this in the > specification. Done.
58 readonly attribute boolean silent;	62 readonly attribute boolean silent;

59 [RuntimeEnabled=NotificationExperimental, CallWith=ScriptState] readonly att ribute any data;	63 [RuntimeEnabled=NotificationExperimental, CallWith=ScriptState] readonly att ribute any data;

60	64

61 [MeasureAs=NotificationClosed] void close();	65 [MeasureAs=NotificationClosed] void close();

62 };	66 };

OLD	NEW