tree 8c2378717e5864c8d87207ae85ba99c50a373944
parent ca8fa1f267ac2dc7f0c4dd11135d1b3928672e6c
author Tom Sepez <tsepez@chromium.org> 1579893670 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1579893670 +0000

Avoid redundant EnumFontList() invocations.

Instead, use the (idempotent) CFX_FontMapper::LoadInstalledFonts()
which makes this exact call but at most once per font mapper. Continue
to call this in CreateFontStream() even though it appears redundant
because I've not run down all possible call paths.

For the simple_xfa.pdf document, this results in a 4x reduction in
the number of times the font directories are scanned. This is a
contributor to the referenced bug, but other problems still remain.

Bug: pdfium:1460
Change-Id: Ie0329dd329e586db02436b963352a66a6c4e751e
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/65532
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>