<Swing Dev> [12] RFR JDK-8213843: Changing L&F from Nimbus to Window L&F causes NPE in SwingSet2

[Prasanta Sadhukhan]

On 22-Nov-18 11:03 AM, Prasanta Sadhukhan wrote:
> HI Sergey,
>
> I am not sure why which requires some investigation. I see none of the 
> Synth*UI class has caching, maybe it was done intentionally. Since it 
> is quite close to RDP1 and there are other issues need to be fixed, I 
> would like to defer this investigation after RDP2 (when the jdk13 ws 
> would be opened). Meanwhile, I would like this fix to go in as it 
> breaks SwingSet2 demo.
>
Also, it seems other L&Fs does not use SynthContext kind of context 
creation for each Synth*UI which is check in 
getPreferredSize,MinimumSize and it creates a new context every time 
getCOntext() is called, so probably we need to cache the context also 
instead of recreating it everytime. Needs further investigation.

> Regards
> Prasanta
> On 22-Nov-18 1:47 AM, Sergey Bylokhov wrote:
>> Hi, Prasanta
>>
>> On 21/11/2018 02:27, Prasanta Sadhukhan wrote:
>>> where caching of SynthButtonUI is done. It seems caching of button 
>>> ui object causes SynthButtoUI.getMinimumSize() to be called again
>>> whereas without caching, the SynthButtonUI class is not invoked when 
>>> Container.layout() lay outs the components when we switch L&F from 
>>> nimbus to something else.
>>
>> Why caching works for other looks&feels?
>>
>>
>>
>