Closed Bug 678588 Opened 8 years ago Closed 8 years ago
Device Interval to 1 hour
In bug 664792 we got rid of heartbeat-triggered syncs because regular syncs are nearly as efficient as doing heartbeats. However, we forgot to drop the singleDeviceInterval to the heartbeat frequency which was 1 hour.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #552740 - Flags: review?(mconnor)
Comment on attachment 552754 [details] [diff] [review] v1 At which point... why not just use idleInterval? But... minimal is good for Fx7
Attachment #552754 - Flags: review?(mconnor) → review+
(In reply to Mike Connor [:mconnor] from comment #3) > At which point... why not just use idleInterval? But... minimal is good for > Fx7 Yep. Also, still useful to keep them separate for QA because it allows us to test different code paths.
Comment on attachment 552754 [details] [diff] [review] v1 Requesting approval for Aurora. The last follow-up for bug 664792 (I swear!). Risk is minimal, it just changes a pref value that we forgot to adjust.
Attachment #552754 - Flags: approval-mozilla-aurora?
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Attachment #552754 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Target Milestone: mozilla8 → mozilla7
I backed this out of mozilla-beta due to bug 598774: http://hg.mozilla.org/releases/mozilla-beta/rev/e5566280b79d I left in in aurora8 and mozilla-central for now. If you disagree, please email me directly or firstname.lastname@example.org.
Anything QA should be looking for with this backout, str, etc?
I disagree with this backout, will follow up as advised in comment 9.
Actually, because of the investigation in bug 598774 we are going to re-land this on beta8.
Reopening this as we've decided to back this out altogether (see bug 694149). Will be WONTFIXing once bug 694149 lands.
WONTFIXing this as planned (comment 14)
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → WONTFIX
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.