Bug 1850293 - Don't use IntrinsicState() in VideoFrame constructor. r…

…=smaug

State() is cheaper, and non-virtual, and should be up-to-date. I'm
removing the concept of "intrinsic" states and this caught my eye.

Differential Revision: https://phabricator.services.mozilla.com/D187036

dom/media/webcodecs/VideoFrame.cpp

@@ -1174,7 +1174,7 @@ already_AddRefed<VideoFrame> VideoFrame::Constructor(
   }
 
   // Check the usability.
-  if (aImageElement.IntrinsicState().HasState(ElementState::BROKEN)) {
+  if (aImageElement.State().HasState(ElementState::BROKEN)) {
     aRv.ThrowInvalidStateError("The image's state is broken");
     return nullptr;
   }
@@ -1233,7 +1233,7 @@ already_AddRefed<VideoFrame> VideoFrame::Constructor(
   }
 
   // Check the usability.
-  if (aSVGImageElement.IntrinsicState().HasState(ElementState::BROKEN)) {
+  if (aSVGImageElement.State().HasState(ElementState::BROKEN)) {
     aRv.ThrowInvalidStateError("The SVG's state is broken");
     return nullptr;
   }