Closed Bug 1706453 Opened 4 years ago Closed 4 years ago

Remove reference to nsIMediaManagerService.mediaCaptureWindowState optionally iterating iframes

Categories

(Core :: WebRTC: Audio/Video, task, P3)

Product:
Core
Component:
WebRTC: Audio/Video
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
90 Branch
Tracking Flags:
Tracking Status
firefox90 --- fixed

People

(Reporter: bryce, Assigned: bryce)

References

Details

Attachments

(1 file)

Bug 1706453 - Reword mediaCaptureWindowState docstring to make clear it's not recursive. r?jib
48 bytes, text/x-phabricator-request
Details | Review

Since https://phabricator.services.mozilla.com/D93079 it looks like the comment at https://searchfox.org/mozilla-central/rev/9b430bb1a11d7152cab2af4574f451ffb906b052/dom/media/nsIMediaManager.idl#27-28 still refers to optionally checking iframes. My understanding is this is no longer possible, and that checking of descendants is done via other mechanisms (such as iterating activeMediaCaptureWindows in the manager and checking for iframes there).

Pushed by bvandyk@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b7eaa0c29a5f Reword mediaCaptureWindowState docstring to make clear it's not recursive. r=jib DONTBUILD

https://hg.mozilla.org/mozilla-central/rev/b7eaa0c29a5f

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox90: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: