Hi Alexander,
Since the component is initialized only and is not added to any window
the EDT is not really necessary.
Anyway I added it and also took into account your other comments here:
http://cr.openjdk.java.net/~ssadetsky/8076164/webrev.01/
--Semyon
On 6/30/2015 4:24 PM, Alexander Scherbatiy wrote:
The fix looks good to me.
Just update the copyright in the test, use Swing component on EDT
and format the code.
Thanks,
Alexandr.
On 6/25/2015 5:27 PM, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
bug: https://bugs.openjdk.java.net/browse/JDK-8076164
webrev: http://cr.openjdk.java.net/~ssadetsky/8076164/webrev.00/
It is a regression from 6973777. In 6973777 GlyphView minimum span
with calculation was changed to use text wrapping into several lines.
That was necessary to fix several JTextPane scenarios. But this
change did not preserve the generic rule: if resizing is disabled in
getResizeWeight() then the preferred size should be used as the
minimum. Since originally the GlyphView was not resizable the fix
broke JTextFiled's I18N view because text wrapping is not applicable
to it by design.
The solution restores the logic for JTextFiled's I18N content view.
--Semyon
