Set default size of video in media document as 300 * 150
authoreojin.ham <eojin.ham@samsung.com>
Sat, 4 May 2013 08:35:58 +0000 (17:35 +0900)
committerGerrit Code Review <gerrit2@kim11>
Mon, 6 May 2013 02:22:48 +0000 (11:22 +0900)
[Title] Set default size of video in media document as 300 * 150
[Issue#] https://tizendev.org/bugs/browse/N_SE-37219
[Problem] Video is shown as thin line in media document when failed to get natural video size.
[Cause] It set default size of video in media document as 300 * 1.
[Solution] Set default size of video in media document as 300 * 150 when it has video stream.

Change-Id: I48fce28b8e79ace772185f3353ebb0d33c8f45ab

Source/WebCore/rendering/RenderVideo.cpp

index 044fbd6..1dcfa8d 100644 (file)
@@ -133,7 +133,11 @@ IntSize RenderVideo::calculateIntrinsicSize()
     // size since they also have audio-only files. By setting the intrinsic
     // size to 300x1 the video will resize itself in these cases, and audio will
     // have the correct height (it needs to be > 0 for controls to render properly).
+#if ENABLE(TIZEN_GSTREAMER_VIDEO)
+    if (video->ownerDocument() && video->ownerDocument()->isMediaDocument() && player && !player->hasVideo())
+#else
     if (video->ownerDocument() && video->ownerDocument()->isMediaDocument())
+#endif
         return IntSize(defaultSize().width(), 1);
 
     return defaultSize();