[Laszlo-checkins] r16835 - openlaszlo/trunk/WEB-INF/lps/lfc/compiler

[max@openlaszlo.org]

Author: max
Date: 2010-06-29 15:16:34 -0700 (Tue, 29 Jun 2010)
New Revision: 16835

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/compiler/Class.lzs
Log:
Change 20100625-maxcarlson-L by maxcarlson at friendly on 2010-06-25 14:35:41 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Remove Safari-specific kludge for Safari 5.0

Bugs Fixed: LPP-8704 - Revert r15423 when webkit bug 33450 is resolved

Technical Reviewer: ptw
QA Reviewer: hminsky

Details: Conditionalize the hack for Safari 5 as it's not needed anymore.  I couldn't figure out the correct webkit version # for when the fix went in...

Tests: Recompiling the dev console works fine, examples/components/component_sampler.lzx?lzr=dhtml is fine



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/compiler/Class.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/compiler/Class.lzs	2010-06-29 20:06:31 UTC (rev 16834)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/compiler/Class.lzs	2010-06-29 22:16:34 UTC (rev 16835)
@@ -201,7 +201,8 @@
  * @access private
  */
 if ($dhtml) {
-  if (lz.embed.browser.isSafari && (lz.embed.browser.version == "531.21")) {
+  // This no longer appears to be present in Safari 5
+  if (lz.embed.browser.isSafari && (lz.embed.browser.version >= "531.21") && (lz.embed.browser.version < "533.16")) {
     Instance.prototype.addProperty('$lzsc$safarikludge', function $lzsc$safarikludge () {});
   }
 }