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

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

Issue 15127007: Automatically detect inline-able doc in IDL files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: IDL Changes and Inlining return types Created 7 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/alarms.idl
diff --git a/chrome/common/extensions/api/alarms.idl b/chrome/common/extensions/api/alarms.idl
index 47ccb2a61979081c1db863c3aac79b79b99bf2fe..bfef2fdd337f6d55653230cb19b1aa8bf7d137c6 100644
--- a/chrome/common/extensions/api/alarms.idl
+++ b/chrome/common/extensions/api/alarms.idl
@@ -51,13 +51,11 @@ namespace alarms {
//
// In order to reduce the load on the user's machine, Chrome limits alarms
// to at most once every 1 minute 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 minute after "now" without
- // warning but won't actually cause the alarm to fire for at least 1 minute.
+ // more. That is, setting <code><var>delayInMinutes</var></code> or
not at google - send to devlin 2013/05/15 03:55:15 Either <var> or <code>, not both; which looks bett
jshumway 2013/05/15 04:51:37 I made it code to be consistent with the last IDL
+ // <code><var>periodInMinutes</var></code> to less than <code>1</code> will
+ // not be honored and will cause a warning. <code><var>when</var></code>
+ // can be set to less than 1 minute after "now" without warning but won't
+ // actually cause the alarm to fire for at least 1 minute.
//
// 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.

Powered by Google App Engine
This is Rietveld 408576698