Closed Bug 729248 Opened 8 years ago Closed 8 years ago

Smarter upload of our clients record

Categories

(Firefox for Android :: Android Sync, defect, P2)

ARM
Android
defect

Tracking

()

VERIFIED FIXED
mozilla14
Tracking Status
firefox13 --- verified
firefox14 --- verified

People

(Reporter: rnewman, Assigned: emtwo)

References

Details

(Whiteboard: [qa!])

Should only upload when we're getting dangerously close to our TTL, just as desktop does.

This will involve tracking a server-side timestamp.

Whenever we upload, bump this timestamp from X-Weave-Timestamp.

Whenever we download our client record, because another client has changed it, bump this timestamp from the modified time.

Whenever we process commands, the subsequent upload of a commandless record will result in bumping the timestamp.

(And the post-command upload should have an If-Unmodified-Since header to avoid losing a command.)
Depends on: 731125
QA:

With this change, Android Sync will no longer upload a client record on every sync. It will upload on its first sync, and every seven days thereafter, or whenever it processes a command.

(It's not currently smart enough to know when the server has been wiped, and reupload.)

The easiest way to verify this is to check desktop: when it syncs, it won't see the mobile device's client record as changed.

Sync mobile, then desktop, then mobile again.

Now when you next sync desktop, you'll see:

1331800456513	Sync.Service	DEBUG	Refreshing client list.
1331800456514	Sync.Engine.Clients	INFO	0 outgoing items pre-reconciliation
1331800456516	Sync.Engine.Clients	INFO	Records: 0 applied, 0 successfully, 0 failed to apply, 0 newly failed to apply, 0 reconciled.
Whiteboard: [qa+]
https://hg.mozilla.org/mozilla-central/rev/575bc07d40aa
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Whiteboard: [qa+] → [qa!]
Product: Mozilla Services → Android Background Services
Product: Android Background Services → Firefox for Android
You need to log in before you can comment on or make changes to this bug.