Closed Bug 1333798 Opened 7 years ago Closed 7 years ago

session restored tabs seem to cluster in one content process

Categories

(Core :: DOM: Content Processes, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: bkelly, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [e10s-multi:+])

Since multi-e10s was enabled in nightly with 2 content processes I have noticed:

1) Of my nine tabs, the vast majority end up in one content process.  8-to-1 and 7-to-2 so far.
2) Most of the tabs in the larger content process were opened by session restore when I launched the browser.
3) Most of the tabs in the larger content process are pinned tabs that load immediately.

Does session restore start before we have spawned the second content process or something?
Blocks: ss-perf
Whiteboard: [e10s-multi:?]
Whiteboard: [e10s-multi:?] → [e10s-multi:+]
Ben, I tried to reproduce this but couldn't. Is this something you can reproduce reliably? Maybe some of those tabs are special (I would check if we skip any process in ContentParent::RandomSelect because of the opener).
Flags: needinfo?(bkelly)
Well, as bug 1342195 I find it hard to interpret the tab selection policy.  You can probably close this if you can't reproduce.  I really wish we could move to something predictable even if its not perfect.  Seems like people are bikeshedding bug 1333799.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(bkelly)
Resolution: --- → INVALID
(In reply to Ben Kelly [not reviewing due to deadline][:bkelly] from comment #2)
> Well, as bug 1342195 I find it hard to interpret the tab selection policy. 
> You can probably close this if you can't reproduce.  I really wish we could
> move to something predictable even if its not perfect.  Seems like people
> are bikeshedding bug 1333799.

Yeah, the 'choose the process with the least tabs' approach mentioned there seems like a good start, I'll talk to Blake if he can work on it next week or I'll pick it up.
You need to log in before you can comment on or make changes to this bug.