pdfium_test: print out some annotations

samples/pdfium_test.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include <list>
 #include <sstream>
 #include <string>
 #include <utility>
 #include <vector>
 
 #if defined PDF_ENABLE_SKIA && !defined _SKIA_SUPPORT_
 #define _SKIA_SUPPORT_
 #endif
 
 #include "public/fpdf_dataavail.h"
+#include "public/fpdf_doc.h"
 #include "public/fpdf_edit.h"
 #include "public/fpdf_ext.h"
 #include "public/fpdf_formfill.h"
 #include "public/fpdf_text.h"
 #include "public/fpdfview.h"
 #include "samples/image_diff_png.h"
 #include "testing/test_support.h"
 
 #ifdef PDF_ENABLE_V8
 #include "v8/include/libplatform/libplatform.h"
@@ skipping 17 matching lines @@
   OUTPUT_BMP,
   OUTPUT_EMF,
 #endif
 #ifdef PDF_ENABLE_SKIA
   OUTPUT_SKP,
 #endif
 };
 
 struct Options {
   Options()
-      : show_config(false), send_events(false), output_format(OUTPUT_NONE) {}
+      : show_config(false),
+        send_events(false),
+        show_annotations(false),
+        output_format(OUTPUT_NONE) {}
 
   bool show_config;
   bool send_events;
+  bool show_annotations;
   OutputFormat output_format;
   std::string scale_factor_as_string;
   std::string exe_path;
   std::string bin_directory;
   std::string font_directory;
 };
 
 static bool CheckDimensions(int stride, int width, int height) {
   if (stride < 0 || width < 0 || height < 0)
     return false;
@@ skipping 263 matching lines @@
     return false;
   }
   options->exe_path = args[0];
   size_t cur_idx = 1;
   for (; cur_idx < args.size(); ++cur_idx) {
     const std::string& cur_arg = args[cur_idx];
     if (cur_arg == "--show-config") {
       options->show_config = true;
     } else if (cur_arg == "--send-events") {
       options->send_events = true;
+    } else if (cur_arg == "--show-annotations") {
+      options->show_annotations = true;
     } else if (cur_arg == "--ppm") {
       if (options->output_format != OUTPUT_NONE) {
         fprintf(stderr, "Duplicate or conflicting --ppm argument\n");
         return false;
       }
       options->output_format = OUTPUT_PPM;
     } else if (cur_arg == "--png") {
       if (options->output_format != OUTPUT_NONE) {
         fprintf(stderr, "Duplicate or conflicting --png argument\n");
         return false;
@@ skipping 121 matching lines @@
         FORM_OnMouseMove(form, page, 0, x, y);
       } else {
         fprintf(stderr, "mousemove: bad args\n");
       }
     } else {
       fprintf(stderr, "Unrecognized event: %s\n", tokens[0].c_str());
     }
   }
 }
 
