1285300 - Sync bookmark errors appear to drain battery

Status: RESOLVED INCOMPLETE

(Firefox for iOS :: Sync, defect, P3)

Description

[Nick Nguyen [:osunick]]

I've experienced the "Sync is experiencing issues syncing bookmarks" bug for the last week, and I haven't bothered to fix it. During this time, Firefox v5 has ballooned to 64% of my battery usage. Fixing the bookmark issue has now dropped battery usage to a reasonable amount. Can we look into this issue, and most importantly, confirm that there isn't a power consumption issue related to sync?

Comment 1

[Nick Nguyen [:osunick]]

Attachment: Slack for iOS Upload.png

Comment 2

[Richard Newman [:rnewman]]

If there are no new bookmarks to download, an attempt to sync involves only a cheap database check for consistency, then a real attempt to do the work. The only way it'd heavily drain battery is if the cheap check passed but the real attempt failed, and you are a heavy Firefox user (lots of bookmarks, lots of time with Firefox open on your phone).

Do you still get battery drain if you uncheck "Bookmarks" in the Sync preferences pane on your desktop? 

Note: doing so will turn off bookmark sync for all of your devices, and if you re-check it, you will probably find that your bookmarks sync successfully -- the root cause of this issue is that desktop routinely uploads corrupt data to the server, and unchecking and re-checking the box will cause it to do a fresh (and likely successful) reupload.

Comment 3

[Nick Nguyen [:osunick]]

Yes, this appears to be the case. If I uncheck 'bookmarks' as per the workaround then battery usage drops dramatically.

Comment 4

[Richard Newman [:rnewman]]

How many bookmarks do you have, Nick?

  sqlite3 places.sqlite "SELECT COUNT(*) FROM moz_bookmarks"

in your profile directory should work.


We shouldn't bother trying again if the last merge attempt failed for inconsistency and nothing new has been buffered since the last attempt. Emily, did any work along those lines make it into the tree?

Comment 5

[Emily Toop (:fluffyemily)]

I don't believe any of that work has made it into the current codebase. It was one of the next things on the list to tackle.

Comment 6

[Nick Nguyen [:osunick]]

1052 bookmarks (I know, I know)

Comment 7

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Closing this bug because we have seen no evidence of this recently.