Closed
Bug 949695
Opened 11 years ago
Closed 11 years ago
land init-with-fxa Weave.js code to elm
Categories
(Cloud Services :: Server: Firefox Accounts, defect)
Cloud Services
Server: Firefox Accounts
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla29
People
(Reporter: warner, Assigned: warner)
References
Details
(Whiteboard: [qa+])
Attachments
(2 files, 1 obsolete file)
5.54 KB,
patch
|
Details | Diff | Splinter Review | |
143.58 KB,
patch
|
ttaubert
:
feedback+
|
Details | Diff | Splinter Review |
This patch, from mhammond and ckarlof, changes the Sync XPCOM service to be initialized with an FxAccount instead of purely from previously-saved Sync account data. If we can land it on elm, we can get an elm build which does fxa+sync, but that will probably break old-style sync.
Updated•11 years ago
|
Assignee: rfkelly → warner-bugzilla
Comment 1•11 years ago
|
||
This will be interesting to test then.
1. That this patch does indeed make fxa+sync happen
2. This this patch does indeed break old style sync
Whiteboard: [qa+]
Assignee | ||
Comment 2•11 years ago
|
||
I think the goal is to land a version that has a pref to toggle between the two modes, which will make it easier to land on m-c in the long run too.
Comment 3•11 years ago
|
||
Attachment #8346838 -
Attachment is obsolete: true
Comment 4•11 years ago
|
||
Pushed to elm as 90dc9c1209d9
Updated•11 years ago
|
Comment 5•11 years ago
|
||
This is a patch containing the differences between mozilla-central and elm that relate to this bug. It is all a little messy as stuff has been landing on elm without review, but now there's a bit of a rush to land it to central. As a result I might have screwed the patch up in some way (eg, missed a relevant file) - if I've done that, please refer to the elm branch.
In particular, this patch is relatively huge. If it helps, the test changes might be better viewed using the following revisions on elm:
da352ff3e3ee: Bug 949695 followup - add resetCredentials/resetSyncKey to avoid BrowserID setters from being called.
ebbb473f88e8: Bug 949695 tests followup - Add ability to run tests twice - once with each identity manager - and have test_service_startOver use it.
b3d8ec11e222: Bug 949695 tests followup - convert test_service_wipeServer to identity_test infrastructure
10e5556d93be: Bug 949695 tests followup - convert more tests to run with both identity managers.
67c515f2064e: Bug 949695 tests followup - fix test_handleSyncError to reset error account so it passes when run twice
This patch should be just related to changes to Sync itself to make use of the browserid_identity module. Also note that this patch and the one in bug 949259 really aren't stand-alone.
Requesting review from rnewman and feedback from ttaubert.
Attachment #8350473 -
Flags: review?(rnewman)
Attachment #8350473 -
Flags: feedback?(ttaubert)
Comment 6•11 years ago
|
||
Comment on attachment 8350473 [details] [diff] [review]
fxaccounts_sync.patch
Review of attachment 8350473 [details] [diff] [review]:
-----------------------------------------------------------------
I'm not too familiar with all the sync code but this looks ok to me.
Attachment #8350473 -
Flags: feedback?(ttaubert) → feedback+
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/90dc9c1209d9
https://hg.mozilla.org/mozilla-central/rev/da352ff3e3ee
https://hg.mozilla.org/mozilla-central/rev/ebbb473f88e8
https://hg.mozilla.org/mozilla-central/rev/b3d8ec11e222
https://hg.mozilla.org/mozilla-central/rev/10e5556d93be
https://hg.mozilla.org/mozilla-central/rev/67c515f2064e
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Comment 8•11 years ago
|
||
Comment on attachment 8350473 [details] [diff] [review]
fxaccounts_sync.patch
Guess I'll clear this here review flag, then :D
Attachment #8350473 -
Flags: review?(rnewman)
Comment 9•11 years ago
|
||
(In reply to Richard Newman [:rnewman] from comment #8)
> Guess I'll clear this here review flag, then :D
Oh, darn. Can you please file a follow-up for all the comments you had so we can fix this later? Sorry about that.
Comment 10•11 years ago
|
||
Cleaning up Resolved/Fixed bugs from December's first release.
Verified that we now have a working first-release of FxA to Desktop/Android Nightly.
Re-open as needed.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•