MSE-in-Workers content update PR #22242 contained a few items needing correction or clarification #22327
Labels
Content:WebAPI
Web API docs
effort: medium
This task is a medium effort.
help wanted
If you know something about this topic, we would love your help!
MDN URL
https://developer.mozilla.org/en-US/docs/Web/API/Media_Source_Extensions_API
What specific section or headline is this issue about?
Multiple cross-cutting areas of the MSE MDN documentation updated by PR #22242 (MSE-in-Workers content additions) contained items I found during my PR review unfortunately after the PR had already been approved and merged.
What information was incorrect, unhelpful, or incomplete?
Below is copied verbatim from the PR #22242 conversations where I originally noted the issues on my eventual review of the PR:
#22242 (comment):
In Chrome, srcObject on HTMLMediaElement is only currently usable for associating a MediaStream or a Dedicated Worker MediaSource's MediaSourceHandle. Main thread MediaSources cannot provide handles yet, nor does srcObject work for plain MediaSource objects in Chrome.
#22242 (comment):
Preview URL verifications:
Finally, on at least one page (MediaSource.handle), the "Note: This feature is available in Web Workers" link is broken. Maybe this is due to the preview linkage or perhaps it's a real broken linkage.
What did you expect to see?
See the detailed list of questions and items I noted as possibly incorrect/needing clarification/incomplete, above.
Do you have any supporting links, references, or citations?
The MSE spec and my mse-in-workers-demo are already both linked appropriately and contain the best information.
This issue is meant to fix some post-merge items I found in my eventual PR review of #22242. See also the related data update PR (mdn/browser-compat-data#18189) and similar issue I filed today on the post-merge items I found with it, too (mdn/browser-compat-data#18220).
Do you have anything more you want to share?
Thank you for doing this content update! I apologize for not getting my review comments entered more rapidly on this, necessitating this issue as a follow-up of the already-merged PR #22242
The text was updated successfully, but these errors were encountered: