[Laszlo-checkins] r14321 - openlaszlo/trunk/lps/components/extensions/av

[max@openlaszlo.org]

Author: max
Date: 2009-07-14 16:10:35 -0700 (Tue, 14 Jul 2009)
New Revision: 14321

Modified:
   openlaszlo/trunk/lps/components/extensions/av/videoview.lzx
Log:
Change 20090714-maxcarlson-Y by maxcarlson at Bank.local on 2009-07-14 11:29:42 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix videoviews for paused/single frame videos

Bugs Fixed: LPP-8293 - test-video.lzx: sound plays but no video appears (swf8 & swf9)

Technical Reviewer: sarah at ultrasaurus.com
QA Reviewer: rajubitter at me.com

Details: Move URL setting before playback setup to ensure the stream isn't reset by the call to mediastream._resetStream() in mediastream._updateUrl().

Tests: test/video/test-video.lzx shows paused video on the left.  test/video/test-stream-video-sep.lzx pauses the video when 'set URL' buttons are clicked, and clicking 'play' causes the correct video to play.



Modified: openlaszlo/trunk/lps/components/extensions/av/videoview.lzx
===================================================================
--- openlaszlo/trunk/lps/components/extensions/av/videoview.lzx	2009-07-14 18:33:53 UTC (rev 14320)
+++ openlaszlo/trunk/lps/components/extensions/av/videoview.lzx	2009-07-14 23:10:35 UTC (rev 14321)
@@ -169,6 +169,9 @@
 
         this._urldel = new LzDelegate( this, "_setPlayStream", 
                                         this.stream, "onplaying")
+        if (this.url) {
+            this._updateStreamUrl();
+        }
         if (this.stream.playing || this.stream.type == 'http') {
             // show poster frame now for http streams
             if (!this.stream.playing && this.starttime != -1) {
@@ -176,9 +179,6 @@
             }
             this._setPlayStream();
         }
-        if (this.url) {
-            this._updateStreamUrl();
-        }
 
         if (this.cam) {
             this.stream.setAttribute('cam', this.cam);