Push the initial transform down into SkPDFDevice. (Chrome side).

skia/ext/vector_platform_device_skia.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "skia/ext/vector_platform_device_skia.h"
 
 #include "skia/ext/bitmap_platform_device.h"
 #include "third_party/skia/include/core/SkClipStack.h"
 #include "third_party/skia/include/core/SkDraw.h"
 #include "third_party/skia/include/core/SkRect.h"
 #include "third_party/skia/include/core/SkRegion.h"
 #include "third_party/skia/include/core/SkScalar.h"
 
 namespace skia {
 
 SkDevice* VectorPlatformDeviceSkiaFactory::newDevice(SkCanvas* noUsed,
                                                      SkBitmap::Config config,
                                                      int width, int height,
                                                      bool isOpaque,
                                                      bool isForLayer) {
   SkASSERT(config == SkBitmap::kARGB_8888_Config);
-  SkPDFDevice::OriginTransform flip = SkPDFDevice::kFlip_OriginTransform;
-  if (isForLayer)
-    flip = SkPDFDevice::kNoFlip_OriginTransform;
-  return new VectorPlatformDeviceSkia(width, height, flip);
+  SkMatrix initialTransform;
+  initialTransform.reset();
+  if (isForLayer) {
+    initialTransform.setTranslate(0, height);
+    initialTransform.preScale(1, -1);
+  }
+  return new VectorPlatformDeviceSkia(width, height, initialTransform);
 }
 
 static inline SkBitmap makeABitmap(int width, int height) {
   SkBitmap bitmap;
   bitmap.setConfig(SkBitmap::kNo_Config, width, height);
   return bitmap;
 }
 
 VectorPlatformDeviceSkia::VectorPlatformDeviceSkia(
-    int width, int height, SkPDFDevice::OriginTransform flip)
+    int width, int height, const SkMatrix& initialTransform)
     : PlatformDevice(makeABitmap(width, height)),
-      pdf_device_(new SkPDFDevice(width, height, flip)) {
+      pdf_device_(new SkPDFDevice(width, height, initialTransform)) {
   pdf_device_->unref();  // SkRefPtr and new both took a reference.
-  base_transform_.reset();
 }
 
 VectorPlatformDeviceSkia::~VectorPlatformDeviceSkia() {
 }
 
 bool VectorPlatformDeviceSkia::IsVectorial() {
   return true;
 }
 
 bool VectorPlatformDeviceSkia::IsNativeFontRenderingAllowed() {
@@ skipping 22 matching lines @@
   SkCanvas canvas(raster_surface_.get());
   SkPaint black;
   black.setColor(SK_ColorBLACK);
   canvas.drawPaint(black);
   return raster_surface_->BeginPlatformPaint();
 }
 
 void VectorPlatformDeviceSkia::EndPlatformPaint() {
   DCHECK(raster_surface_ != NULL);
   SkPaint paint;
-  pdf_device_->drawSprite(SkDraw(),
-                          raster_surface_->accessBitmap(false),
-                          base_transform_.getTranslateX(),
-                          base_transform_.getTranslateY(),
-                          paint);
+  pdf_device_->drawSprite(SkDraw(), raster_surface_->accessBitmap(false),
+                          0, 0, paint);
   raster_surface_ = NULL;
 }
 
 SkDeviceFactory* VectorPlatformDeviceSkia::getDeviceFactory() {
   return SkNEW(VectorPlatformDeviceSkiaFactory);
 }
 
 uint32_t VectorPlatformDeviceSkia::getDeviceCapabilities() {
   return kVector_Capability;
 }
 
 int VectorPlatformDeviceSkia::width() const {
   return pdf_device_->width();
 }
 
 int VectorPlatformDeviceSkia::height() const {
   return pdf_device_->height();
 }
 
 void VectorPlatformDeviceSkia::setMatrixClip(const SkMatrix& matrix,
                                              const SkRegion& region,
                                              const SkClipStack& stack) {
-  SkMatrix transform = base_transform_;
-  transform.preConcat(matrix);
-
-  DCHECK(SkMatrix::kTranslate_Mask == base_transform_.getType() ||
-      SkMatrix::kIdentity_Mask == base_transform_.getType());
-  SkRegion clip = region;
-  clip.translate(base_transform_.getTranslateX(),
-                 base_transform_.getTranslateY());
-
-  pdf_device_->setMatrixClip(transform, clip, stack);
+  pdf_device_->setMatrixClip(matrix, region, stack);
 }
 
 bool VectorPlatformDeviceSkia::readPixels(const SkIRect& srcRect,
                                           SkBitmap* bitmap) {
   return false;
 }
 
 void VectorPlatformDeviceSkia::drawPaint(const SkDraw& draw,
                                          const SkPaint& paint) {
   pdf_device_->drawPaint(draw, paint);
@@ skipping 95 matching lines @@
 
 #if defined(OS_WIN)
 void VectorPlatformDeviceSkia::drawToHDC(HDC dc,
                                          int x,
                                          int y,
                                          const RECT* src_rect) {
   SkASSERT(false);
 }
 #endif
 
-void VectorPlatformDeviceSkia::setInitialTransform(int xOffset, int yOffset,
-                                                   float scale_factor) {
-  // TODO(vandebo) Supporting a scale factor is some work because we have to
-  // transform both matrices and clips that come in, but Region only supports
-  // translation.  Instead, we could change SkPDFDevice to include it in the
-  // initial transform.  Delay that work until we would use it.  Also checked
-  // in setMatrixClip.
-  DCHECK_EQ(1.0f, scale_factor);
-
-  base_transform_.setTranslate(xOffset, yOffset);
-  SkScalar scale = SkFloatToScalar(scale_factor);
-  base_transform_.postScale(scale, scale);
-
-  SkMatrix matrix;
-  matrix.reset();
-  SkRegion region;
-  SkClipStack stack;
-  setMatrixClip(matrix, region, stack);
-}
-
 }  // namespace skia