SkPDF: Change handling of large-numbers in matricies.

src/pdf/SkPDFDevice.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
-#include <tuple>
 #include "SkPDFDevice.h"
 
 #include "SkAnnotation.h"
 #include "SkColor.h"
 #include "SkColorFilter.h"
 #include "SkClipStack.h"
 #include "SkData.h"
 #include "SkDraw.h"
 #include "SkGlyphCache.h"
 #include "SkPaint.h"
@@ skipping 22 matching lines @@
 #define DPI_FOR_RASTER_SCALE_ONE 72
 
 // Utility functions
 
 static bool excessive_translation(const SkMatrix& m) {
     const SkScalar kExcessiveTranslation = 8192.0f;
     return SkScalarAbs(m.getTranslateX()) > kExcessiveTranslation
         || SkScalarAbs(m.getTranslateY()) > kExcessiveTranslation;
 }
 
-static std::tuple<SkMatrix, SkVector> untranslate(const SkDraw& d) {
+static SkMatrix untranslate(const SkMatrix& matrix, SkScalar x, SkScalar y) {
     // https://bug.skia.org/257 If the translation is too large,
     // PDF can't exactly represent the float values as numbers.
-    SkScalar translateX = d.fMatrix->getTranslateX() / d.fMatrix->getScaleX();
-    SkScalar translateY = d.fMatrix->getTranslateY() / d.fMatrix->getScaleY();
-    SkMatrix mat = *d.fMatrix;
-    mat.preTranslate(-translateX, -translateY);
-    SkASSERT(SkScalarAbs(mat.getTranslateX()) <= 1.0);
-    return std::make_tuple(mat, SkVector::Make(translateX, translateY));
+    SkMatrix result(matrix);
+    result.preTranslate(x, y);
+    return result;
 }
 
 // If the paint will definitely draw opaquely, replace kSrc_Mode with
 // kSrcOver_Mode.  http://crbug.com/473572
 static void replace_srcmode_on_opaque_paint(SkPaint* paint) {
     if (kSrcOver_SkXfermodeInterpretation
         == SkInterpretXfermode(*paint, false)) {
         paint->setXfermode(nullptr);
     }
 }
@@ skipping 725 matching lines @@
     if (!contentEntry->fState.fMatrix.invert(&inverse)) {
         return;
     }
     inverse.mapRect(&bbox);
 
     SkPDFUtils::AppendRectangle(bbox, &contentEntry->fContent);
     SkPDFUtils::PaintPath(paint.getStyle(), SkPath::kWinding_FillType,
                           &contentEntry->fContent);
 }
 
