SkPDF: roll back 53b1c09 and 05b48e2

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 "SkPDFDevice.h"
 
 #include "SkAnnotation.h"
@@ skipping 23 matching lines @@
 #include "SkSurface.h"
 #include "SkTextFormatParams.h"
 #include "SkTemplates.h"
 #include "SkTypefacePriv.h"
 #include "SkXfermodeInterpretation.h"
 
 #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 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.
-    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);
     }
 }
 
 static void emit_pdf_color(SkColor color, SkWStream* result) {
@@ skipping 723 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& srcDraw,
+void SkPDFDevice::drawPoints(const SkDraw& d,
                              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, *srcDraw.fMatrix, annotation)) {
+        if (handlePointAnnotation(points, count, *d.fMatrix, annotation)) {
             return;
         }
     }
-    SkMatrix newMatrix;
-    SkDraw d(srcDraw);
-    SkTArray<SkPoint> pointsCopy;
-    if (excessive_translation(*d.fMatrix)) {
-        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);
         pointDraw.fDevice = this;
@@ skipping 100 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& srcDraw,
+void SkPDFDevice::drawRect(const SkDraw& d,
                            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)) {
-        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 27 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& srcDraw,
+void SkPDFDevice::drawPath(const SkDraw& d,
                            const SkPath& origPath,
                            const SkPaint& srcPaint,
                            const SkMatrix* prePathMatrix,
                            bool pathIsMutable) {
-    SkMatrix newMatrix;
-    SkDraw d(srcDraw);
+    SkPaint paint = srcPaint;
+    replace_srcmode_on_opaque_paint(&paint);
     SkPath modifiedPath;
     SkPath* pathPtr = const_cast<SkPath*>(&origPath);
-    if (excessive_translation(*d.fMatrix)) {
-        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);
 
     SkMatrix matrix = *d.fMatrix;
     if (prePathMatrix) {
         if (paint.getPathEffect() || paint.getStyle() != SkPaint::kFill_Style) {
-            if (pathIsMutable) {
-                pathPtr->transform(*prePathMatrix);
-            } else {
-                pathPtr->transform(*prePathMatrix, &modifiedPath);
+            if (!pathIsMutable) {
                 pathPtr = &modifiedPath;
                 pathIsMutable = true;
             }
+            origPath.transform(*prePathMatrix, pathPtr);
         } else {
             matrix.preConcat(*prePathMatrix);
         }
     }
 
     if (paint.getPathEffect()) {
         if (d.fClip->isEmpty()) {
             return;
         }
-        bool fill;
-        if (pathIsMutable) {
-            fill = paint.getFillPath(*pathPtr, pathPtr);
-        } else {
-            fill = paint.getFillPath(*pathPtr, &modifiedPath);
+        if (!pathIsMutable) {
             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& srcDraw, const void* text, size_t len,
+void SkPDFDevice::drawText(const SkDraw& d, const void* text, size_t len,
                            SkScalar x, SkScalar y, const SkPaint& srcPaint) {
-    SkMatrix newMatrix;
-    SkDraw d(srcDraw);
-    if (excessive_translation(*d.fMatrix)) {
-        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;
     }
     SkPaint paint = srcPaint;
@@ skipping 38 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& srcDraw, const void* text, size_t len,
+void SkPDFDevice::drawPosText(const SkDraw& d, const void* text, size_t len,
                               const SkScalar pos[], int scalarsPerPos,
-                              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)) {
-        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];
-        }
-    }
-
+                              const SkPoint& offset, const SkPaint& srcPaint) {
     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);
             }
             positions = &positionsBuffer[0];
@@ skipping 1017 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);
 }