[Laszlo-dev] For Review: Change 20070821-maxcarlson-p Summary: Inline setAttribute

[Max Carlson]

Change 20070821-maxcarlson-p by maxcarlson at plastik on 2007-08-21  
13:13:26 PDT
     in /Users/maxcarlson/openlaszlo/wafflecone
     for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone

Summary: Inline setAttribute

New Features:

Bugs Fixed: LPP-4513 'Compiler Improvements to address performance'

Technical Reviewer: ptw
QA Reviewer: henry
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: LzContextMenu.js, LzTextSprite.js, LzInputTextSprite.js,  
drawview.lzx, embednew.js, iframemanager.js - Add #pragma  
"passThrough=true" to prevent div.setAttribute() from being inlined.

JavascriptGenerator.java - Inline calls to setAttribute(), unless  
#pragma "passThrough=true".

CodeGenerator.java - Inline calls to setAttribute(), unless #pragma  
"passThrough=true".


Tests: All demos run as before (but faster in swf and dhtml, but  
http://localhost:8080/wafflecone/test/smoke/smokecheck.lzx fails to  
compile.  Any suggestions on where to look would be appreciated.

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzContextMenu.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzInputTextSprite.js
M      WEB-INF/lps/server/src/org/openlaszlo/sc/JavascriptGenerator.java
M      WEB-INF/lps/server/src/org/openlaszlo/sc/CodeGenerator.java
M      lps/components/extensions/drawview.lzx
M      lps/includes/source/embednew.js
M      lps/includes/source/iframemanager.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070821- 
maxcarlson-p.tar