Remove uses of SkBitmap::Config (deprecated) from core/ and platform/.

Source/platform/graphics/UnacceleratedImageBufferSurface.cpp

Index: Source/platform/graphics/UnacceleratedImageBufferSurface.cpp
diff --git a/Source/platform/graphics/UnacceleratedImageBufferSurface.cpp b/Source/platform/graphics/UnacceleratedImageBufferSurface.cpp
index 40812df5e776038dbafc423a578370ea08678312..fe6057d1d2377fb57117dc3cfb81624249c0192c 100644
--- a/Source/platform/graphics/UnacceleratedImageBufferSurface.cpp
+++ b/Source/platform/graphics/UnacceleratedImageBufferSurface.cpp
@@ -38,14 +38,10 @@ namespace WebCore {
 
 UnacceleratedImageBufferSurface::UnacceleratedImageBufferSurface(const IntSize& size, OpacityMode opacityMode)
     : ImageBufferSurface(size, opacityMode)
+    , m_canvas(adoptPtr(SkCanvas::NewRasterN32(size.width(), size.height())))

[Stephen Chennney, 2014/03/18 12:47:57]

I understand that this CL is not changing the behavior, but is this not one of
the cases where we are violating ref counting semantics? If so, I would tend to
fix it now.

[jbroman, 2014/03/18 13:44:52]

Yes, it is; SkCanvas extends SkRefCnt (so it's possible for others to take
references and have the canvas freed from under them). I'm happy to change this
to a RefPtr.

 {
-    SkBitmap bitmap;
-    bitmap.setConfig(SkBitmap::kARGB_8888_Config, size.width(), size.height());
-    bitmap.allocPixels();
-    if (bitmap.isNull())
-        return;
-    m_canvas = adoptPtr(new SkCanvas(bitmap));
-    clear();
+    if (m_canvas)
+        clear();
 }
 
 } // namespace WebCore