517597 - waste fewer resources while a user only has one client active

Status: RESOLVED FIXED

(Firefox :: Sync, defect, P1)

Description

[Mike Connor [:mconnor]]

Attachment: force timers to 24 hours with a single client set up.

POC patch attached, default to 24 hours between syncs for users with a single client.

With this we'll sync after each startup, and then either manually or every 24 hours after that.  Manual syncs will bump the next sync to 24 hours out from then.

To do/consider:

* Store next sync time as a pref, enforce 24 hours or manual syncs across restarts.
* Look at tracker-based sync calls to determine.
* Disable tab sync with a single client active.

Comment 1

[Mike Connor [:mconnor]]

Attachment: patch

What this does:

* default sync interval for single-client users is 24 hours.  Manual syncs are allowed in this case.
* scheduling works across sessions for all users (this helps with backoff too)
* We only sync on startup if we're past the next scheduled sync time.
* Tabs are not synced when there is a single client active.

Comment 2

[Mike Connor [:mconnor]]

Attachment: take this further

This patch is on top of bug 518075, and there's a bit of cleanup for that yet.

This moves the consts to constants.js, and defines thresholds and intervals for different client modes.  Basically, single-client should remain unchanged, but multi-client without mobile clients will have.

Comment 3

[Ed Lee :Mardak]

http://hg.mozilla.org/labs/weave/rev/98757a88f66c
Adjust how often Weave syncs based on the number of clients and only sync tabs if there are multiple clients.