Revert 91504 - Stop using deprecated factory API for SkDevice

skia/ext/vector_platform_device_cairo_linux.cc

 // Copyright (c) 2011 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_cairo_linux.h"
 
 #include <cairo.h>
 #include <cairo-ft.h>
 
 #include <ft2build.h>
@@ skipping 50 matching lines @@
 
 // Verify cairo surface after creation/modification.
 bool IsContextValid(cairo_t* context) {
   return cairo_status(context) == CAIRO_STATUS_SUCCESS;
 }
 
 }  // namespace
 
 namespace skia {
 
+SkDevice* VectorPlatformDeviceCairoFactory::newDevice(SkCanvas* ignored,
+                                                      SkBitmap::Config config,
+                                                      int width, int height,
+                                                      bool isOpaque,
+                                                      bool isForLayer) {
+  SkASSERT(config == SkBitmap::kARGB_8888_Config);
+  return CreateDevice(NULL, width, height, isOpaque);
+}
+
 // static
-PlatformDevice* VectorPlatformDeviceCairo::CreateDevice(cairo_t* context,
-                                                        int width, int height,
-                                                        bool isOpaque) {
+PlatformDevice* VectorPlatformDeviceCairoFactory::CreateDevice(cairo_t* context,
+                                                               int width,
+                                                               int height,
+                                                               bool isOpaque) {
   // TODO(myhuang): Here we might also have similar issues as those on Windows
   // (vector_canvas_win.cc, http://crbug.com/18382 & http://crbug.com/18383).
   // Please note that is_opaque is true when we use this class for printing.
   // Fallback to bitmap when context is NULL.
   if (!isOpaque || NULL == context) {
     return BitmapPlatformDevice::Create(width, height, isOpaque);
   }
 
+  PlatformDevice* device =
+    VectorPlatformDeviceCairo::create(context, width, height);
+  return device;
+}
+
+VectorPlatformDeviceCairo* VectorPlatformDeviceCairo::create(
+    PlatformSurface context,
+    int width,
+    int height) {
   SkASSERT(cairo_status(context) == CAIRO_STATUS_SUCCESS);
   SkASSERT(width > 0);
   SkASSERT(height > 0);
 
+  // TODO(myhuang): Can we get rid of the bitmap? In this vectorial device,
+  // the content of this bitmap might be meaningless. However, it does occupy
+  // lots of memory space.
   SkBitmap bitmap;
   bitmap.setConfig(SkBitmap::kARGB_8888_Config, width, height);
 
   return new VectorPlatformDeviceCairo(context, bitmap);
 }
 
 VectorPlatformDeviceCairo::VectorPlatformDeviceCairo(PlatformSurface context,
                                                      const SkBitmap& bitmap)
     : PlatformDevice(bitmap),
       context_(context) {
   SkASSERT(bitmap.getConfig() == SkBitmap::kARGB_8888_Config);
 
   // Increase the reference count to keep the context alive.
   cairo_reference(context_);
 
   transform_.reset();
 }
 
 VectorPlatformDeviceCairo::~VectorPlatformDeviceCairo() {
   // Un-ref |context_| since we referenced it in the constructor.
   cairo_destroy(context_);
 }
 
+SkDeviceFactory* VectorPlatformDeviceCairo::onNewDeviceFactory() {
+  return SkNEW(VectorPlatformDeviceCairoFactory);
+}
+
 PlatformDevice::PlatformSurface
 VectorPlatformDeviceCairo::BeginPlatformPaint() {
   return context_;
 }
 
-SkDevice* VectorPlatformDeviceCairo::onCreateCompatibleDevice(
-                                                        SkBitmap::Config config,
-                                                        int width, int height,
-                                                         bool isOpaque, Usage) {
-  SkASSERT(config == SkBitmap::kARGB_8888_Config);
-  return CreateDevice(NULL, width, height, isOpaque);
-}
-
 uint32_t VectorPlatformDeviceCairo::getDeviceCapabilities() {
   return SkDevice::getDeviceCapabilities() | kVector_Capability;
 }
 
 void VectorPlatformDeviceCairo::drawBitmap(const SkDraw& draw,
                                            const SkBitmap& bitmap,
                                            const SkIRect* srcRectOrNull,
                                            const SkMatrix& matrix,
                                            const SkPaint& paint) {
   SkASSERT(bitmap.getConfig() == SkBitmap::kARGB_8888_Config);
@@ skipping 547 matching lines @@
     DCHECK(it->second.font_stream);
 
     cairo_font_face_destroy(it->second.cairo_face);
     // |it->second.ft_face| is handled by Cairo.
     it->second.font_stream->unref();
   }
   g_font_cache->clear();
 }
 
 }  // namespace skia