Closed
Bug 1315673
Opened 8 years ago
Closed 7 years ago
Browser restart sometimes loses containers
Categories
(Core :: DOM: Security, defect, P1)
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.
Reporter | ||
Updated•8 years ago
|
Blocks: ContextualIdentity
Component: Security → DOM: Security
Comment 1•8 years ago
|
||
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]
Comment 2•8 years ago
|
||
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)
Reporter | ||
Comment 3•8 years ago
|
||
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)
Comment 4•8 years ago
|
||
Awesome, thanks :) I'll see if I can manage to reproduce this on my machine.
Comment 5•8 years ago
|
||
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.
Comment 6•8 years ago
|
||
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)
Reporter | ||
Comment 7•8 years ago
|
||
I'm fine to close this as INVALID. I haven't seen it lately either.
Flags: needinfo?(rlb)
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Updated•7 years ago
|
Flags: needinfo?(amarchesini)
You need to log in
before you can comment on or make changes to this bug.
Description
•