Let IdleService avoid polling for idle time

NEW
Unassigned

Status

()

8 years ago
4 years ago

People

(Reporter: azakai, Unassigned)

Tracking

unspecified
All
Windows 8.1
Points:
---

Firefox Tracking Flags

(fennec-)

Details

(Reporter)

Description

8 years ago
Polling runs a constant 5-second timer, which is wasteful (both bad for battery life on mobile, and bad for the cycle collector in general).

We can let idleservice implementations have callbacks for activity, which if present would remove the need for polling the idle time manually.

See also discussion in bug 564118.
(Reporter)

Comment 1

8 years ago
Also, when removing the need for polling, we should fix the bug mentioned in bug 564118 where polling and non-polling widget systems behave differently if ResetIdleTimeout is not called.
tracking-fennec: --- → ?

Updated

8 years ago
tracking-fennec: ? → ---
tracking-fennec: --- → ?

Updated

8 years ago
Assignee: nobody → azakai
tracking-fennec: ? → 2.0+

Updated

8 years ago
tracking-fennec: 2.0+ → 2.0-
(Reporter)

Comment 2

8 years ago
Fennec 2.0 shouldn't block on this, this should be done afterwards.
tracking-fennec: 2.0- → ---
(In reply to comment #2)
> Fennec 2.0 shouldn't block on this, this should be done afterwards.
Er, wasn't it marked as blocking-?
(Reporter)

Comment 4

8 years ago
It was blocking-fennec=2.0-, I thought that meant it was desired for 2.0, but not mandatory? I was mistaken, sorry about that...

Updated

8 years ago
tracking-fennec: --- → 2.0-
(Reporter)

Comment 5

8 years ago
(I won't be able to get around to this bug for a while.)
Assignee: azakai → nobody
Whiteboard: [fennec-4.1?]
Still need this?
tracking-fennec: - → ?
Whiteboard: [fennec-4.1?]
(Reporter)

Comment 7

8 years ago
Not really I guess, Android doesn't allow polling anyhow and we disabled it on Maemo.
(In reply to comment #7)
> Not really I guess, Android doesn't allow polling anyhow and we disabled it on
> Maemo.
You maybe don't need this for mobile handheld devices, but this would certainly help for laptops.
tracking-fennec: ? → -

Comment 9

6 years ago
Modifying this to put it on our radar for investigation.
OS: All → Windows 8 Metro
(Assignee)

Updated

4 years ago
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.