On 5/24/2013 11:46 AM, dmitry markov wrote:
Hello,Could toy review the fix: bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8014863 webrev: http://cr.openjdk.java.net/~vkarnauk/8014863/webrev.00/The fix removes break points caching from GlyphView. If some text is inserted/removed into/from the document contained several elements, all elements views will re-calculate their break points.
For example the ParagraphView.calculateMinorAxisRequirements method can invoke several methods like getBreakWeight() and calculateMinorAxisRequirements() that both leads to GlyphView.getBreakSpot() method invocation on the same method when the text is not changed. So the breakSpots cache is still useful in this case. May be there are some missed places during text updating where the breakSpots cache should be cleaned.
The getNumberOfTextLines() method in the test uses the JEditorPane on the main thread. There is the invokeOnEDT() method in test Util class that
helps to return a result from the EDT thread. Thanks, Alexandr.
Thanks, Dmitry.
