Closed Bug 568256 Opened 14 years ago Closed 14 years ago

form history changes breaks form sync, FF search bar, probably more

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mconnor, Assigned: Mardak)

References

Details

Attachments

(2 files)

bug 439716 changed the underlying implementation, so we'll need to handle that case in various places. Affects today's nightly, so we have to suck it up and fix this for 1.3
Flags: blocking-weave1.3+
Err, how did this break Weave? There shouldn't have been any interface changes or significant changes in functionality -- it was just a straight C++ to JS port.
Weave hardcodes the uuid of the class so it can override it. The uuid it's using matched up with the CID which is now gone. http://hg.mozilla.org/labs/weave/file/19c24f8a50bb/source/components/FormNotifier.js#l7 http://hg.mozilla.org/mozilla-central/rev/cabbe056fd9c#l1.7
Not sure if this is legit, but we could fix it from the satchel side by reverting back to the same class id we had before.
Attached patch v1 (deleted) — Splinter Review
Attachment #447569 - Flags: review?(mconnor)
Blocks: 568292
Comment on attachment 447569 [details] [diff] [review] v1 Great, thanks, please land ASAP and I'll prep to push a b8.
Attachment #447569 - Flags: review?(mconnor) → review+
http://hg.mozilla.org/labs/weave/rev/5bd2b30e1556 Check for both old and new class IDs and use the one that exists when wrapping. Also add FormTracker as the observer object so that its notify function is correctly called by nsHTMLFormElement.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.3
With FS 1.3rc2 on all clients: Web form data is syncing to and from minefield (Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.3a5pre) Gecko/20100526 Minefield/3.7a5pre) However, search bar entries are not syncing out from minefield. Entries on 3.6.4 do sync back to minefield. 1) Perform a search from the search bar on minefield 2) Sync minefield, wait for it to complete 3) Sync 3.6.4 I'll attach a log from a third client (3.6.4) on it's initial sync. It also does not get any searchBar data from Minefield. Outgoing log from Minefield: 2010-05-27 08:23:47 Net.Resource DEBUG GET success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/meta/global 2010-05-27 08:23:47 Service.Main DEBUG Weave Version: 1.3rc2 Local Storage: 2 Remote Storage: 2 2010-05-27 08:23:47 Engine.Clients INFO 0 outgoing items pre-reconciliation 2010-05-27 08:23:47 Engine.Clients INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:47 Engine.Clients DEBUG Total (ms): sync 1, processIncoming 1, uploadOutgoing 0, syncStartup 0, syncFinish 0 2010-05-27 08:23:47 Engine.Bookmarks INFO 0 outgoing items pre-reconciliation 2010-05-27 08:23:47 Engine.Bookmarks INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:47 Engine.Bookmarks DEBUG Total (ms): sync 2, processIncoming 1, uploadOutgoing 0, syncStartup 0, syncFinish 0 2010-05-27 08:23:47 Engine.Forms INFO 2 outgoing items pre-reconciliation 2010-05-27 08:23:47 Engine.Forms INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:47 Engine.Forms INFO Uploading all of 2 records 2010-05-27 08:23:47 Collection DEBUG POST Length: 751 2010-05-27 08:23:47 Collection DEBUG POST success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/forms 2010-05-27 08:23:47 Engine.Forms DEBUG Total (ms): sync 410, processIncoming 0, uploadOutgoing 409, syncStartup 0, syncFinish 0, createRecord 0 2010-05-27 08:23:47 Engine.History INFO 2 outgoing items pre-reconciliation 2010-05-27 08:23:47 Engine.History INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:47 Engine.History INFO Uploading all of 2 records 2010-05-27 08:23:47 Collection DEBUG POST Length: 1160 2010-05-27 08:23:47 Collection DEBUG POST success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/history 2010-05-27 08:23:47 Engine.History DEBUG Total (ms): sync 488, processIncoming 0, uploadOutgoing 487, syncStartup 1, syncFinish 0, createRecord 20 2010-05-27 08:23:47 Engine.Passwords INFO 1 outgoing items pre-reconciliation 2010-05-27 08:23:47 Engine.Passwords INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:47 Engine.Passwords INFO Uploading all of 1 records 2010-05-27 08:23:47 Collection DEBUG POST Length: 678 2010-05-27 08:23:48 Collection DEBUG POST success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/passwords 2010-05-27 08:23:48 Engine.Passwords DEBUG Total (ms): sync 464, processIncoming 0, uploadOutgoing 463, syncStartup 1, syncFinish 0, createRecord 3 2010-05-27 08:23:48 Engine.Prefs INFO 0 outgoing items pre-reconciliation 2010-05-27 08:23:48 Engine.Prefs INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:48 Engine.Prefs DEBUG Total (ms): sync 1, processIncoming 0, uploadOutgoing 0, syncStartup 0, syncFinish 0 2010-05-27 08:23:48 Engine.Tabs INFO 1 outgoing items pre-reconciliation 2010-05-27 08:23:48 Engine.Tabs INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:23:48 Engine.Tabs INFO Uploading all of 1 records 2010-05-27 08:23:48 Collection DEBUG POST Length: 1905 2010-05-27 08:23:49 Collection DEBUG POST success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/tabs 2010-05-27 08:23:49 Engine.Tabs DEBUG Total (ms): sync 598, processIncoming 1, uploadOutgoing 597, syncStartup 0, syncFinish 0, getAllClients 0, locallyOpenTabMatchesURL 24, createRecord 12 2010-05-27 08:23:49 Service.Main INFO Sync completed successfully in 32.95 secs. incoming log from 3.6.4: 2010-05-27 08:25:50 Net.Resource DEBUG GET success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/info/collections 2010-05-27 08:25:51 Net.Resource DEBUG GET success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/meta/global 2010-05-27 08:25:51 Service.Main DEBUG Weave Version: 1.3rc2 Local Storage: 2 Remote Storage: 2 2010-05-27 08:25:51 Engine.Clients INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:51 Engine.Clients INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:25:51 Engine.Clients DEBUG Total (ms): sync 1, processIncoming 1, uploadOutgoing 0, syncStartup 0, syncFinish 0 2010-05-27 08:25:51 Engine.Bookmarks INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:51 Engine.Bookmarks INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:25:51 Engine.Bookmarks DEBUG Total (ms): sync 2, processIncoming 0, uploadOutgoing 0, syncStartup 1, syncFinish 0 2010-05-27 08:25:51 Engine.Forms INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:51 Engine.Forms INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:25:51 Engine.Forms DEBUG Total (ms): sync 1, processIncoming 1, uploadOutgoing 0, syncStartup 0, syncFinish 0 2010-05-27 08:25:51 Engine.History INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:51 Collection DEBUG GET success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/history?newer=1274966171.36&full=1&sort=index 2010-05-27 08:25:51 Engine.History INFO Records: 2 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:25:51 Engine.History DEBUG Total (ms): sync 517, processIncoming 514, uploadOutgoing 0, syncStartup 0, syncFinish 0, findDupe 0, createRecord 32, isEqual 32, reconcile 33 2010-05-27 08:25:51 Engine.Passwords INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:52 Collection DEBUG GET success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/passwords?newer=1274966171.84&full=1&sort=index 2010-05-27 08:25:52 Engine.Passwords INFO Records: 0 applied, 1 reconciled, 0 left to fetch 2010-05-27 08:25:52 Engine.Passwords DEBUG Total (ms): sync 463, processIncoming 461, uploadOutgoing 0, syncStartup 1, syncFinish 0, createRecord 2, isEqual 2, reconcile 4 2010-05-27 08:25:52 Engine.Prefs INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:52 Engine.Prefs INFO Records: 0 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:25:52 Engine.Prefs DEBUG Total (ms): sync 1, processIncoming 1, uploadOutgoing 0, syncStartup 0, syncFinish 0 2010-05-27 08:25:52 Engine.Tabs INFO 0 outgoing items pre-reconciliation 2010-05-27 08:25:52 Store.Tabs DEBUG Adding remote tabs from tracy's Firefox on trunkSyncrc2-tracy-walkers-macbook-pro 2010-05-27 08:25:52 Collection DEBUG GET success 200 https://sj-weave08.services.mozilla.com/1.0/mozqasj2/storage/tabs?newer=1274966172.43&full=1&sort=index 2010-05-27 08:25:52 Engine.Tabs INFO Records: 1 applied, 0 reconciled, 0 left to fetch 2010-05-27 08:25:52 Engine.Tabs DEBUG Total (ms): sync 459, processIncoming 459, uploadOutgoing 0, syncStartup 0, syncFinish 0, findDupe 0, reconcile 0 2010-05-27 08:25:52 Service.Main INFO Sync completed successfully in 2.35 secs.
Attached file intial sync log on 3rd client (deleted) —
Attachment #447745 - Attachment description: intiial sync log on 3rd client → intial sync log on 3rd client
Attachment #447745 - Attachment mime type: application/octet-stream → text/html
Attachment #447745 - Attachment mime type: text/html → text/plain
Blocks: 568587
Still not working on: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a5pre) Gecko/20100527 Minefield/3.7a5pre (.NET CLR 3.5.30729)
(In reply to comment #13) > Still not working on: > > Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a5pre) Gecko/20100527 > Minefield/3.7a5pre (.NET CLR 3.5.30729) The fix requires that Weave (now Firefox Sync) has been updated to 1.3. If it's still not working after you update, please give your weave version.
Investigating this bug for being potential crossweave automation test case candidate.
Flags: in-testsuite?
item is undefined JS Stack trace: getGUID("end","today")@forms.js:79 < In fact, "item" object is used, even if undefined. Must do a check before. Else, it breaks synchronization.
Gaétan: This bug is fixed. Please file a new bug for whatever issue you're seeing.
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.

Attachment

General

Creator:
Created:
Updated:
Size: