Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(313)

Unified Diff: skia/ext/bitmap_platform_device_skia.h

Issue 2611153002: Delete Mac & Cairo skia::BitmapPlatformDevice impls (Closed)
Patch Set: shared lib deps change Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « skia/ext/bitmap_platform_device_mac.cc ('k') | skia/ext/bitmap_platform_device_skia.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skia/ext/bitmap_platform_device_skia.h
diff --git a/skia/ext/bitmap_platform_device_skia.h b/skia/ext/bitmap_platform_device_skia.h
index 024064f4dac3c1914ae7141f840603ed5f1ae294..42a3be1d3422f194d2beab63e2701914d0dbd020 100644
--- a/skia/ext/bitmap_platform_device_skia.h
+++ b/skia/ext/bitmap_platform_device_skia.h
@@ -21,15 +21,21 @@ namespace skia {
// shared memory between the renderer and the main process at least. In this
// case we'll probably create the buffer from a precreated region of memory.
// -----------------------------------------------------------------------------
-class BitmapPlatformDevice : public SkBitmapDevice, public PlatformDevice {
+class BitmapPlatformDevice final : public SkBitmapDevice,
+ public PlatformDevice {
public:
// Construct a BitmapPlatformDevice. |is_opaque| should be set if the caller
- // knows the bitmap will be completely opaque and allows some optimizations.
- // The bitmap is not initialized.
+ // knows the bitmap will be completely opaque and allows some optimizations
+ // (the bitmap is not initialized to 0 when is_opaque == true).
static BitmapPlatformDevice* Create(int width, int height, bool is_opaque);
- // This doesn't take ownership of |data|. If |data| is null, the bitmap
- // is not initialized to 0.
+ // This doesn't take ownership of |data|. If |data| is null and |is_opaque|
+ // is false, the bitmap is initialized to 0.
+ //
+ // Note: historicaly, BitmapPlatformDevice impls have had diverging
+ // initialization behavior for null |data| (Cairo used to initialize, while
+ // the others did not). For now we stick to the more conservative Cairo
+ // behavior.
static BitmapPlatformDevice* Create(int width, int height, bool is_opaque,
uint8_t* data);
@@ -44,9 +50,6 @@ class BitmapPlatformDevice : public SkBitmapDevice, public PlatformDevice {
SkBaseDevice* onCreateDevice(const CreateInfo&, const SkPaint*) override;
private:
- NativeDrawingContext BeginPlatformPaint(const SkMatrix& transform,
- const SkIRect& clip_bounds) override;
-
DISALLOW_COPY_AND_ASSIGN(BitmapPlatformDevice);
};
« no previous file with comments | « skia/ext/bitmap_platform_device_mac.cc ('k') | skia/ext/bitmap_platform_device_skia.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698