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

Source/platform/graphics/UnacceleratedImageBufferSurface.cpp

 /*
  * Copyright (c) 2013, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 22 matching lines @@
 
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "wtf/PassOwnPtr.h"
 
 namespace WebCore {
 
 UnacceleratedImageBufferSurface::UnacceleratedImageBufferSurface(const IntSize& size, OpacityMode opacityMode)
     : ImageBufferSurface(size, opacityMode)
 {
     SkBitmap bitmap;
-    bitmap.setConfig(SkBitmap::kARGB_8888_Config, size.width(), size.height());
-    bitmap.allocPixels();
+    bitmap.allocN32Pixels(size.width(), size.height());

[reed1, 2014/03/17 13:25:18]

allocN32Pixels returns false on error, so you can use that to know if you should
return early.

if (!bitmap.allocN32Pixels(...)) return;

Alternatively, you perhaps should call SkCanvas::NewRasterN32(...) here, which
returns NULL on failure, skipping the need to create a temp bitmap which you
(the caller) don't need.

[jbroman, 2014/03/17 16:46:38]

Done. Didn't realize SkCanvas::NewRasterN32 existed. Refactored in a way that
should not affect the overall behaviour of this constructor.

     if (bitmap.isNull())
         return;
     m_canvas = adoptPtr(new SkCanvas(bitmap));
     clear();
 }
 
 } // namespace WebCore