google-code-export / flowplayer-core

Automatically exported from code.google.com/p/flowplayer-core
2 stars 0 forks source link

rtmp, bwcheck, autoPlay: false, autoBuffering: true freezes/crashes some browsers #610

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Google Chrome with Flash 11.3
2. http://flowplayer.blacktrash.org/test/bwcheck-onstart.html

What is the expected output? What do you see instead?

Expected: players stop at first frame.
Instead: Chrome freezes. Crash reported for IE here: 
http://flowplayer.org/forum/support.html?id=101424

Interestingly enough there is no problem with Flash 10.3 in older browsers(!)

P.S. I am aware that autoBuffering: true in connection with rtmp is highly 
problematic and questionable, with bwcheck obviously even more so. I can't tell 
whether allowing autoBuffering: true for rtmp becomes too hacky. If it does, we 
should be clear about disallowing it.

Original issue reported on code.google.com by blacktrashproduct on 24 Jul 2012 at 12:56

GoogleCodeExporter commented 9 years ago
Forgot to add: bwcheck's checkOnStart must be true for the above to happen.

Original comment by blacktrashproduct on 24 Jul 2012 at 1:00

GoogleCodeExporter commented 9 years ago
I discovered a bizarre crash which wasn't happening before, and only in chrome 
and was due to the frame seek during startup before pausing which I changed to 
a delay before pausing instead. 

Original comment by electrot...@gmail.com on 24 Jul 2012 at 5:21

GoogleCodeExporter commented 9 years ago
It is that problem i'm afraid, and strange it was never doing it before perhaps 
previous versions ???? I fixed this a while ago. 

https://dl.dropbox.com/u/3394987/flowplayer.rtmp-3.2.10.zip

Ive requested it be released once the changes to the slowmotion plugin were 
made. 

Original comment by electrot...@gmail.com on 24 Jul 2012 at 5:39

GoogleCodeExporter commented 9 years ago
You should confer with Anssi. This is now the 2nd instance of some muddle up in 
the release - controlbar + menu as well. I have no hand in the release process.

Original comment by blacktrashproduct on 24 Jul 2012 at 9:25

GoogleCodeExporter commented 9 years ago
You should confer with Anssi. This is now the 2nd instance of some muddle up in 
the release - controlbar + menu as well. I have no hand in the release process.

Isn't your change checked into svn? Isn't there a commit log where you should 
be able to check whether it got in in time?

Original comment by blacktrashproduct on 24 Jul 2012 at 9:26

GoogleCodeExporter commented 9 years ago
Hi as soon as the slow motion changes were locked in I did send an email ;) The 
controlbar was also supposed to be updated with the accessibility changes, I 
don't have control of this either ... There is no other system in place other 
than email ;)

Original comment by electrot...@gmail.com on 25 Jul 2012 at 6:56

GoogleCodeExporter commented 9 years ago
According to the customer in http://flowplayer.org/forum/support.html?id=101424
https://dl.dropbox.com/u/3394987/flowplayer.rtmp-3.2.10.zip
did not solve the problem - or rather the latest source compiled.

Original comment by blacktrashproduct on 25 Jul 2012 at 10:49

GoogleCodeExporter commented 9 years ago
Nope doesn't happen for me, perhaps its a cache problem. That is the problem I 
assure you ;)

Original comment by electrot...@gmail.com on 26 Jul 2012 at 6:12

GoogleCodeExporter commented 9 years ago
I confirm that https://dl.dropbox.com/u/3394987/flowplayer.rtmp-3.2.10.zip 
fixes the problem for me[tm].

Original comment by blacktrashproduct on 27 Jul 2012 at 9:43

GoogleCodeExporter commented 9 years ago
thnx. I dont have an answer why the 100ms seek was causing the issue in chrome. 
ok to close ? 

Original comment by electrot...@gmail.com on 27 Jul 2012 at 4:24

GoogleCodeExporter commented 9 years ago
i'll leave this open until the plugin is actually updated. It seems quite a few 
haven't been pushed live. 

Original comment by electrot...@gmail.com on 28 Jul 2012 at 11:18

GoogleCodeExporter commented 9 years ago
yeah, let's hope the release process becomes more reliable - this would indeed 
allow us to close bugs in dev with a clear conscience.

Original comment by blacktrashproduct on 29 Jul 2012 at 8:30

GoogleCodeExporter commented 9 years ago
let me know if the release file is ok though it looks like more changes are 
pending ;)

Original comment by electrot...@gmail.com on 10 Aug 2012 at 11:04

GoogleCodeExporter commented 9 years ago
I've updated
http://flowplayer.blacktrash.org/test/bwcheck-onstart.html
to the latest release and can confirm that the Chrome freeze is gone.

I don't have IE9 to test.

Original comment by blacktrashproduct on 10 Aug 2012 at 12:26

GoogleCodeExporter commented 9 years ago
Ok thanks, shall we close, there is some more pending commits to rtmp which 
will go through soon. 

Original comment by electrot...@gmail.com on 14 Aug 2012 at 10:07

GoogleCodeExporter commented 9 years ago
Let's close and pray ;-)

Original comment by blacktrashproduct on 20 Aug 2012 at 3:47