Tab sync not displaying all virtual tabs



Cloud Services
10 years ago
9 years ago


(Reporter: thunder, Assigned: myk)


Firefox Tracking Flags

(Not tracked)



(2 attachments)



10 years ago
Created attachment 327429 [details]
Virtual tabs json file

I get an empty notification popup (with the tab sync notification, but no entries in it).  In my 'tabs from other computers' menu, I see two listed:

Mozilla Firefox Start Page
2loglines (4 unread, 12 pinned)

I'm attaching my virtual tabs file.

Comment 1

10 years ago
Looking at this...
Assignee: nobody → myk

Comment 2

10 years ago
Created attachment 327449 [details] [diff] [review]
patch v1: fixes problem

Here's a patch that fixes the problem.  Before I check it in, I want to test what happens if we apply this validation step on command creation/editing rather than only at display time.

Comment 3

10 years ago
changeset:   844:c6c4a13cc763
tag:         tip
date:        Mon Jun 30 15:13:07 2008 -0700
summary:     bug 442711: validate virtual tabs to make sure they have the minimal information necessary to recreate them before trying to save them or notify the user about them

Fixed with an updated patch that logs a warning and doesn't do anything when _createCommand or _editCommand is called for an invalid tab (one that doesn't contain the minimum amount of information necessary to recreate the tab).

Note that I left in the code that validates tabs when the Tabs From Other Computers menu and notification are displayed, which is redundant.  Hmm, maybe I should remove that redundancy.
Last Resolved: 10 years ago
Resolution: --- → FIXED

Comment 4

10 years ago
Note: I think Firefox bug 366509 is what caused the data to get into the corrupted state that triggered this bug.

The fix for this bug obviously doesn't address that problem but should work around it to the degree possible (i.e. if corruption occurs when tabs are synced from client A to the server, the corrupted tab won't be added to the virtual tabs on client B, so it'll get removed from the server the next time client B syncs).


9 years ago
Component: Weave → General
Product: Mozilla Labs → Weave
Target Milestone: -- → ---
QA Contact: weave → general
You need to log in before you can comment on or make changes to this bug.