[FIX]Re: BrowserCommentTextMorph Re: 3.9 browser speed

karl karl.ramberg at chello.se
Sun Nov 5 21:21:58 UTC 2006


karl skrev:
> karl skrev:
>> Andreas Raab skrev:
>>> Pavel Krivanek wrote:
>>>> the reason will be in the gradient background of windows. The
>>>> background is processed during every update now but the previous
>>>> versions were more optimized.
>>>
>>> Good theory, but I don't think this is the problem. When I turn off 
>>> both TTFs and gradients in 3.9 I still get something like here:
>>>
>>>     Run 1:  811 msecs
>>>     Run 2: 1191 msecs
>>>     Run 3: 1483 msecs
>>>     Run 4: 1867 msecs
>>>
>>> It's a little better but not significantly (and it still feels way 
>>> too slow). BTW, if anyone wants to repeat the experiments, the 
>>> changes that I did for this test were:
>>> * setting the window title font to a non-TTF (Accuny 14 in my case)
>>> * change SystemWindow>>gradientWithColor: to return just the color
>>> So I'm still at a loss what has changed to introduce that slowdown.
>>>
>>> Hm ... let me try something. Oh, yes, now *that* would explain it. 
>>> Turn on #debugShowDamage in 3.9 and watch how it redraws the entire 
>>> screen almost every time you click anywhere in a browser. Ouch. No 
>>> idea why that is but it sure explains the effect. 
>> Commenting out some code  in BrowserCommentTextMorph reduces the 
>> bogus invalidation, so we have to debug the class 
>> BrowserCommentTextMorph to see what make it behave so bad.
>> Karl
>>
>>
> I tracked the issue down to SystemWindows>>addPaneSplitters  which 
> removes  and adds pane splitters when the comment pane is added to the 
> bottom of the  browser.
> I'm not sure how to fix the issue, maybe hiding and showing the pane 
> is better than deleting and reinstalling it.
> The root cause of the problem seem to be 
> Morph>>privateAddMorph:atIndex: which do a lot of invaliditation that 
> cause a massive redraw when it's sent.
> Karl
>
>
BorderedMorph wanted more specific position of the morphs added, 
otherwise they would be added to the top left corner. Hope this cures it :-)

Karl


-------------- next part --------------
A non-text attachment was scrubbed...
Name: BorderedMorph.kfr.2.cs.gz
Type: application/x-gzip
Size: 974 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20061105/c267c63f/BorderedMorph.kfr.2.cs.bin


More information about the Squeak-dev mailing list