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

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

Issue 11818010: Make chrome.alarms API accept non-integers and too-short delays. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: After discussion, switch to 1-minute minimum alarm time Created 7 years, 11 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/alarms.idl
diff --git a/chrome/common/extensions/api/alarms.idl b/chrome/common/extensions/api/alarms.idl
index ce153f68bb7b6b343fa535f1619f7711f37dc639..07ccab6e9f03ffeb8111e3456fc06ae69b083dff 100644
--- a/chrome/common/extensions/api/alarms.idl
+++ b/chrome/common/extensions/api/alarms.idl
@@ -49,9 +49,18 @@ namespace alarms {
// the same name (or no name if none is specified), it will be cancelled and
// replaced by this alarm.
//
- // Note that granularity is not guaranteed: times are more of a hint to the
- // browser. For performance reasons, alarms may be delayed an arbitrary
- // amount of time before firing.
+ // In order to reduce the load on the user's machine, Chrome limits alarms
+ // to at most once every 1 minutes but may delay them an arbitrary amount
+ // more. That is, setting <code>$ref:[alarms.AlarmCreateInfo.delayInMinutes
+ // delayInMinutes]</code> or
+ // <code>$ref:[alarms.AlarmCreateInfo.periodInMinutes
+ // periodInMinutes]</code> to less than <code>1</code> will not be honored
+ // and will cause a warning. <code>$ref:[alarms.AlarmCreateInfo.when
+ // when]</code> can be set to less than 1 minutes after "now" without
+ // warning but won't actually cause the alarm to fire for at least 1 minutes.
+ //
+ // To help you debug your app or extension, when you've loaded it unpacked,
+ // there's no limit to how often the alarm can fire.
//
// |name|: Optional name to identify this alarm. Defaults to the empty
// string.

Powered by Google App Engine
This is Rietveld 408576698