1734099 - We should not unload tabs with webrtc peer connections

Status: RESOLVED FIXED

(Firefox :: Tabbed Browser, enhancement)

Description

[Toshihito Kikuchi [:toshi]]

The tab unloader deprioritizes tabs with webrtc streams. Here are the ideas from the media team. We should check at least HasActivePeerconnections.

• media recorder needs a stream to record, so it should be covered by browserHasStreams. It looks like browserHasStreams covers tracks from getUserMedia and getDisplayMedia. There could also be ones generated from canvas capture and webAudio. canvases are not rendered in background tabs so canvas capture is not very useful then.

• Media recorder typically is being used to record a stream from a camera/mic, as pehrsons says, or maybe from a Peerconnection (but that would be covered by HasActivePeerconnections). A WebAudio or screencapture source would be about the only thing not covered; I don't know if there are easy ways to check if mediarecorder is in use; I don't think there is. pehrsons that does remind me that tabs using MediaRecorder shouldn't be allowed in the bfcache, though...

Comment 1

[Toshihito Kikuchi [:toshi]]

Attachment: Bug 1734099 - Never unload tabs with active RTCPeerConnection instances. r=peterv,jesup

This patch makes sure the Tab Unloading feature does not unload tabs that have
active peer connections not to disrupt browsing experience based on WebRTC peer
connections.

To enable JS code to access the number of active peer connections, this patch
moves nsPIDOMWindowInner::mTotalActivePeerConnections to WindowContext as
a synched field of the top window.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/812ed5275f0d
Never unload tabs with active RTCPeerConnection instances.  r=peterv,jesup

Comment 3

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/812ed5275f0d