calIObserver.onAlarm is never called

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: jminta, Assigned: jminta)

Tracking

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
one would expect that calIObserver.onAlarm would be called whenever an alarm is fired.  This, however, is not the case.  It seems there's a bit of duplicate functionality between this and the alarmService.  Both are supposed to announce alarms, although only one works.
If the functionality is duplicated, just remove one. In this case, the one in calIObserver.
(Assignee)

Comment 2

13 years ago
Created attachment 220860 [details] [diff] [review]
remove calIObserver.onAlarm

Removes calIObserver.onAlarm.  If you want to observe alarms, register yourself with calIAlarmService.
Assignee: base → jminta
Status: NEW → ASSIGNED
Attachment #220860 - Flags: first-review?(dmose)
Comment on attachment 220860 [details] [diff] [review]
remove calIObserver.onAlarm

looks good; r=dmose.  Sorry for the delay.
Attachment #220860 - Flags: first-review?(dmose) → first-review+
(Assignee)

Comment 4

13 years ago
patch checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.