Re: RFR: 8342565: [TestBug] StubTextLayout [v2]

2025-03-06 Thread Andy Goryachev
On Tue, 4 Feb 2025 08:53:31 GMT, Marius Hanl wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 19 additional >> commits s

Re: RFR: 8342565: [TestBug] StubTextLayout [v2]

2025-02-04 Thread Andy Goryachev
On Tue, 4 Feb 2025 08:50:43 GMT, Marius Hanl wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 19 additional >> commits s

Re: RFR: 8342565: [TestBug] StubTextLayout [v2]

2025-02-04 Thread Andy Goryachev
On Tue, 4 Feb 2025 08:53:31 GMT, Marius Hanl wrote: > `S`, `M` or `L` as variable names (minor) the meaning should be obvious from the context (small, medium, large) and I don't want code like this to take two pages: testScrollTo(360, 1, new Integer[] { S, S, S, S, S, S, S, S, S, S, S, S, S, S

Re: RFR: 8342565: [TestBug] StubTextLayout [v2]

2025-02-04 Thread Marius Hanl
On Wed, 29 Jan 2025 19:01:40 GMT, Andy Goryachev wrote: >> Changed the StubTextLayout to use product PrismTextLayout with much >> simplified glyph layout (via stubbed fonts). The new layout assumes all the >> glyphs are squares of font size, while the bold type face produces wider >> glyphs (

Re: RFR: 8342565: [TestBug] StubTextLayout [v2]

2025-02-04 Thread Marius Hanl
On Wed, 29 Jan 2025 19:01:40 GMT, Andy Goryachev wrote: >> Changed the StubTextLayout to use product PrismTextLayout with much >> simplified glyph layout (via stubbed fonts). The new layout assumes all the >> glyphs are squares of font size, while the bold type face produces wider >> glyphs (

Re: RFR: 8342565: [TestBug] StubTextLayout [v2]

2025-01-29 Thread Andy Goryachev
> Changed the StubTextLayout to use product PrismTextLayout with much > simplified glyph layout (via stubbed fonts). The new layout assumes all the > glyphs are squares of font size, while the bold type face produces wider > glyphs (by 1 pixel). The default font size has changed from 10 to 12