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

Unified Diff: chrome/common/extensions/api/experimental.alarms.idl

Issue 10217018: Alarm resolution changed to minutes and minimum delay added. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Sinked. Created 8 years, 7 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/common/extensions/api/experimental.alarms.idl
diff --git a/chrome/common/extensions/api/experimental.alarms.idl b/chrome/common/extensions/api/experimental.alarms.idl
index c23a0ffadc1cc8891c12b3cf6f0478cf806c569b..b3fdd1ae8d51e4734deec918cda57653e519d3ac 100644
--- a/chrome/common/extensions/api/experimental.alarms.idl
+++ b/chrome/common/extensions/api/experimental.alarms.idl
@@ -10,10 +10,10 @@ namespace experimental.alarms {
// Name of this alarm.
DOMString name;
- // Original length of time in seconds after which the onAlarm event should
+ // Original length of time in minutes after which the onAlarm event should
// fire.
// TODO: need minimum=0
- long delayInSeconds;
+ double delayInMinutes;
// True if the alarm repeatedly fires at regular intervals, false if it
// only fires once.
@@ -23,12 +23,12 @@ namespace experimental.alarms {
// TODO(mpcomplete): rename to CreateInfo when http://crbug.com/123073 is
// fixed.
dictionary AlarmCreateInfo {
- // Length of time in seconds after which the onAlarm event should fire.
+ // Length of time in minutes after which the onAlarm event should fire.
// Note that granularity is not guaranteed: this value is more of a hint to
// the browser. For performance reasons, alarms may be delayed an arbitrary
// amount of time before firing.
// TODO: need minimum=0
- long delayInSeconds;
+ double delayInMinutes;
// True if the alarm should repeatedly fire at regular intervals. Defaults
// to false.

Powered by Google App Engine
This is Rietveld 408576698