bugGnash - The GNU Flash player - Bugs: bug #25766, Size of video is wrong with XVideo...

 
 

bug #25766: Size of video is wrong with XVideo enabled

Submitter:  Julien <mjules>
Submitted:  Wed 04 Mar 2009 07:24:28 PM UTC
   
 
Category:  video Severity:  3 - Normal
Release:  trunk Status:  Wont Fix
Privacy:  Public Assigned to:  bjacques
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 12 Mar 2012 07:23:28 AM UTC, comment #4: 

No more XVideo support in master branch..
commit:67551e3 cleaned up the last bits

Sandro Santilli <strk>
Group Member
Fri 28 May 2010 03:49:03 PM UTC, comment #3: 

I think there is little we can do about this bug:
- X.org does not (yet) support drawing XVideo to an offscreen buffer (pixmap). We can thus not scale a (streaming) video independently of the rest of rendering.
- Agg can't draw using Xlib; we thus don't have the option of synchronously drawing to the X window, which would ensure that the image(s) get drawn in the correct order.

Both issues could be resolved upstream.

Bastiaan Jacques <bjacques>
Group Member
Sun 14 Mar 2010 02:56:33 AM UTC, comment #2: 

I've analysed this problem in some detail now.

XVideo is used currently to do all of Gnash's scaling. This job is normally done by the renderer, but in the Xv case, the renderer is told not to do any scaling. So, the rendered image is uploaded to the Xv port with the appropriate scaling information.

In the case of YouTube, however, the situation is not so pleasant. YouTube monitors the (browser) window for resize events, and responds to them by changing the Movie scale accordingly. The renderer is never explicitly informed of this, but it is generally told by the GUI that it should render to a larger or smaller area. The only difference now is that the Shapes that the renderer is asked to draw suddenly have a changed transformation matrix.

Now back to the Xv case, where we tell the renderer not to do scaling. In the YouTube case, the renderer will still, however, receive an altered transformation matrix associated with Shapes to be renderered. These shapes, then, will no longer be drawn because they are outside of the area the renderer draws to.

Bastiaan Jacques <bjacques>
Group Member
Wed 04 Mar 2009 07:51:53 PM UTC, comment #1: 

Offhand I would say the issue is that Xv (deliberately) messes up scaling. I'm sure we can do something to mend the issue, though.

Bastiaan Jacques <bjacques>
Group Member
Wed 04 Mar 2009 07:24:28 PM UTC, original submission:  

Hi,

I successfully built gnash trunk with xvideo support in order to test and I show a bug :
size of the video is different with or without Xv.
for instance :
 http://icanhascheezburger.com/2009/02/25/funny-pictures-video-bunny-bed/

is smaller with Xv than without.
gnash has space reserved (and i can right click to have menu) but it only use a small part.

At the contrary, some others video are greater than the place of the player. See for example (yes it's rick roll'd but it's real anyway :D)
http://www.youtube.com/watch?v=oHg5SJYRHA0

I attach logs with verbosity at 3.

I'm using mandrivalinux 2008.1 x86_64 with Nouveau driver ( http://nouveau.freedesktop.org/ )

thanks
Julien

Julien <mjules>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #19936:  withoutXv.png added by rsavoye (178KiB - image/png - screenshots with and witout Xv)
file #19935:  withXv.png added by rsavoye (223KiB - image/png - screenshots with and witout Xv)
file #17584:  gnash-dbg.log-rickroll-with-xv added by mjules (30KiB - application/octet-stream - logs for the two video with and without Xv)
file #17585:  gnash-dbg.log-rickroll-without-xv added by mjules (26KiB - application/octet-stream - logs for the two video with and without Xv)
file #17586:  gnash-dbg.log-rabbit-with-xv added by mjules (29KiB - application/octet-stream - logs for the two video with and without Xv)
file #17587:  gnash-dbg.log-rabbit-without-xv added by mjules (24KiB - application/octet-stream - logs for the two video with and without Xv)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rsavoye (Updated the item)
  • -email is unavailable- added by strk (Updated the item)
  • -email is unavailable- added by bjacques (Posted a comment)
  • -email is unavailable- added by mjules (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 10 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-03-12 strk Open/ClosedOpen Closed
    2010-05-28 bjacques StatusConfirmed Wont Fix
    2010-03-13 rsavoye Attached File- Added withXv.png, #19935
        Attached File- Added withoutXv.png, #19936
    2009-03-24 strk StatusNone Confirmed
    2009-03-04 bjacques Assigned toNone bjacques
    2009-03-04 mjules Attached File- Added gnash-dbg.log-rickroll-with-xv, #17584
        Attached File- Added gnash-dbg.log-rickroll-without-xv, #17585
        Attached File- Added gnash-dbg.log-rabbit-with-xv, #17586
        Attached File- Added gnash-dbg.log-rabbit-without-xv, #17587

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code