tree 535350d4be37905acca8775b4cbf3e0b4744735b
parent 038d39b314a6c6327959e97b0c7d3d64c1edf183
author Lei Zhang <thestig@chromium.org> 1708629198 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1708629198 +0000

Stop making CPDF_IccProfile an Observable

This effectively reverts https://pdfium-review.googlesource.com/58231
and makes CPDF_DocPageData hold on to CPDF_IccProfile via RetainPtr once
again. As a result, the cached CPDF_IccProfile entries remain alive for
much longer and that can greatly increase the cache hit rate. Thus PDFs
that have frequently used, but expensive to parse ICC entries render
much faster.

At the same time, the CPDF_DocPageData class members are ordered
correctly, such that there are no circular dependencies that can
complicate CPDF_DocPageData destruction.

Bug: chromium:326161385,pdfium:2131
Change-Id: I837251a366d5d17c500afa6f137d77d6d1005f61
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/116891
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Thomas Sepez <tsepez@google.com>