[Laszlo-dev] For Review: Change 20081127-bargull-2Ji Summary: misc data changes (II - swf-kernel)

[André Bargull]

Change 20081127-bargull-2Ji by bargull at dell--p4--2-53 on 2008-11-27 10:07:23
in /home/Admin/src/svn/openlaszlo/trunk
for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: misc data changes (II - swf-kernel)

New Features:

Bugs Fixed: LPP-7350

Technical Reviewer: hminsky
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
LzLibraryLoader:
- marked overriden methods
LzMediaLoader:
- added "minHeader" attribute (was declared in LzLoader, but actually 
only used for LzMediaLoader)
LzLoader:
- extend from LzEventable, because it takes part in the event-system
- removed stale attributes:
- doPersist: was used for persistent connection
- attachDepth/minHeader: only used in LzMediaLoader
- removed all xml-loading specific methods/attributes
- copyTaskDel, startCopyTask(), removeCopyTask()
- oncontent, requestheaders, setHeaders()
- requestDirectXML(), rawtext, xmlloadobj, gotRawData()
- copyFlashXML(), copyQueue, translateXML(), copyFlashData(), 
queuedCopyFlashXML(), copyLoopsPerFrame, maxLoopTime, 
queuedCopyFlashXML_internal()
Motivation: the general LzLoader class does not need to know how to load 
and parse xml-data. Therefore I've created a specific subclass 
(XMLLoader). Similar to the LzMediaLoader class, which is specific for 
media-loading.
LzXMLLoader:
- contains all xml-code from LzLoader
- renamed requestDirectXML() to request()
- also moved LzLoadQueue.XMLOnDataHandler to this class
LzXMLTranslator:
- call LzXMLLoader instead of LzLoader to translate XML
LzLoadQueue:
- updated code to use LzXMLLoader
- use encodeURIComponent() instead of escape() for string-encoding 
(encodeURIComponent() is just an alias for escape() in the swf8-runtime, 
but nevertheless we always call encodeURIComponent(), so let's be 
consistent and call the function under that name)
- renamed loadMovieProxiedOrDirect() to loadMovie()
- moved XMLOnDataHandler to LzXMLLoader (see above)
- updated description
LzHTTPLoader:
- use LzXMLLoader instead of LzLoader
- removed the whole dataloader-movieclip code, we don't use that anymore
- removed LzHTTPLoader.urlencode(), no where referenced and we don't 
need another alias for escape() ;-)



Tests:
alldata passes

Files:
M WEB-INF/lps/lfc/kernel/swf/LzXMLTranslator.as
M WEB-INF/lps/lfc/kernel/swf/LzLoader.lzs
M WEB-INF/lps/lfc/kernel/swf/LzMediaLoader.lzs
M WEB-INF/lps/lfc/kernel/swf/LzLoadQueue.as
A WEB-INF/lps/lfc/kernel/swf/LzXMLLoader.lzs
M WEB-INF/lps/lfc/kernel/swf/LzLibraryLoader.lzs
M WEB-INF/lps/lfc/kernel/swf/Library.lzs
M WEB-INF/lps/lfc/kernel/swf/LzHTTPLoader.as

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20081127-bargull-2Ji.tar