805369 - Implement lazy loading for caldav worker & shut it down when not needed.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Calendar, defect, P3)

Description

[James Lal [:lightsofapollo]]

The caldav worker takes about 3mb of memory but we only need it when syncing the calendars. Its a quick and easy win to lazy-load the worker (don't start it until the first request) and shut it down after a small period of inactivity (around 10 seconds would be appropriate).

(I am not too clear on the blocking status regarding memory usage but given that we save up to 20% of the total memory usage of calendar when not actively syncing we should do this).

The basics for this are in place already, we just need to defer the startup and implement the inactivity timeout in worker/manager.js

Comment 1

[Marijn Haverbeke]

I'm working on this. My bugzilla user apparently doesn't have the privilege required to assign bugs, so I'm commenting to discourage others to also start on it. If someone with privileges comes by, I'd be grateful if they'd properly assign the bug to me.

Comment 2

[Jason Smith [:jsmith]]

(In reply to Marijn Haverbeke from comment #1)
> I'm working on this. My bugzilla user apparently doesn't have the privilege
> required to assign bugs, so I'm commenting to discourage others to also
> start on it. If someone with privileges comes by, I'd be grateful if they'd
> properly assign the bug to me.

And...assigned!

Comment 3

[James Lal [:lightsofapollo]]

Hey Marijn,

I had started a very rough patch for this I could share the (mostly-non functional) version with you it may help you get started.

Comment 4

[Marijn Haverbeke]

I'm at the 'almost working patch' stage, but I'd definitely be interested in your approach.

Comment 5

[Marijn Haverbeke]

Attachment: fix for the problem

Find it at https://github.com/mozilla-b2g/gaia/pull/6283