commit d5c4e5e7a201c31e2333d7e284503c4d05b3aa31
author Lei Zhang <thestig@chromium.org> Thu Oct 21 21:58:18 2021 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Oct 21 21:58:18 2021 +0000
tree 1cebbe1f2833a7107b7280ad541c8ec7e5b319c1
parent f40ccf6ea90619e4373dea395c1032501a5b4c5a

Remove fxcrt::CollectionSize() usage in CFX_FontMapper::GetFaceSize().

Return size_t instead, and use size_t consistently in related code.

Change-Id: I6c9c2d8ed4f226f2e4ed24c39da6c8c4a9511173
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/86350
Commit-Queue: Hui Yingst <nigi@chromium.org>
Reviewed-by: Hui Yingst <nigi@chromium.org>

core/fxge/cfx_fontmapper.h

diff --git a/core/fxge/cfx_fontmapper.h b/core/fxge/cfx_fontmapper.h
index e40b662..4622145 100644
--- a/core/fxge/cfx_fontmapper.h
+++ b/core/fxge/cfx_fontmapper.h
@@ -69,8 +69,8 @@
                                     CFX_SubstFont* pSubstFont);
 
   bool IsBuiltinFace(const RetainPtr<CFX_Face>& face) const;
-  int GetFaceSize() const;
-  ByteString GetFaceName(int index) const { return m_FaceArray[index].name; }
+  size_t GetFaceSize() const;
+  ByteString GetFaceName(size_t index) const { return m_FaceArray[index].name; }
   bool HasInstalledFont(ByteStringView name) const;
   bool HasLocalizedFont(ByteStringView name) const;
 
@@ -83,7 +83,7 @@
 
 #ifdef PDF_ENABLE_XFA
   std::unique_ptr<uint8_t, FxFreeDeleter> RawBytesForIndex(
-      uint32_t index,
+      size_t index,
       size_t* returned_length);
 #endif  // PDF_ENABLE_XFA