Max,

Line 236 has a typo:

             if (! lz.embed[iframe.owner) {

should have been

             if (! lz.embed[iframe.owner]) {

notice the missing ]. With the added ] it works as expected.

-Maynard

Max Carlson wrote:
> Change 20100119-maxcarlson-o by [email protected] on 2010-01-19 13:52:28 
> MST
>     in /Users/maxcarlson/openlaszlo/trunk-clean
>     for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Add test to prevent null dereferencing in iframemanager
> 
> Bugs Fixed: LPP-8722 -  javascript error in iframemanager.js when prompted to 
> upgrade the flash player.
> 
> Technical Reviewer: ptw
> QA Reviewer: mdemmon
> 
> Details: Add a test to prevent null dereferencing of the property 
> Lz[iframe.owner].
> 
> Tests: See LPP-8722
> 
> Files:
> M       lps/includes/source/iframemanager.js
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20100119-maxcarlson-o.tar


_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to