From 8866a7c2231a4c3ff656f929177fe4e739cef49f Mon Sep 17 00:00:00 2001
From: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Date: Mon, 9 Jan 2012 16:51:35 +0100
Subject: [PATCH] vaapisink: fix calculation of render region.

---
 gst/vaapi/gstvaapisink.c | 52 +++++++++++++-----------------------------------
 1 file changed, 14 insertions(+), 38 deletions(-)

diff --git a/gst/vaapi/gstvaapisink.c b/gst/vaapi/gstvaapisink.c
index 6b82cfa..252d5f2 100644
--- a/gst/vaapi/gstvaapisink.c
+++ b/gst/vaapi/gstvaapisink.c
@@ -293,7 +293,6 @@ gst_vaapisink_ensure_render_rect(GstVaapiSink *sink, guint width, guint height)
 {
     GstVaapiRectangle * const display_rect = &sink->display_rect;
     guint num, den, display_par_n, display_par_d;
-    double display_ratio;
     gboolean success;
 
     GST_DEBUG("ensure render rect within %ux%u bounds", width, height);
@@ -316,47 +315,24 @@ gst_vaapisink_ensure_render_rect(GstVaapiSink *sink, guint width, guint height)
     GST_DEBUG("video size %dx%d, calculated ratio %d/%d",
               sink->video_width, sink->video_height, num, den);
 
-    if ((sink->video_height % den) == 0) {
-        GST_DEBUG("keeping video height");
-        display_rect->width  =
-            gst_util_uint64_scale_int(sink->video_height, num, den);
-        display_rect->height = sink->video_height;
-    }
-    else if ((sink->video_width % num) == 0) {
-        GST_DEBUG("keeping video width");
-        display_rect->width  = sink->video_width;
-        display_rect->height =
-            gst_util_uint64_scale_int(sink->video_width, den, num);
+    display_rect->width = gst_util_uint64_scale_int(height, num, den);
+    if (display_rect->width <= width) {
+        GST_DEBUG("keeping window height");
+        display_rect->height = height;
     }
     else {
-        GST_DEBUG("approximating while keeping video height");
-        display_rect->width  =
-            gst_util_uint64_scale_int(sink->video_height, num, den);
-        display_rect->height = sink->video_height;
-    }
-    display_ratio = (gdouble)display_rect->width / display_rect->height;
-    GST_DEBUG("scaling to %ux%u", display_rect->width, display_rect->height);
-
-    if (sink->fullscreen || sink->foreign_window ||
-        display_rect->width > width || display_rect->height > height) {
-        if (sink->video_width > sink->video_height) {
-            display_rect->width  = width;
-            display_rect->height = width / display_ratio;
-        }
-        else {
-            display_rect->width  = height * display_ratio;
-            display_rect->height = height;
-        }
+        GST_DEBUG("keeping window width");
+        display_rect->width  = width;
+        display_rect->height =
+            gst_util_uint64_scale_int(width, den, num);
     }
+    GST_DEBUG("scaling video to %ux%u", display_rect->width, display_rect->height);
 
-    if (sink->fullscreen) {
-        display_rect->x  = (width  - display_rect->width)  / 2;
-        display_rect->y  = (height - display_rect->height) / 2;
-    }
-    else {
-        display_rect->x  = 0;
-        display_rect->y  = 0;
-    }
+    g_assert(display_rect->width  <= width);
+    g_assert(display_rect->height <= height);
+
+    display_rect->x = (width  - display_rect->width)  / 2;
+    display_rect->y = (height - display_rect->height) / 2;
 
     GST_DEBUG("render rect (%d,%d):%ux%u",
               display_rect->x, display_rect->y,
-- 
2.7.4