+void ShowAnnotations(const FPDF_DOCUMENT& doc, const int page_index) {
+  FPDF_PAGE page = FPDF_LoadPage(doc, page_index);

[Lei Zhang, 2016/05/23 16:47:33]

Why not put the ShowAnnotations() call inside RenderPage() after it has already
loaded the page and triggered the onload form actions?

+  if (!page) {
+    fprintf(stderr, "bad page %d\n", page_index);
+    return;
+  }
+  printf("page %d [%g,%g]:\n", page_index, FPDF_GetPageWidth(page),
+         FPDF_GetPageHeight(page));
+  int startPos = 0;
+  FPDF_LINK linkAnnot;
+  while (FPDFLink_Enumerate(page, &startPos, &linkAnnot)) {
+    FS_RECTF rect = {0.0f, 0.0f, 0.0f, 0.0f};
+    (void)FPDFLink_GetAnnotRect(linkAnnot, &rect);
+    FPDF_ACTION action = FPDFLink_GetAction(linkAnnot);
+    if (action && PDFACTION_URI == FPDFAction_GetType(action)) {
+      auto len = FPDFAction_GetURIPath(doc, action, nullptr, 0);

[Lei Zhang, 2016/05/23 16:47:33]

write out unsigned int instead of using auto?

+      std::vector<char> buffer(len);
+      (void)FPDFAction_GetURIPath(doc, action, &buffer[0], buffer.size());
+      printf("  <%g,%g,%g,%g>  uri: \"%.*s\"\n", rect.left, rect.top,

[Lei Zhang, 2016/05/23 16:47:32]

I see 5 conversion specifiers and 6 variables.

+             rect.right, rect.bottom, (int)buffer.size(), &buffer[0]);

[Lei Zhang, 2016/05/23 16:47:33]

Borrow PRIuS from base/format_macros.h for size_t if needed?

+    } else if (FPDF_DEST dest = FPDFLink_GetDest(doc, linkAnnot)) {
+      unsigned long dest_page = FPDFDest_GetPageIndex(doc, dest);
+      FS_FLOAT x = 0.0f;
+      FS_FLOAT y = 0.0f;
+      (void)FPDFDest_GetLocationInPage(dest, &x, &y);
+      printf("  <%g,%g,%g,%g>  page: %lu; point: (%g,%g)\n", rect.left,
+             rect.top, rect.right, rect.bottom, dest_page, x, y);
+    }
+  }
+  FPDF_ClosePage(page);
+  printf("\n");
+}
+
 bool RenderPage(const std::string& name,
                 const FPDF_DOCUMENT& doc,
                 const FPDF_FORMHANDLE& form,
                 const int page_index,
                 const Options& options,
                 const std::string& events) {
   FPDF_PAGE page = FPDF_LoadPage(doc, page_index);
   if (!page) {
     return false;
   }
@@ skipping 193 matching lines @@
         fprintf(stderr, "Unknown error in checking if page %d is available.\n",
                 i);
         return;
       }
     }
     if (RenderPage(name, doc, form, i, options, events)) {
       ++rendered_pages;
     } else {
       ++bad_pages;
     }
+    if (options.show_annotations) {
+      ShowAnnotations(doc, i);
+    }
   }
 
   FORM_DoDocumentAAction(form, FPDFDOC_AACTION_WC);
 
 #ifdef PDF_ENABLE_XFA
   // Note: The shut down order here is the reverse of the non-XFA branch order.
   // Need to work out if this is required, and if it is, the lifetimes of
   // objects owned by |doc| that |form| reference.
   FPDF_CloseDocument(doc);
   FPDFDOC_ExitFormFillEnvironment(form);
@@ skipping 25 matching lines @@
 #ifdef PDF_ENABLE_XFA
   config.append(maybe_comma);
   config.append("XFA");
   maybe_comma = ",";
 #endif  // PDF_ENABLE_XFA
   printf("%s\n", config.c_str());
 }
 
 static const char usage_string[] =
     "Usage: pdfium_test [OPTION] [FILE]...\n"
-    "  --show-config     - print build options and exit\n"
-    "  --send-events     - send input described by .evt file\n"
-    "  --bin-dir=<path>  - override path to v8 external data\n"
-    "  --font-dir=<path> - override path to external fonts\n"
-    "  --scale=<number>  - scale output size by number (e.g. 0.5)\n"
+    "  --show-config      - print build options and exit\n"
+    "  --send-events      - send input described by .evt file\n"
+    "  --bin-dir=<path>   - override path to v8 external data\n"
+    "  --font-dir=<path>  - override path to external fonts\n"
+    "  --show-annotations - print information about URL and internal\n"
+    "                       link annotations in the document.\n"
+    "  --scale=<number>   - scale output size by number (e.g. 0.5)\n"
 #ifdef _WIN32
     "  --bmp - write page images <pdf-name>.<page-number>.bmp\n"
     "  --emf - write page meta files <pdf-name>.<page-number>.emf\n"
 #endif  // _WIN32
     "  --png - write page images <pdf-name>.<page-number>.png\n"
 #ifdef PDF_ENABLE_SKIA
     "  --skp - write page images <pdf-name>.<page-number>.skp\n"
 #endif
     "";
 
@@ skipping 77 matching lines @@
   }
 
   FPDF_DestroyLibrary();
 #ifdef PDF_ENABLE_V8
   v8::V8::ShutdownPlatform();
   delete platform;
 #endif  // PDF_ENABLE_V8
 
   return 0;
 }