1274896 - Reset the Sync client on places autorestore

Status: RESOLVED DUPLICATE of bug 1199077

(Firefox :: Sync, defect)

Description

[Mark Hammond [:markh] [:mhammond]]

Split from bug 1230011 comment 5:

rnewman:
> I agree that on auto-recovery we should do a fresh sync to make 
> sure we didn't accidentally roll back by a week and 'miss' a 
> bunch of changes that we already saw. That's the approach we 
> take on iOS, too.

In terms of desktop Sync, "fresh sync" translates to Service.resetClient(), but not .wipeServer(), right?

So the 2 tasks here are:

* Work out some sane way for Places to let us know that an "auto restore" of bookmarks happened, and ensure that our existing "bookmarks-restore-success" observer does *not* kick in here (that's bug 1230011; user-initiated-restore and auto-restore get different treatment. Note that auto-restore almost certainly happens before Sync is initialized.

* Make the .resetClient() call when Sync initializes and it finds an auto restore was done.

Comment 1

[Richard Newman [:rnewman]]

(In reply to Mark Hammond [:markh] from comment #0)

> In terms of desktop Sync, "fresh sync" translates to Service.resetClient(),
> but not .wipeServer(), right?

Yes, though only reset bookmarks and history. No point re-syncing passwords and prefs…


> So the 2 tasks here are:

Yup!

Comment 2

[Edwin Wong [:edwong]]

Triage: dupe of bug 1199077