From 72919470a993ea68e427e80a619235f136aae668 Mon Sep 17 00:00:00 2001 From: sbarman Date: Thu, 16 Jun 2011 12:03:50 -0700 Subject: [PATCH] Added comments --- pdf.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/pdf.js b/pdf.js index 17537d233..904ff3906 100644 --- a/pdf.js +++ b/pdf.js @@ -2134,6 +2134,10 @@ var CanvasGraphics = (function() { fn = new PDFFunction(this.xref, fnObj); var gradient = this.ctx.createLinearGradient(x0, y0, x1, y1); + + // 10 samples seems good enough for now, but probably won't work + // if there are sharp color changes. Ideally, we could see the + // current image size and base the # samples on that. var step = (t1 - t0) / 10; for (var i = t0; i <= t1; i += step) { @@ -2146,6 +2150,8 @@ var CanvasGraphics = (function() { // HACK to draw the gradient onto an infinite rectangle. // PDF gradients are drawn across the entire image while // Canvas only allows gradients to be drawn in a rectangle + // Also, larger numbers seem to cause overflow which causes + // nothing to be drawn. this.ctx.fillRect(-1e10, -1e10, 2e10, 2e10); },