RESOLVED FIXED 171249
Allow tabs with capturing audio or video to exceed background CPU limit 
https://bugs.webkit.org/show_bug.cgi?id=171249
Summary Allow tabs with capturing audio or video to exceed background CPU limit
Chris Dumez
Reported 2017-04-24 14:00:13 PDT
Allow tabs with capturing audio or video to exceed background CPU limit as this is a legit case of potentially high background CPU use.
Attachments
Patch (1.98 KB, patch)
2017-04-24 14:14 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2017-04-24 14:13:09 PDT
<rdar://problem/31796314>
Chris Dumez
Comment 2 2017-04-24 14:14:40 PDT
Created attachment 308010 [details] Patch
Geoffrey Garen
Comment 3 2017-04-24 14:34:45 PDT
Comment on attachment 308010 [details] Patch Did you mean to put this up for review? If so, r=me
Chris Dumez
Comment 4 2017-04-24 15:43:27 PDT
Committed r215699: <http://trac.webkit.org/changeset/215699>
youenn fablet
Comment 5 2017-04-24 16:56:57 PDT
Comment on attachment 308010 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=308010&action=review > Source/WebKit2/UIProcess/WebProcessProxy.cpp:1177 > + if (page->hasActiveAudioStream() || page->hasActiveVideoStream()) { LGTM. Seeing again the names in this context, something like isCapturingAudio/isCapturingVideo might be clearer than hasActiveVideoStream/hasActiveVideoStream.
Note You need to log in before you can comment on or make changes to this bug.