bugGnash - The GNU Flash player - Bugs: bug #23130, new "youtube beta"...

 
 

bug #23130: new "youtube beta" doesn't work with GST backend

Submitted by:  Robert Millan <robertmh>
Submitted on:  Thu 01 May 2008 06:23:16 PM UTC  
 
Category: video-gstSeverity: 4 - Important
Release: NoneStatus: Fixed
Privacy: PublicAssigned to: Bastiaan Jacques <bjacques>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Tue 02 Dec 2008 02:54:47 PM UTC, comment #16:

It's fine with gst.

Benjamin Wolsey <bwy>
Project Member
Thu 11 Sep 2008 12:57:01 PM UTC, comment #15:

works with ffmpeg...

Sandro Santilli <strk>
Project Member
 Spam posted by an anonymous
 Spam posted by an anonymous
Sat 17 May 2008 10:06:57 AM UTC, comment #14:

After ubuntu upgrade (and possibly youtube upgrade too)
I can now handle to see the video IFF I request video
restart (Movie->Restart). I guess it's still due to events
and changes based on browser cache.
I noted the buffers don't feel if we press 'pause'.
Maybe adding proper support for buffering would fix this
(and only pausing playback, not queuing or full pipeline).

Sandro Santilli <strk>
Project Member
Tue 13 May 2008 10:08:10 AM UTC, comment #13:

The issue is VideoPlayer.seekTo calls VideoPlayer.play
which calls VideoPlayer.seekTo again and again.

R1.seekTo = function (seconds /=R2/ ,allowSeekAhead /=R3/ ) {
if( !this.videoLoader.isMetaDataLoaded() ) {
this.getCurrentVideoData().startSeconds = seconds;
this.getCurrentVideoData().startingFromOffset = true;
this.play();

Commenting out the call to .play() in watch-hacked.flm fixes it:

push r:1, 'seekTo'
function2 (r:2='seconds', r:3='allowSeekAhead') (r:1='this')
push 'CALL seekTo'
trace
push 0, r:this, 'videoLoader'
getMember
push 'isMetaDataLoaded'
callMethod
not
not
branchIfTrue label10
push 0, r:this, 'getCurrentVideoData'
callMethod
push 'startSeconds', r:seconds
setMember
push 0, r:this, 'getCurrentVideoData'
callMethod
push 'startingFromOffset', TRUE
setMember
//push 0, r:this, 'play'
//callMethod
//pop
branch label11
label10:

Anonymous
Mon 12 May 2008 04:23:56 PM UTC, comment #12:

FYI:

Not sending any event fixes the runs with gstreamer for me.
Tested by commenting out line 521 in NetStream.cpp (push of status to _statusQueue in setStatus).

Sandro Santilli <strk>
Project Member
Mon 12 May 2008 03:59:14 PM UTC, comment #11:

I use Mandriva and have the same problem with two different versions: one running gst 0.10.14 and one with 0.10.19. Ffmpeg works fine, gstreamer doesn't.

Benjamin Wolsey <bwy>
Project Member
Mon 12 May 2008 03:48:55 PM UTC, comment #10:

How can I extract version of the gnomevfs plugin being used by gnash ?

Sandro Santilli <strk>
Project Member
Wed 07 May 2008 06:00:09 PM UTC, comment #9:

Is it expected to get no ERROR lien from gnash ?

Sandro Santilli <strk>
Project Member
Wed 07 May 2008 05:56:35 PM UTC, comment #8:

That movie works fine here. If it doesn't for you then I'm almost certain you've fallen into the GnomeVFS trap: the latest release of GnomeVFS has a bug that breaks certain URLs; there's a patch in the development branch, which has made it into Ubuntu, I'm told. I'm also told Ubuntu is the only distribution with this issue so far.

Bastiaan Jacques <bjacques>
Project MemberIn charge of this item.
Wed 07 May 2008 05:24:53 PM UTC, comment #7:

Bastiaan, I'm tracking this:
http://www.youtube.com/watch?v=cN6TiQ5w8V0

Not sure if it's the beta or not, probably not, there
are others. Anyway, the above is what I'm tracking
and revealing that bug.

Sandro Santilli <strk>
Project Member
Wed 07 May 2008 05:17:11 PM UTC, comment #6:

Hm, I don't see the "try the new beta" link...

Bastiaan Jacques <bjacques>
Project MemberIn charge of this item.
Wed 07 May 2008 05:08:24 PM UTC, comment #5:

So, it turns out ffmpeg handles to get to a proper
get_video call, while gstreamer does'nt.
For gst, it seems the code thinks the movie is over immediately.

bug #22735 (comment 3) contains some info that seem related

Sandro Santilli <strk>
Project Member
Wed 07 May 2008 01:33:59 PM UTC, comment #4:

super in interval callbacks should be fixed now.
there still isn't a super.onProgress, dunno if it's
ok or not. Ming decompiler isn't helping to understand
the code

Sandro Santilli <strk>
Project Member
Wed 07 May 2008 12:18:26 PM UTC, comment #3:

Next problem:

methods called by setInterval fail to set a 'super'

Here's a quick testcase:

function A() {};
A.prototype.test = function() { trace('A.test'); };

function B() {};
B.prototype = new A;
B.prototype.test = function() { super.test(); trace('B.test'); };

o = new B;

o.test();

this.loaderInterval = _global.setInterval(o,'test',250);

Sandro Santilli <strk>
Project Member
Sat 03 May 2008 12:20:00 AM UTC, comment #2:

First problem fixed. Next ?

Sandro Santilli <strk>
Project Member
Fri 02 May 2008 11:47:39 PM UTC, comment #1:

First problem is Object.registerClass looking in the
wrong symbol library. Dunno if it's supposed to exist
a single or multiple libraries yet.

Sandro Santilli <strk>
Project Member
Thu 01 May 2008 06:23:16 PM UTC, original submission:

Youtube now features a link titled "Try the New YouTube Player Beta!" below every video (can't provide a direct link, as it calls some javascript magic to set user settings).

When Gnash attempts to play a video using the Beta version, it stalls.

Robert Millan <robertmh>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by bwy (Posted a comment)
  • -unavailable- added by bjacques (Posted a comment)
  • -unavailable- added by strk (Posted a comment)
  • -unavailable- added by robertmh (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 12 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 02 Dec 2008 02:54:47 PM UTCbwyStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 30 Sep 2008 10:38:28 PM UTCstrkStatusIn Progress=>Ready For Test
    Thu 11 Sep 2008 12:57:01 PM UTCstrkCategoryActionScript=>video-gst
      StatusConfirmed=>In Progress
      Summarynew "youtube beta" doesn\'t work=>new "youtube beta" doesn't work with GST backend
    Wed 07 May 2008 05:08:24 PM UTCstrkStatusIn Progress=>Confirmed
      Assigned tostrk=>bjacques
    Fri 02 May 2008 11:47:39 PM UTCstrkCategoryNone=>ActionScript
      Severity3 - Normal=>4 - Important
      StatusNone=>In Progress
      Assigned toNone=>strk

    Back to the top


    Powered by Savane 3.1-cleanup1