596044 - Let IdleService avoid polling for idle time

Status: RESOLVED DUPLICATE of bug 1768920

(Core :: Widget, defect)

Description

[Alon Zakai (:azakai)]

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.

Comment 1

[Alon Zakai (:azakai)]

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.

Comment 2

[Alon Zakai (:azakai)]

Fennec 2.0 shouldn't block on this, this should be done afterwards.

Comment 3

[Shawn Wilsher :sdwilsh]

(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-?

Comment 4

[Alon Zakai (:azakai)]

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...

Comment 5

[Alon Zakai (:azakai)]

(I won't be able to get around to this bug for a while.)

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Still need this?

Comment 7

[Alon Zakai (:azakai)]

Not really I guess, Android doesn't allow polling anyhow and we disabled it on Maemo.

Comment 8

[Shawn Wilsher :sdwilsh]

(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.

Comment 9

[Jim Mathies [:jimm]]

Modifying this to put it on our radar for investigation.