[Laszlo-checkins] r16691 - openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml

[max@openlaszlo.org]

Author: max
Date: 2010-06-10 08:25:49 -0700 (Thu, 10 Jun 2010)
New Revision: 16691

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
Log:
Change 20100610-maxcarlson-p by maxcarlson at Friendly.local on 2010-06-10 09:03:59 MDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix rotation in Chrome DHTML.

Bugs Fixed: LPP-9116 - DHTML: rotation not working in Chrome 5

Technical Reviewer: hminsky
QA Reviewer: ptw

Details: setRotation() should rely on the capabilities system - there's no need to check to see which browser is running.

Tests:  This works in Chrome DHTML:
<canvas> 
    <text onclick="this.animate('rotation', 360, 1000, true)">Hello Laszlo!</text>
</canvas>



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js	2010-06-10 15:07:32 UTC (rev 16690)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js	2010-06-10 15:25:49 UTC (rev 16691)
@@ -2894,10 +2894,7 @@
 }
 
 LzSprite.prototype.setRotation = function(r) {    
-    var browser = lz.embed.browser;
-    if (browser.isSafari || browser.isFirefox) {
-        this.__LZdiv.style[LzSprite.__styleNames.transform] = 'rotate(' + r + 'deg)';
-    }
+    this.__LZdiv.style[LzSprite.__styleNames.transform] = 'rotate(' + r + 'deg)';
 }
 
 LzSprite.prototype.backgroundrepeat = null;