Approved!

Philip Romanik wrote:
Change 20080214-Philip-4 by [EMAIL PROTECTED] on 2008-02-14 08:54:44 EST
    in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Access text cache as LzTextSprite.prototype._sizecache

New Features:

Bugs Fixed: LPP-5390

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

Documentation:

Release Notes:

Details:
The code used to access the global cache through 'this'. However, once the cache is full and is cleared, access through 'this' it not reliable. I changed this._ sizecache to LzTextSprite.prototype._sizecache. Although I only need to do this for the line that actually resets the cache, I changed all references to make it
 clear this is a global.


Tests:
See test in LPP-5390. This is slow to run since you need to wait until the _size cache.counter reaches 1000. During testing I set the cache size to 100 (ie. LzTe xtSprite.prototype.__sizecacheupperbound = 100) to verify the bug and the fix.

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20080214-Philip-4.tar


--
Regards,
Max Carlson
OpenLaszlo.org

Reply via email to