Alarm resolution changed to minutes and minimum delay added.

chrome/common/extensions/api/experimental.alarms.idl

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..fcd30b440b6fb9c3c47634d003949e64f0552282 100644
--- a/chrome/common/extensions/api/experimental.alarms.idl
+++ b/chrome/common/extensions/api/experimental.alarms.idl
@@ -10,10 +10,9 @@ namespace experimental.alarms {
     // Name of this alarm.
     DOMString name;
 
-    // Original length of time in seconds after which the onAlarm event should
-    // fire.
-    // TODO: need minimum=0
-    long delayInSeconds;
+    // Original length of time in minutes after which the onAlarm event should
+    // fire. minimum = 5 minute.

[Matt Perry, 2012/04/25 19:11:06]

Keep the "minimum" stuff as a TODO. What I meant was that we need a way of
tagging this in the IDL as having a minumum, so it could be enforced by the code
generator.

[Matt Tytel, 2012/04/30 23:36:21]

Done.

+    double delayInMinutes;
 
     // True if the alarm repeatedly fires at regular intervals, false if it
     // only fires once.
@@ -23,12 +22,11 @@ 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;
+    // amount of time before firing. minimum = 1 minute.
+    double delayInMinutes;
 
     // True if the alarm should repeatedly fire at regular intervals. Defaults
     // to false.