commit 80191e69c97a4198e29f31a62776b3e7e8c39793
author Ben Wagner <bungeman@google.com> Tue Oct 08 15:12:54 2024 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Wed Oct 09 15:06:29 2024 +0000
tree 54048b7d1314f64caa47996a1c0d3dfb4b4a9de5
parent d38148cdf98cc9ba9f4c4447ab378782df0678fa

[pdf] Emit `/Tabs /S` on each page

Set the tab order to "structure order".

Bug: 40864338
Change-Id: Ic0a5fdd5996052668c1026d969749d432f167778
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/908456
Reviewed-by: Michael Ludwig <michaelludwig@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>

src/pdf/SkPDFDocument.cpp

diff --git a/src/pdf/SkPDFDocument.cpp b/src/pdf/SkPDFDocument.cpp
index e73742b..48dd664 100644
--- a/src/pdf/SkPDFDocument.cpp
+++ b/src/pdf/SkPDFDocument.cpp
@@ -393,6 +393,10 @@
     // The StructParents unique identifier for each page is just its
     // 0-based page index.
     page->insertInt("StructParents", SkToInt(this->currentPageIndex()));
+
+    // Tabs is PDF 1.5, but setting it checks an accessibility box.
+    page->insertName("Tabs", "S");
+
     fPages.emplace_back(std::move(page));
 }