Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(852)

Unified Diff: gm/gmmain.cpp

Issue 270263009: Allow for no pdf rasterizers in gm. (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/utils/SkPDFRasterizer.h » ('j') | src/utils/SkPDFRasterizer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gm/gmmain.cpp
===================================================================
--- gm/gmmain.cpp (revision 14665)
+++ gm/gmmain.cpp (working copy)
@@ -1330,6 +1330,8 @@
#ifdef SK_BUILD_NATIVE_PDF_RENDERER
{ &SkNativeRasterizePDF, "native", true },
#endif // SK_BUILD_NATIVE_PDF_RENDERER
+ // The following exists so that this array is never zero length.
epoger 2014/05/09 14:35:30 I take it you investigated making the code properl
bungeman-skia 2014/05/09 14:56:15 Well, it's technically against the c++ specificati
epoger 2014/05/09 15:15:36 Wow, "The More You Know". Thanks; I'm a little sh
+ { &SkNoRasterizePDF, "none", false},
};
static const char kDefaultsConfigStr[] = "defaults";
« no previous file with comments | « no previous file | src/utils/SkPDFRasterizer.h » ('j') | src/utils/SkPDFRasterizer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698