Closed Bug 1168501 Opened 10 years ago Closed 10 years ago

Timed syncs for history

Categories

(Firefox for iOS :: Sync, defect)

All
iOS 8
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: rnewman, Assigned: rnewman)

References

Details

(Whiteboard: noteworthy)

Attachments

(1 file)

47 bytes, text/x-github-pull-request
st3fan
: review+
Details | Review
This is very under-specified right now.
We want this for history sync. Necessary: backoff!
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Depends on: 1168890
Attached file Pull req.
This is on top of basic backoff handling, earlier in the PR. We kick off a 15-minute timer when the app is foregrounded, and stop it when we go bye-bye. Backoff handling -- or no account, or any other NotStarted cause -- will cause the sync to short-circuit fail with no work being done, so this timer is very cheap; this is a little simpler than twisting together both the app lifecycle and account state.
Attachment #8613848 - Flags: review?(sarentz)
Comment on attachment 8613848 [details] [review] Pull req. LGTM. Just added a comment about the weak AppDelegate.profile, which I think may not be correct or at least not do what you would expect.
Attachment #8613848 - Flags: review?(sarentz) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Depends on: 1171144
Summary: Timed syncs → Timed syncs for history
Whiteboard: noteworthy
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: