[Laszlo-dev] For Review: Change 20090322-sallen-F Summary: videoplayer improvements and documentation

[Sarah Allen]

test-videoplayer-http.lzx is in the changeset.  I don't think that  
warning is new, but I'll check

On Mar 23, 2009, at 6:26 PM, Max Carlson wrote:

> Hmm - test-stream-video-sep.lzx shows an error in the debugger:
> WARNING @test-stream-video-sep.lzx#18: No netconnection defined for  
> #s.
>
> and test-videoplayer-http.lzx isn't checked in - can you add it?   
> Thanks!
>
> Sarah Allen wrote:
>> Hey Max or Raju,
>> Can you look over these videoplayer changes?  the mediastream  
>> changes are just additional debugging.  Would also love help  
>> figuring out how to fix scrubbing and that annoying thing where  
>> playing isn't set to false when the video stops playing, but those  
>> are unrelated to this change.
>> Thanks!
>> Sarah
>> Change 20090322-sallen-F by sallen at sallen-mac.local <mailto:sallen at sallen-mac.local 
>> > on 2009-03-22 13:23:40 PDT
>>    in /Users/sarah/src/svn/openlaszlo/trunk-sandbox
>>    for http://svn.openlaszlo.org/openlaszlo/trunk
>> Summary: videoplayer improvements and documentation
>> New Features:
>> LPP-7956 should be able to show first frame with videoplayer  
>> component
>> LPP-7952 videoplayer: should be able to hide time display and  
>> volume control
>> LPP-7960 videoplayer needs a way to maintain aspect ratio for  
>> videoplayer
>> Technical Reviewer: (pending)
>> QA Reviewer: (pending)
>> Doc Reviewer: (pending)
>> Documentation:
>>    Added in-lne docs for videoplayer component
>> Release Notes:
>> Details:
>> For some reason, the stream's playing attribute is not set to false  
>> in test-videoplayer-http.lzx
>> However, in test-stream-video-rtmp-sep.lzx with the same video file  
>> it works. I don't believe it is related to any of my changes, but  
>> the behavior is annoying combined with the fact that scrubbing
>> the video is broken (and has been for quite a while, if not forever).
>> Tests:
>> Files:
>> A      test/video/test-videoplayer-http.lzx
>> M      test/video/list.html
>> M      lps/components/extensions/av/mediastream.lzx
>> M      lps/components/av/videoplayer.lzx
>> M      lps/components/av/videoscreen.lzx
>> M      lps/components/av/videoslider.lzx
>> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090322-sallen-F.tar
>
> -- 
> Regards,
> Max Carlson
> OpenLaszlo.org