Closed Bug 106691 Opened 19 years ago Closed 15 years ago
Use Carbon Events to optimize PLEvent handling
summary says it all.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.6
Target Milestone: mozilla0.9.6 → mozilla0.9.7
Target Milestone: mozilla0.9.7 → mozilla0.9.8
Now that 78611 has been checked in, i think that this bug can be marked WONTFIX/INVALID.
pav, you're probably correct but i still want to make sure that we're not doing any polling.
with work in 105445, we have covered removing polling from timers and PLEvent processing. duping this to that. *** This bug has been marked as a duplicate of 105445 ***
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
ok, more work is required than i thought. un-duping. first, under conrad's patch in the other bug, it just calls ::WakeUpProcess() which delivers a null event to the app and wakes it up. When not calling WNE, my guess is that this is a no-op since we're not polling and there is no concept of idle events. So we need to post a carbon event of our own. Hopefully, though, this doesn't change the event flow (remember dougt's patch which posted events and starved update events?) Another option would be to poll the PLEvent Q on a regular carbon timer. While just as ugly, i think carbon timers are still more efficient CPU-wise than polling with WNE. Too much brainpower required before vacation, i'll pick this up in january.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
not gonna happen in 098
Target Milestone: mozilla0.9.8 → mozilla0.9.9
over to dagley, i probably won't have time for these by 099 with the embedding work i'm getting pulled into.
Assignee: pinkerton → sdagley
Status: ASSIGNED → NEW
This isn't about timers any more.
Summary: Rewrite timers to use CarbonEvents → Use Carbon Events to optimize PLEvent handling
Target Milestone: mozilla0.9.9 → mozilla1.0
Since I don't report into Internet Technologies anymore this bug needs a new owner -> saari
Assignee: sdagley → saari
Conrad did this already.
Assignee: saari → ccarlen
This was done. But see also bug 282940.
Status: NEW → RESOLVED
Closed: 19 years ago → 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.