tree 17e2be36d5bdde1a848a3be5fe1ffef703da46ef
parent 6f1b15f6fb27b155e4f6cc565f398739198e1e2e
author Lei Zhang <thestig@chromium.org> 1579300502 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1579300502 +0000

Improve FPDFLink_LoadWebLinks() performance.

Internally, FPDFLink_LoadWebLinks() calls CPDF_LinkExtract::ParseLink(),
which in turn calls CPDF_TextPage::GetPageText() repeatedly to get
sub-strings from CPDF_TextPage. GetPageText() has to do many repeated
calculations each time it is called, so this makes ParseLink() really
slow for pages with lots of text.

Avoud this problem by calling CPDF_TextPage::GetAllPageText() once to
extract the text. Then use WideString::Mid() to get the desired
sub-strings.

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