Add libpoppler PDF rasterizer

src/utils/SkPDFRasterizer.cpp

+
+/*
+ * Copyright 2013 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#ifdef SK_BUILD_FOR_WIN32
+        #pragma warning(push)
+        #pragma warning(disable : 4530)
+#endif
+
+#include <poppler-document.h>
+#include <poppler-image.h>
+#include <poppler-page.h>
+#include <poppler-page-renderer.h>
+
+#include "SkPDFRasterizer.h"
+#include "SkColorPriv.h"
+
+bool SkPopplerRasterizePDF(SkStream* pdf, SkBitmap* output) {

[edisonn, 2013/08/08 20:14:17]

instead of a function pointer, what about an abstract class?

Pd viewer can render on gpu, or raster, and it would need a config too.

[ducky, 2013/08/08 21:35:03]

This was originally designed as a drop-in replacement for the Mac renderer, and
keeps the same input/output types.

I still think that this is a fairly clean and minimal interface, though.
Multiple configs can still be handled in gmmain using a function that
essentially does this, but with the native viewer code (so you would have
adaptors like native_render_gpu, native_render_raster).

I'm less sure about using an abstract class, since it would require some
unintuitive methods (like AbstractPDFRenderer::next(), or return an array of
bitmaps with descriptions), nor would you be able to select particular configs
to run in the gmtool without significant added complexity.

+  size_t size = pdf->getLength();
+  void* buffer = sk_malloc_throw(size);
+  pdf->read(buffer, size);
+
+  SkAutoTDelete<poppler::document> doc(
+      poppler::document::load_from_raw_data((const char*)buffer, size));
+  if (!doc.get() || doc->is_locked()) {
+    return false;
+  }
+
+  SkAutoTDelete<poppler::page> page(doc->create_page(0));
+  poppler::page_renderer renderer;
+  poppler::image image = renderer.render_page(page.get());
+
+  if (!image.is_valid() || image.format() != poppler::image::format_argb32) {
+    return false;
+  }
+
+  size_t width = image.width(), height = image.height();
+  size_t rowSize = image.bytes_per_row();
+  char *imgData = image.data();
+
+  SkBitmap bitmap;
+  bitmap.setConfig(SkBitmap::kARGB_8888_Config, width, height);
+  bitmap.allocPixels();

[edisonn, 2013/08/08 20:14:17]

check alloc pixels, we will crash here if we fail

[ducky, 2013/08/08 21:35:03]

Done.

+  bitmap.eraseColor(SK_ColorWHITE);
+  SkPMColor* bitmapPixels = (SkPMColor*)bitmap.getPixels();
+
+  // do pixel-by-pixel copy to deal with RGBA ordering conversions
+  for (size_t y = 0; y < height; y++) {
+    char *rowData = imgData;
+    for (size_t x = 0; x < width; x++) {
+      U8CPU r = (unsigned char)rowData[2];
+      U8CPU g = (unsigned char)rowData[1];
+      U8CPU b = (unsigned char)rowData[0];
+
+      SkPMColor out = SkPackARGB32(
+          255, r, g, b);
+
+      *bitmapPixels = out;
+
+      bitmapPixels++;
+      rowData += 4;
+    }
+    imgData += rowSize;
+  }
+
+  output->swap(bitmap);
+
+  return true;
+}