[Laszlo-dev] For Review: Change 20080507-Philip-8. Summary:, runlzteset fails because isIE is missing

[André Bargull]

Not approved!
First include in "Library.lzs" is now "LFC.js" and in this file, there 
is the definition of the global "getTimer()"-function, which refers to 
LzTimeKernel. So you cannot make LzTimeKernel the last include without 
further adjustments.

> Change 20080507-Philip-8 by Philip at Philip-DC on 2008-05-07 20:54:59 EDT
>      in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk
>      for http://svn.openlaszlo.org/openlaszlo/trunk
>
> Summary: runlzteset fails because isIE is missing
>
> New Features:
>
> Bugs Fixed: LPP-5954
>
> Technical Reviewer: max
> QA Reviewer: (pending)
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details:
> The browser detect code at the front of LzTimeKernel.js fails in 
> rhino because Lz.__BrowserDetect does not exist. My first solution 
> was to modify rhino.js, but this causes other code to execute in the 
> kernel (in bLzSprite.__updateQuirks()). I added a new quirk and it is 
> only true for IE. LzTimKernel.js was the first loaded
> kernel file and I moved it to the last because it now refers to 
> something in LzSprite.
>
> Library.lzs: Move LzTimeKernel to the end to remove forward reference
> LzSprite.js: Add ie_timer_closure quirk which is true for IE
> LzTimeKernel.js: use ie_timer_closure quirk instead of Lz.__BrowserDetect
>
>
> Tests:
> The isIE error is gone from running buildtools/runlztest.sh. I see 
> other errors when I run this but they don't appear to be caused by this issue.
>
> Files:
> M      WEB-INF/lps/lfc/kernel/dhtml/LzTimeKernel.js
> M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
> M      WEB-INF/lps/lfc/kernel/dhtml/Library.lzs
>
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20080507-Philip-8.tar