[MSE] Add a Modules/mediasource/README.md file
Created attachment 446404 [details] Patch
Comment on attachment 446404 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=446404&action=review > Source/WebCore/Modules/mediasource/README.md:7 > +The Media Source Extensions specification defines a set of classes which allows clients to implement their own loading, buffering, and variant switching behavior, as opposed to allowing the UA to handle same. Maybe "as opposed to requiring the UA to" instead of "allowing"? > Source/WebCore/Modules/mediasource/README.md:59 > +MediaSample is an abstract base class representing a sample parsed from a media segment. MediaSamples have `presentationTime()`, `decodeTime()`, and `duration()`, which are used to order these samples relative to one another in a [SampleMap](#samplemap). For codecs which support frame reordering, `presentationTime()` and `decodeTime()` for each sample may differ. It is probably worth mentioning `MediaTime`, and why we use it instead of `double` for time values..
Comment on attachment 446404 [details] Patch I like this patch! View in context: https://bugs.webkit.org/attachment.cgi?id=446404&action=review > Source/WebCore/Modules/mediasource/README.md:54 > + Maybe we can add some information about SourceBufferPrivateAVFObjC and SourceBufferParser?
Created attachment 446412 [details] Patch for landing
(In reply to Peng Liu from comment #3) > Comment on attachment 446404 [details] > Patch > > I like this patch! > > View in context: > https://bugs.webkit.org/attachment.cgi?id=446404&action=review > > > Source/WebCore/Modules/mediasource/README.md:54 > > + > > Maybe we can add some information about SourceBufferPrivateAVFObjC and > SourceBufferParser? Good idea! I'll do that in a follow-up patch.
Committed r286738 (244988@main): <https://commits.webkit.org/244988@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 446412 [details].
<rdar://problem/86227732>
Reopening to attach new patch.
Created attachment 446421 [details] [fast-cq] Follow-up patch
ChangeLog entry in Source/WebCore/ChangeLog contains OOPS!.
Created attachment 446430 [details] [fast-cq] Follow-up patch
Committed r286742 (244989@main): <https://commits.webkit.org/244989@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 446430 [details].