[api-minor] A couple of smaller PDFPageProxy.getStructTree
fixes (PR 13171 follow-up)
#13221
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ensure that
getStructTree
won't break withdisableAutoFetch = true
set (PR 13171 follow-up)Open http://localhost:8888/web/viewer.html?file=/test/pdfs/pdf.pdf#disableStream=true&disableAutoFetch=true and observe the following message in the console (repeated for each page of the document):
[api-minor] Let
PDFPageProxy.getStructTree
returnnull
, rather than an empty structTree, for documents without any accessibility data (PR 13171 follow-up)This is first of all consistent with the existing API, where we return
null
when the data in question doesn't exist. Secondly, it should also be (slightly) more efficient since there's less dummy-data that we need to transfer between threads.Finally, this prevents us from adding an empty/unnecessary span to every single page even in documents without any structure tree data.