Add onDrawBitmapLattice(), avoid unnecessary bitmap->image copy

src/pdf/SkPDFCanvas.cpp

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkLatticeIter.h"
 #include "SkPDFCanvas.h"
 #include "SkPDFDevice.h"
@@ skipping 77 matching lines @@
 void SkPDFCanvas::onDrawImageLattice(const SkImage* image,
                                      const Lattice& lattice,
                                      const SkRect& dst,
                                      const SkPaint* paint) {
     SkLatticeIter iter(image->width(), image->height(), lattice, dst);
     SkRect srcR, dstR;
     while (iter.next(&srcR, &dstR)) {
         this->drawImageRect(image, srcR, dstR, paint);
     }
 }
+
+void SkPDFCanvas::onDrawBitmapLattice(const SkBitmap& bitmap,

[reed1, 2016/08/16 11:28:47]

General question for PDF canvas (not strictly related to this CL)

- can it fold bitmaps->images at this level, to reduce the number of internal
methods? (as pictures do)?

[hal.canary, 2016/08/16 14:57:33]

No!

You need to keep the bitmaps until we get to the de-duplication stage.  As long
as out clients are using bitmaps, we need to keep de-duping them.

+                                      const Lattice& lattice,
+                                      const SkRect& dst,
+                                      const SkPaint* paint) {
+    SkLatticeIter iter(bitmap.width(), bitmap.height(), lattice, dst);
+    SkRect srcR, dstR;
+    while (iter.next(&srcR, &dstR)) {
+        this->drawBitmapRect(bitmap, srcR, dstR, paint);
+    }
+}
+