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) {
+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);
+  return VectorPlatformDeviceCairoFactory::CreateDevice(NULL, width, height,

[alokp, 2011/06/28 16:19:46]

Two things:
1. In chromium functions are defined in the same order they are declared in
header file.
2. Why do we need a factory here?

+                                                       isOpaque);
 }
 
 // static
 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.
@@ skipping 34 matching lines @@
   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_;
 }
 
 uint32_t VectorPlatformDeviceCairo::getDeviceCapabilities() {
   return SkDevice::getDeviceCapabilities() | kVector_Capability;
 }
 
 void VectorPlatformDeviceCairo::drawBitmap(const SkDraw& draw,
@@ skipping 552 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