[Bug 743908] simplevideomark: crash when launch with max property values

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=743908
  GStreamer | gst-plugins-bad | unspecified

Vineeth <vineeth.tm> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #296270|none                        |needs-work
             status|                            |

--- Comment #2 from Vineeth <vineeth.tm at samsung.com> 2015-02-09 04:50:50 UTC ---
Review of attachment 296270:
 --> (https://bugzilla.gnome.org/review?bug=743908&attachment=296270)

Hi,

This way of fixing wont solve the actual issue.
Instead of throwing error, when req_width and height are more than video width
and height,
we should recalculate the actual drawable width and height and display the
partially visible video mark, in case half the width/height are outside video
width and height, or not show the videomark in case it exists completely
outside the video width and height.
You can refer to how textoverlay is handled to see the expected behavior.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.