commit 1ea042cff23e0e96cf569eb31e6cc55f16ad80b1
author Lei Zhang <thestig@chromium.org> Mon May 15 18:34:36 2023 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon May 15 18:34:36 2023 +0000
tree df942fe3a26bc51e400f56601bbe27560179fef0
parent 4ff4ca9ab8092b65ad6c2065402c15aad4d22a87

Remove CollectionSize() use in CPDF_StructTree::LoadPageTree()

Just use a size_t instead of converting to uint32_t via
CollectionSize().

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

core/fpdfdoc/cpdf_structtree.cpp

diff --git a/core/fpdfdoc/cpdf_structtree.cpp b/core/fpdfdoc/cpdf_structtree.cpp
index 67ac948..a8eb0a7 100644
--- a/core/fpdfdoc/cpdf_structtree.cpp
+++ b/core/fpdfdoc/cpdf_structtree.cpp
@@ -15,7 +15,6 @@
 #include "core/fpdfapi/parser/cpdf_reference.h"
 #include "core/fpdfdoc/cpdf_numbertree.h"
 #include "core/fpdfdoc/cpdf_structelement.h"
-#include "core/fxcrt/stl_util.h"
 
 namespace {
 
@@ -63,16 +62,16 @@
   if (!pKids)
     return;
 
-  uint32_t dwKids = 0;
+  size_t kids_count;
   if (pKids->IsDictionary())
-    dwKids = 1;
+    kids_count = 1;
   else if (const CPDF_Array* pArray = pKids->AsArray())
-    dwKids = fxcrt::CollectionSize<uint32_t>(*pArray);
+    kids_count = pArray->size();
   else
     return;
 
   m_Kids.clear();
-  m_Kids.resize(dwKids);
+  m_Kids.resize(kids_count);
 
   RetainPtr<const CPDF_Dictionary> pParentTree =
       m_pTreeRoot->GetDictFor("ParentTree");