Bug 1240478 - Assert that we know the video size on 'loadedmetadata'. r=jesup

2016-01-21 19:42:55 +08:00
parent 8b4764234c
commit 0222a03f29
1 changed files with 4 additions and 0 deletions
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -3397,6 +3397,10 @@ void HTMLMediaElement::MetadataLoaded(const MediaInfo* aInfo,
  if (IsVideo() && HasVideo()) {
    DispatchAsyncEvent(NS_LITERAL_STRING("resize"));
  }
+  NS_ASSERTION(!HasVideo() ||
+               (mMediaInfo.mVideo.mDisplay.width > 0 &&
+                mMediaInfo.mVideo.mDisplay.height > 0),
+               "Video resolution must be known on 'loadedmetadata'");
  DispatchAsyncEvent(NS_LITERAL_STRING("loadedmetadata"));
  if (mDecoder && mDecoder->IsTransportSeekable() && mDecoder->IsMediaSeekable()) {
    ProcessMediaFragmentURI();