Closed Bug 623665 Opened 10 years ago Closed 8 years ago

Sync botches sessions for History

Categories

(Firefox :: Sync, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: sdwilsh, Unassigned)

References

Details

Places has a concept of a session which basically maps to the navigation in a tab.  Every page that is loaded in it has the same session (sorta, it's complicated).  Right now, when Sync adds visits to other computers, it passes in 0 for the session id (a reasonable thing to do, probably).  However, this makes sessions rather useless.

Sync can't do much better unless Places changes how it tags sessions to use a GUID so it'd be unique across computers (would need to file a new Places bug for that).  It's also possible that we just want to drop the concept all together (in which case, this bug gets WONTFIXed and we file a new Places bug).
Do we really use the session concept for anything in Places?  I remember when we added it, but I don't think we ever really managed to do something useful with it, which I suspect is the reason no one's noticed this before.
(In reply to comment #1)
> Do we really use the session concept for anything in Places?  I remember when
> we added it, but I don't think we ever really managed to do something useful
> with it, which I suspect is the reason no one's noticed this before.
I don't believe we do, and mak and I have talked about removing it in the past.  I'd be interested in hearing what mak and dietrich have to say about this.
IIRC we removed the only usage of it (history sidebar) long ago. We discussed removing the functionality from the backend too. The only repercussion this might have is on extensions that are using the functionality.
Depends on: 821535
WONTFIX per discussion.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
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.