tree 00965b7833279fbd983aadaf4d06d8be8ad96f99
parent df2f16a642614ed3a84bf3f26999bee0c2816900
author Tom Sepez <tsepez@chromium.org> 1627428926 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1627428926 +0000

Avoid using FX_SAFE_* types in other header files.

By necessity, this drags in all of the safe_numerics arithmetic
headers, which are quite large. In one place, we use Optional<>
(since optional.h is much much smaller) to return a non-result,
converting back to the safe cases only in .cpp files.

-- rename member to ensure all cases are caught.
-- use const Optional<> locals where possible to prevent additional
   arithmetic against the results which may be unsafe.
-- add missing include in fuzzer.

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