Introduce PlatformBitmap, which is a minimal helper class that wraps an SkBitmap

skia/ext/platform_canvas.h

 // 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.
 
 #ifndef SKIA_EXT_PLATFORM_CANVAS_H_
 #define SKIA_EXT_PLATFORM_CANVAS_H_
 
 // The platform-specific device will include the necessary platform headers
 // to get the surface type.
+#include "base/basictypes.h"
 #include "skia/ext/platform_device.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 
 namespace skia {
 
 // This class is a specialization of the regular SkCanvas that is designed to
 // work with a PlatformDevice to manage platform-specific drawing. It allows
 // using both Skia operations and platform-specific operations.
 class SK_API PlatformCanvas : public SkCanvas {
  public:
@@ skipping 124 matching lines @@
   PlatformSurface GetPlatformSurface() { return platform_surface_; }
  private:
   SkCanvas* canvas_;
   PlatformSurface platform_surface_;
 
   // Disallow copy and assign
   ScopedPlatformPaint(const ScopedPlatformPaint&);
   ScopedPlatformPaint& operator=(const ScopedPlatformPaint&);
 };
 
+class SK_API PlatformBitmap {
+ public:
+  PlatformBitmap();

[brettw, 2012/10/09 21:59:17]

I think you lost the implementation of this function or you forgot to add
platform_canvas.cc to the CL.

+  ~PlatformBitmap();
+
+  // Returns true if the bitmap was able to allocate its surface.
+  bool Allocate(int width, int height, bool is_opaque);
+
+  // Returns the platform surface, or 0 if Allocate() did not return true.
+  PlatformSurface GetSurface() { return surface_; }
+
+  // Return the skia bitmap, which will be empty if Allocate() did not
+  // return true.
+  const SkBitmap& GetBitmap() { return bitmap_; }
+
+ private:
+  SkBitmap bitmap_;
+  PlatformSurface surface_;
+
+  DISALLOW_COPY_AND_ASSIGN(PlatformBitmap);
+};
+
 }  // namespace skia
 
 #endif  // SKIA_EXT_PLATFORM_CANVAS_H_