-void SkPDFDevice::drawPoints(const SkDraw& d,
+void SkPDFDevice::drawPoints(const SkDraw& srcDraw,
                              SkCanvas::PointMode mode,
                              size_t count,
                              const SkPoint* points,
                              const SkPaint& srcPaint) {
     SkPaint passedPaint = srcPaint;
     replace_srcmode_on_opaque_paint(&passedPaint);
 
     if (count == 0) {
         return;
     }
 
     if (SkAnnotation* annotation = passedPaint.getAnnotation()) {
-        if (handlePointAnnotation(points, count, *d.fMatrix, annotation)) {
+        if (handlePointAnnotation(points, count, *srcDraw.fMatrix, annotation)) {
             return;
         }
     }
+    SkMatrix newMatrix;
+    SkDraw d(srcDraw);
+    SkTArray<SkPoint> pointsCopy;
     if (excessive_translation(*d.fMatrix)) {
-        SkVector translate; SkMatrix translateMatrix;
-        std::tie(translateMatrix, translate) = untranslate(d);
-        SkDraw drawCopy(d);
-        drawCopy.fMatrix = &translateMatrix;
-        SkTArray<SkPoint> pointsCopy(points, SkToInt(count));
-        SkPoint::Offset(&pointsCopy[0], SkToInt(count), translate);
-        this->drawPoints(drawCopy, mode, count, &pointsCopy[0], srcPaint);
-        return;  // NOTE: shader behavior will be off.
+        newMatrix = untranslate(*d.fMatrix, points[0].x(), points[0].y());
+        d.fMatrix = &newMatrix;
+        pointsCopy.reset(points, SkToInt(count));
+        SkPoint::Offset(&pointsCopy[0], SkToInt(count),
+                        -points[0].x(), -points[0].y());
+        points = &pointsCopy[0];
     }
 
     // SkDraw::drawPoints converts to multiple calls to fDevice->drawPath.
     // We only use this when there's a path effect because of the overhead
     // of multiple calls to setUpContentEntry it causes.
     if (passedPaint.getPathEffect()) {
         if (d.fClip->isEmpty()) {
             return;
         }
         SkDraw pointDraw(d);
@@ skipping 101 matching lines @@
 
 static SkPDFDict* create_link_named_dest(const SkData* nameData,
                                          const SkRect& r) {
     SkAutoTUnref<SkPDFDict> annotation(create_link_annotation(r));
     SkString name(static_cast<const char *>(nameData->data()),
                   nameData->size() - 1);
     annotation->insertName("Dest", name);
     return annotation.detach();
 }
 
-void SkPDFDevice::drawRect(const SkDraw& d,
+void SkPDFDevice::drawRect(const SkDraw& srcDraw,
                            const SkRect& rect,
                            const SkPaint& srcPaint) {
     SkPaint paint = srcPaint;
     replace_srcmode_on_opaque_paint(&paint);
     SkRect r = rect;
     r.sort();
 
+    SkMatrix newMatrix;
+    SkDraw d(srcDraw);
     if (excessive_translation(*d.fMatrix)) {
-        SkVector translate; SkMatrix translateMatrix;
-        std::tie(translateMatrix, translate) = untranslate(d);
-        SkDraw drawCopy(d);
-        drawCopy.fMatrix = &translateMatrix;
-        SkRect rectCopy = rect;
-        rectCopy.offset(translate.x(), translate.y());
-        this->drawRect(drawCopy, rectCopy, srcPaint);
-        return;  // NOTE: shader behavior will be off.
+        newMatrix = untranslate(*d.fMatrix, r.x(), r.y());
+        d.fMatrix = &newMatrix;
+        r.offsetTo(0, 0);
     }
 
     if (paint.getPathEffect()) {
         if (d.fClip->isEmpty()) {
             return;
         }
         SkPath path;
         path.addRect(r);
         drawPath(d, path, paint, nullptr, true);
         return;
@@ skipping 29 matching lines @@
 void SkPDFDevice::drawOval(const SkDraw& draw,
                            const SkRect& oval,
                            const SkPaint& srcPaint) {
     SkPaint paint = srcPaint;
     replace_srcmode_on_opaque_paint(&paint);
     SkPath  path;
     path.addOval(oval);
     this->drawPath(draw, path, paint, nullptr, true);
 }
 
-void SkPDFDevice::drawPath(const SkDraw& d,
+void SkPDFDevice::drawPath(const SkDraw& srcDraw,
                            const SkPath& origPath,
                            const SkPaint& srcPaint,
                            const SkMatrix* prePathMatrix,
                            bool pathIsMutable) {
+    SkMatrix newMatrix;
+    SkDraw d(srcDraw);
+    SkPath modifiedPath;
+    SkPath* pathPtr = const_cast<SkPath*>(&origPath);
     if (excessive_translation(*d.fMatrix)) {
-        SkVector translate; SkMatrix translateMatrix;
-        std::tie(translateMatrix, translate) = untranslate(d);
-        SkDraw drawCopy(d);
-        drawCopy.fMatrix = &translateMatrix;
-        SkPath pathCopy(origPath);
-        pathCopy.offset(translate.x(), translate.y());
-        this->drawPath(drawCopy, pathCopy, srcPaint, prePathMatrix, true);
-        return;  // NOTE: shader behavior will be off.
+        SkPoint firstPt;
+        if (origPath.getPoints(&firstPt, 1) > 0) {
+            newMatrix = untranslate(*d.fMatrix, firstPt.x(), firstPt.y());
+            d.fMatrix = &newMatrix;
+            modifiedPath = origPath;
+            modifiedPath.offset(-firstPt.x(), -firstPt.y());
+            pathPtr = &modifiedPath;  // NOTE: shader behavior will be off.
+            pathIsMutable = true;
+        }
     }
 
     SkPaint paint = srcPaint;
     replace_srcmode_on_opaque_paint(&paint);
-    SkPath modifiedPath;
-    SkPath* pathPtr = const_cast<SkPath*>(&origPath);
 
     SkMatrix matrix = *d.fMatrix;
     if (prePathMatrix) {
         if (paint.getPathEffect() || paint.getStyle() != SkPaint::kFill_Style) {
-            if (!pathIsMutable) {
+            if (pathIsMutable) {
+                pathPtr->transform(*prePathMatrix);
+            } else {
+                pathPtr->transform(*prePathMatrix, &modifiedPath);
                 pathPtr = &modifiedPath;
                 pathIsMutable = true;
             }
-            origPath.transform(*prePathMatrix, pathPtr);
         } else {
             matrix.preConcat(*prePathMatrix);
         }
     }
 
     if (paint.getPathEffect()) {
         if (d.fClip->isEmpty()) {
             return;
         }
-        if (!pathIsMutable) {
+        bool fill;
+        if (pathIsMutable) {
+            fill = paint.getFillPath(*pathPtr, pathPtr);
+        } else {
+            fill = paint.getFillPath(*pathPtr, &modifiedPath);
             pathPtr = &modifiedPath;
             pathIsMutable = true;
         }
-        bool fill = paint.getFillPath(origPath, pathPtr);
 
         SkPaint noEffectPaint(paint);
         noEffectPaint.setPathEffect(nullptr);
         if (fill) {
             noEffectPaint.setStyle(SkPaint::kFill_Style);
         } else {
             noEffectPaint.setStyle(SkPaint::kStroke_Style);
             noEffectPaint.setStrokeWidth(0);
         }
         drawPath(d, *pathPtr, noEffectPaint, nullptr, true);
@@ skipping 243 matching lines @@
         case SkPaint::kUTF32_TextEncoding:
             transparent.setTextEncoding(srcPaint.getTextEncoding());
             device->drawText(d, text, len, x, y, transparent);
             break;
         default:
             SkFAIL("unknown text encoding");
     }
 }
 
 
-void SkPDFDevice::drawText(const SkDraw& d, const void* text, size_t len,
+void SkPDFDevice::drawText(const SkDraw& srcDraw, const void* text, size_t len,
                            SkScalar x, SkScalar y, const SkPaint& srcPaint) {
+    SkMatrix newMatrix;
+    SkDraw d(srcDraw);
     if (excessive_translation(*d.fMatrix)) {
-        SkVector translate; SkMatrix translateMatrix;
-        std::tie(translateMatrix, translate) = untranslate(d);
-        SkDraw drawCopy(d);
-        drawCopy.fMatrix = &translateMatrix;
-        this->drawText(drawCopy, text, len, x + translate.x(),
-                       y + translate.y(), srcPaint);
-        return;  // NOTE: shader behavior will be off.
+        newMatrix = untranslate(*d.fMatrix, x, y);
+        d.fMatrix = &newMatrix;
+        x = 0;
+        y = 0;
     }
 
     if (!SkPDFFont::CanEmbedTypeface(srcPaint.getTypeface(), fCanon)) {
         // https://bug.skia.org/3866
         SkPath path;
         srcPaint.getTextPath(text, len, x, y, &path);
         this->drawPath(d, path, srcPaint, &SkMatrix::I(), true);
         // Draw text transparently to make it copyable/searchable/accessable.
         draw_transparent_text(this, d, text, len, x, y, srcPaint);
         return;
@@ skipping 40 matching lines @@
         SkString encodedString =
                 format_wide_string(glyphIDsCopy.begin() + consumedGlyphCount,
                                    availableGlyphs, font->multiByteGlyphs());
         content.entry()->fContent.writeText(encodedString.c_str());
         consumedGlyphCount += availableGlyphs;
         content.entry()->fContent.writeText(" Tj\n");
     }
     content.entry()->fContent.writeText("ET\n");
 }
 
-void SkPDFDevice::drawPosText(const SkDraw& d, const void* text, size_t len,
+void SkPDFDevice::drawPosText(const SkDraw& srcDraw, const void* text, size_t len,
                               const SkScalar pos[], int scalarsPerPos,
-                              const SkPoint& offset, const SkPaint& srcPaint) {
+                              const SkPoint& srcOffset, const SkPaint& srcPaint) {
+    if (len == 0) {
+        return;
+    }
+    SkMatrix newMatrix;
+    SkDraw d(srcDraw);
+    SkPoint offset(srcOffset);
+    SkAutoTMalloc<SkScalar> scalarsBuffer;
     if (excessive_translation(*d.fMatrix)) {
-        SkVector translate; SkMatrix translateMatrix;
-        std::tie(translateMatrix, translate) = untranslate(d);
-        SkDraw drawCopy(d);
-        drawCopy.fMatrix = &translateMatrix;
-        SkPoint offsetCopy = offset;
-        SkPoint::Offset(&offsetCopy, 1, translate.x(), translate.y());
-        this->drawPosText(drawCopy, text, len, pos, scalarsPerPos, offsetCopy,
-                          srcPaint);
-        return;  // NOTE: shader behavior will be off.
+        SkPoint first;
+        if (scalarsPerPos != 2) {
+            first.set(pos[0], 0);
+        } else {
+            first.set(pos[0], pos[1]);
+        }
+        newMatrix = untranslate(*d.fMatrix,
+                                first.x() + offset.x(),
+                                first.y() + offset.y());
+        d.fMatrix = &newMatrix;
+        offset.set(0, 0);  // offset -= offset;
+        if (first.x() != 0 || first.y() != 0) {
+            int glyphCount = srcPaint.textToGlyphs(text, len, NULL);
+            if (scalarsPerPos != 2) {
+                scalarsBuffer.reset(glyphCount);
+                for (int i = 0; i < glyphCount; ++i) {
+                    scalarsBuffer[i] = pos[i] - first.x();
+                }
+            } else {
+                scalarsBuffer.reset(2 * glyphCount);
+                for (int i = 0; i < glyphCount; ++i) {
+                    scalarsBuffer[2 * i]     = pos[2 * i]     - first.x();
+                    scalarsBuffer[2 * i + 1] = pos[2 * i + 1] - first.y();
+                }
+            }
+            pos = &scalarsBuffer[0];
+        }
     }
 
     if (!SkPDFFont::CanEmbedTypeface(srcPaint.getTypeface(), fCanon)) {
         const SkPoint* positions = reinterpret_cast<const SkPoint*>(pos);
         SkAutoTMalloc<SkPoint> positionsBuffer;
         if (2 != scalarsPerPos) {
             int glyphCount = srcPaint.textToGlyphs(text, len, NULL);
             positionsBuffer.reset(glyphCount);
             for (int  i = 0; i < glyphCount; ++i) {
                 positionsBuffer[i].set(pos[i], 0.0f);
@@ skipping 1019 matching lines @@
         pdfimage.reset(SkPDFCreateBitmapObject(
                                image, fCanon->fPixelSerializer));
         if (!pdfimage) {
             return;
         }
         fCanon->addPDFBitmap(image->uniqueID(), pdfimage);
     }
     SkPDFUtils::DrawFormXObject(this->addXObjectResource(pdfimage.get()),
                                 &content.entry()->fContent);
 }