Closed Bug 953311 Opened 6 years ago Closed 6 years ago
Idle Service .idl and .cpp comments are incorrect
http://mxr.mozilla.org/mozilla-central/source/widget/nsIIdleService.idl 40 * @note 41 * The subject of the notification the observer will get is always the 42 * nsIIdleService itself. 43 * When the user goes idle, the observer topic is "idle" and when they get 44 * back, the observer topic is "back". 45 * The data param for the notification contains the current user idle time. widget/xpwidgets/nsIdleService.cpp 581: // Now send "back" events to all, if any should have timed out allready, then vs https://developer.mozilla.org/en-US/docs/XPCOM_Interface_Reference/nsIIdleService "and receive a 'back' (Gecko 3 to 15) or 'active' (Gecko 16+) notification when the user starts using their computer again." See also: Bug 953282. When this changed, nobody updated Sync, and perhaps other consumers, too. I see this in Places: toolkit/components/places/nsPlacesExpiration.js 39:const TOPIC_IDLE_END = "back";
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Attachment #8357120 - Flags: review?(gsvelto)
Comment on attachment 8357120 [details] [diff] [review] fix comments I'm no widget peer so I'm unsure if my review is enough but considering it's comments only it shouldn't hurt. I've grepped around to see if there were still uses of the old event and found only what's already been reported as bug 953312.
Attachment #8357120 - Flags: review?(gsvelto) → review+
Considered it's comment only and nobody complained about a patch landing without SR that broke all of the consumers...
Target Milestone: --- → mozilla29
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.