There's a bunch of code that uses HTMLMediaElement in VideoFrameContainer. Most of this is layout specific. If we can factor that out, say back into HTMLMediaElement which knows all about how Gecko does layout, then it's easier to import VideoFrameContainer into the gecko-media Rust crate.

Comment on attachment 8927742 [details] Bug 1416663 - Move Gecko/HTMLMediaElement specific stuff out of VideoFrameContainer. https://reviewboard.mozilla.org/r/199012/#review204034 C/C++ static analysis found 1 defect in this patch. You can run this analysis locally with: `./mach static-analysis check path/to/file.cpp` ::: dom/html/HTMLMediaElement.cpp:6412 (Diff revision 1) > return mDecoderPrincipalChangeObservers.RemoveElement(aObserver); > } > > +void > +HTMLMediaElement::Invalidate(bool aImageSizeChanged, > + Maybe<nsIntSize> aNewIntrinsicSize, Error: Type 'maybe<nsintsize>' (aka 'maybe<intsizetyped<mozilla::gfx::unknownunits> >') must not be used as parameter [clang-tidy: mozilla-non-memmovable-template-arg]

Comment on attachment 8927742 [details] Bug 1416663 - Move Gecko/HTMLMediaElement specific stuff out of VideoFrameContainer. https://reviewboard.mozilla.org/r/199012/#review204032 ::: dom/media/MediaDecoderOwner.h:184 (Diff revision 2) > } > > + // Called by the frame container to notify the layout engine that the > + // size of the image has changed, or the video needs to be be repainted > + // for some other reason. > + virtual void Invalidate(bool aImageSizeChanged, Please provide default implementations for gecko-only functions as above.

Comment on attachment 8927742 [details] Bug 1416663 - Move Gecko/HTMLMediaElement specific stuff out of VideoFrameContainer. https://reviewboard.mozilla.org/r/199012/#review204040 C/C++ static analysis found 1 defect in this patch. You can run this analysis locally with: `./mach static-analysis check path/to/file.cpp` ::: dom/html/HTMLMediaElement.cpp:6412 (Diff revision 2) > return mDecoderPrincipalChangeObservers.RemoveElement(aObserver); > } > > +void > +HTMLMediaElement::Invalidate(bool aImageSizeChanged, > + Maybe<nsIntSize> aNewIntrinsicSize, Error: Type 'maybe<nsintsize>' (aka 'maybe<intsizetyped<mozilla::gfx::unknownunits> >') must not be used as parameter [clang-tidy: mozilla-non-memmovable-template-arg]

Pushed by cpearce@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/442c830d2109 Move Gecko/HTMLMediaElement specific stuff out of VideoFrameContainer. r=jwwang