On Fri, 31 Jan 2025 12:37:04 GMT, Lukasz Kostyra <lkost...@openjdk.org> wrote:
> This is a cleanup follow-up, removing `FontFileWriter.FontTracker` and all
> related uses. `FontTracker` was tracking font size use when SecurityManager
> was present, however since we removed SM, `FontTracker` was no longer
> activated and as such was dead code.
>
> `FontFileWriter.FontTracker` and its use in `FontFileWriter` + related
> methods were removed. This in turn cleaned up `PrismFontFile` and made a
> couple of variables not longer used, including a `bool tracking` argument in
> constructor. These cleanups propagated to `PrismFontFactory`,
> `{CT,DW,FT}Factory` and `{CT,DW,FT}FontFile` classes.
>
> Tests worked the same after this change.
lgtm, with one minor comment - will re-approve if you decide to change.
-------------
Marked as reviewed by angorya (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/1693#pullrequestreview-2587144860
