[Laszlo-dev] For Review: Change 20071113-maxcarlson-3 Summary: Fix the youtube player

[J Crowley]

Approved.

This still doesn't fix ALL the videos, but it gets *most* of them to  
work (all but four of the initial results), which is better than  
none.  If you add "&origin=youtube.com" to the URL conditionally  
(otherwise it breaks half of them; I'm not *entirely* sure what the  
conditions are that require it), it'll get two of the four remaining  
non-working videos to play.  (The stubborn two seem to be the first  
one and Andy McRee playing Rylynn.)

I guess file a bug to get the remaining videos to work, but this is  
great for right now.

On Nov 13, 2007, at 4:56 PM, Max Carlson wrote:

> Change 20071113-maxcarlson-3 by maxcarlson at Roboto on 2007-11-13  
> 13:54:12 PST
>    in /Users/maxcarlson/openlaszlo/trunk
>    for http://svn.openlaszlo.org/openlaszlo/trunk
>
> Summary: Fix the youtube player
>
> New Features:
>
> Bugs Fixed: LPP-5070 - Videos not playing in YouTube demo
>
> Technical Reviewer: jcrowley
> QA Reviewer: promanik
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details: youtube.jsp - Add support for passing the 't' parameter  
> though to get_video.
>
> youtube.lzx - Use proxied mode - the server helps with http redirect  
> calls that sometimes get returned from get_video.
>
>
> Tests: youtube.lzx mostly works now.
>
> Files:
> M      demos/youtube/youtube.jsp
> M      demos/youtube/youtube.lzx
>
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071113-maxcarlson-3.tar