Closed Bug 1315673 Opened 8 years ago Closed 7 years ago

Browser restart sometimes loses containers

Categories

(Core :: DOM: Security, defect, P1)

52 Branch
defect

Tracking

()

RESOLVED INVALID
Tracking Status
firefox52 --- affected

People

(Reporter: rbarnes, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [domsecurity-active][OA][userContextId])

Running Nightly, I have noticed that sometimes when I "restart to update", when the tabs are created in the new browser window, they have the correct URLs, but they are all loaded in the default container (i.e., not colored bar at the top).  

So for example, if you had 2 Gmail accounts open in tabs, account A in container X and account B in the default container, then on browser restart you would get two default container tabs with account B loaded.

Instead, I would expect the new browser window to load the two tabs with the same container assignments as before restart, one tab with container X with account A loaded, and one tab with the default container and account B.
Component: Security → DOM: Security
If this is true, this is not good.  It leaks data across Containers.

Needinfo'ing Kamil to test this.
Flags: needinfo?(kjozwiak)
Priority: -- → P1
Whiteboard: [domsecurity-active][OA][userContextId]
Richard, which platform were you using when you noticed this happening? Did you set any preferences, for example, "Show my windows and tabs from last time"?
Flags: needinfo?(kjozwiak) → needinfo?(rlb)
Platform is MacOS 10.11.6.

I don't think I have any special preferences set.  

Note: I haven't seen this recently, so it might have gone away.  Sorry I don't have cleaner STR.
Flags: needinfo?(rlb)
Awesome, thanks :) I'll see if I can manage to reproduce this on my machine.
IIRC, there was some question about the correct way to handle the origin attributes in the session restore code.  We already have a regression documented in Bug 1291521.
Richard, Kamil can't seem to reproduce this.  And I haven't heard about this issue elsewhere.  Should we close worksforme?

baku, are you aware of any open issues with the code for Session Restore for Containers?
Flags: needinfo?(rlb)
Flags: needinfo?(amarchesini)
I'm fine to close this as INVALID.  I haven't seen it lately either.
Flags: needinfo?(rlb)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Flags: needinfo?(amarchesini)
You need to log in before you can comment on or make changes to this bug.