MediaSource's AVFObjC engine was previously being loaded for interpreting MediaStreams. Making sure that it doesn't get loaded.
<rdar://problem/21735416>
Created attachment 256418 [details] Patch
Comment on attachment 256418 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=256418&action=review Looks good. Please address my concerns and we can get this landed! > Source/WebCore/ChangeLog:8 > + Add a statement like: "Prevent the MediaSource engine from being used to process MediaStreams, since they are not compatible" > Source/WebCore/ChangeLog:10 > + Changed MediaStream to never be loaded Move this explanation down to line 11, where you were saying "Ditto". > Source/WebCore/ChangeLog:11 > + (WebCore::MediaPlayerPrivateAVFoundationObjC::supportsType): Ditto Remove Ditto from this line. > Source/WebCore/ChangeLog:14 > + * platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm: Ditto Remove 'Ditto' from this line. > Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h:99 > + virtual void load(MediaStreamPrivate*) override { setNetworkState(MediaPlayer::FormatError); } No need for virtual here. "override" takes care of that. I think you should move this implementation inside of MediaPlayerPrivateMediaSourceAVFObjC.mm.
Created attachment 256423 [details] Patch
Created attachment 256424 [details] Patch
Comment on attachment 256424 [details] Patch r=me. We can cq+ this if the tests all pass.
Comment on attachment 256424 [details] Patch Clearing flags on attachment: 256424 Committed r186563: <http://trac.webkit.org/changeset/186563>
All reviewed patches have been landed. Closing bug.