In theory, there is no way to tell if a text widget is in a container that is 
growing to fit its content and laying out several times or not.  I've seen some 
corner cases where things "settle down" after several attempts.

So, a bit risky, but we'll see if the other tests pass.

-Alex
________________________________________
From: Justin Mclean <[email protected]>
Sent: Monday, February 17, 2014 3:08 PM
To: [email protected]
Subject: Fix for infinite loop/RTE 1502 when resizing text

Hi,

Just check in a work around for the adding/removing scrollbar infinite loop.  
The checkin tests, Scroller, DropDown and RichText mustella tests pass. Someone 
mind reviewing it and seeing if it's a good solution.

Basically I change the code to count how many times updateDislayList is calls 
in the infinite loop situation (adding both scrollbars but one doesn't fit) and 
if it loops a few times then just give up and not place any scroll bar. There 
already existed similar code to prevent updateDisplayList from looping in 
another situation.

Thanks,
Justin

Reply via email to