[Gnash-commit] [patch #7961] Fix for issues with youtube in NetStream_as.cpp

[Rob Savoye]

Update of patch #7961 (project gnash):

                  Status:                    Done => In Progress            
             Open/Closed:                  Closed => Open                   

    _______________________________________________________

Follow-up Comment #2:

  I dug into the new testsuite failures caused by these last two patches. The
basics of these patches appears correct, but part of the problem is the
support for these isn't implemented in NetStream. For example,
NetStream::getCurrentFPS() is hardcoded to always return 0. There are some
weird behaviours going on as well. Although these are readonly properties, the
current NetStream.as test case tries to set them to both a string and boolean
value, and then expects the return value when the NetStream isn't connected to
be the same as the object it was previous set too. When the test case is run
with the Adobe player, it has this behaviour, so the tests pass. I'm not sure
if that needs to be the same behaviour in Gnash. The support for these new
properties needs to be implemented before most of these tests will pass.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7961>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/