Closed Bug 965474 Opened 8 years ago Closed 8 years ago

Initial Sync isn't triggered automatically

Categories

(Firefox :: Sync, defect)

x86
All
defect
Not set
blocker

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: tracy, Assigned: markh)

References

Details

(Whiteboard: [qa+])

Attachments

(1 file)

Using fx-team-<platform>/1390969090/ build:

0) install build and config new profile to use Fx Accounts
1) Create an account on device A (tested with Win 8.1)
2) Sign into same account on device B (tested with Mac 10.9)

tested results:

In both cases, the door hanger appears saying client is now syncing. But in reality no sync has occurred.

expected results:
after step 1) On client A, an initial upload of all sync data should occur. Then auto-syncs re-occur per various trigger rules.

after step 2) Client B's data should be successfully synced with client A's previously uploaded data. Then auto-syncs re-occur per various trigger rules.
(assuming for now this is desktop-specific)
Component: Firefox Sync: Backend → Server: Firefox Accounts
QA Contact: jbonacci
Whiteboard: [qa+]
James,  I am pretty sure this is client side.  The client knows account was just setup, confirmed by the fact the door hanger notice appears.  It's just not triggering itself to do the initial sync.
You can jump start the auto-sync process by manually Sync Now client A, then Sync Now Client B.  I get succeful sync logs that are obeying timing triggers as well as bookmarking triggers.
Given that, adding some more people...
The doorhanger won't lie anymore when bug 965544 lands. Mark is working on all the sync trigger points. Is there already a bug for that?
Depends on: 965544
Flags: needinfo?(mhammond)
(In reply to Tim Taubert [:ttaubert] from comment #5)
> The doorhanger won't lie anymore when bug 965544 lands. Mark is working on
> all the sync trigger points. Is there already a bug for that?

This is that bug.
Flags: needinfo?(mhammond)
Assignee: nobody → mhammond
We've decided not to block Nightly enabling on this bug.
Blocks: 905997
No longer blocks: 964922
No longer depends on: 965544
Comment on attachment 8368372 [details] [diff] [review]
0007-Bug-965464-allow-for-an-initial-sync-on-new-user-sig.patch

Review of attachment 8368372 [details] [diff] [review]:
-----------------------------------------------------------------

::: services/sync/modules/browserid_identity.js
@@ +125,5 @@
> +            Services.prefs.clearUserPref(PREF_SYNC_SHOW_CUSTOMIZATION);
> +          } else {
> +            // Log out if the user canceled the dialog.
> +            return fxAccounts.signOut();
> +          }

Customization logic should be refactored out to somewhere else at some point.
Attachment #8368372 - Flags: review?(ckarlof) → review+
Component: Server: Firefox Accounts → Firefox Sync: Backend
QA Contact: jbonacci
https://hg.mozilla.org/mozilla-central/rev/b00d55e852bc
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
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.