Optionally clear PlatformCanvas instances.

skia/ext/bitmap_platform_device_mac.cc

 // Copyright (c) 2012 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/bitmap_platform_device_mac.h"
 
 #import <ApplicationServices/ApplicationServices.h>
 #include <time.h>
 
 #include "base/mac/mac_util.h"
@@ skipping 95 matching lines @@
 }
 
 
 // We use this static factory function instead of the regular constructor so
 // that we can create the pixel data before calling the constructor. This is
 // required so that we can call the base class' constructor with the pixel
 // data.
 BitmapPlatformDevice* BitmapPlatformDevice::Create(CGContextRef context,
                                                    int width,
                                                    int height,
-                                                   bool is_opaque) {
+                                                   int flags) {
   SkBitmap bitmap;
   bitmap.setConfig(SkBitmap::kARGB_8888_Config, width, height);
   if (bitmap.allocPixels() != true)
     return NULL;
 
   void* data = NULL;
   if (context) {
     data = CGBitmapContextGetData(context);
     bitmap.setPixels(data);
   } else {
     data = bitmap.getPixels();
 
     // Note: The Windows implementation clears the Bitmap later on.
     // This bears mentioning since removal of this line makes the
     // unit tests only fail periodically (or when MallocPreScribble is set).
-    bitmap.eraseARGB(0, 0, 0, 0);
+    if (flags & FLAGS_INITIALIZED)
+      bitmap.eraseARGB(0, 0, 0, 0);
   }
 
-  bitmap.setIsOpaque(is_opaque);
+  bitmap.setIsOpaque(flags & FLAGS_OPAQUE);
 
   // If we were given data, then don't clobber it!
 #ifndef NDEBUG
-  if (!context && is_opaque) {
+  if (!context && (flags & FLAGS_OPAQUE)) {
     // To aid in finding bugs, we set the background color to something
     // obviously wrong so it will be noticable when it is not cleared
     bitmap.eraseARGB(255, 0, 255, 128);  // bright bluish green
   }
 #endif
 
   if (!context) {
     context = CGContextForData(data, width, height);
     if (!context)
       return NULL;
   } else
     CGContextRetain(context);
 
   BitmapPlatformDevice* rv = new BitmapPlatformDevice(
       new BitmapPlatformDeviceData(context), bitmap);
 
   // The device object took ownership of the graphics context with its own
   // CGContextRetain call.
   CGContextRelease(context);
 
   return rv;
 }
 
 BitmapPlatformDevice* BitmapPlatformDevice::CreateWithData(uint8_t* data,
                                                            int width,
                                                            int height,
-                                                           bool is_opaque) {
+                                                           int flags) {
   CGContextRef context = NULL;
   if (data)
     context = CGContextForData(data, width, height);
 
-  BitmapPlatformDevice* rv = Create(context, width, height, is_opaque);
+  BitmapPlatformDevice* rv = Create(context, width, height, flags);
 
   // The device object took ownership of the graphics context with its own
   // CGContextRetain call.
   if (context)
     CGContextRelease(context);
 
   return rv;
 }
 
 // The device will own the bitmap, which corresponds to also owning the pixel
@@ skipping 53 matching lines @@
 
 const SkBitmap& BitmapPlatformDevice::onAccessBitmap(SkBitmap* bitmap) {
   // Not needed in CoreGraphics
   return *bitmap;
 }
 
 SkDevice* BitmapPlatformDevice::onCreateCompatibleDevice(
     SkBitmap::Config config, int width, int height, bool isOpaque,
     Usage /*usage*/) {
   SkASSERT(config == SkBitmap::kARGB_8888_Config);
-  return BitmapPlatformDevice::Create(NULL, width, height, isOpaque);
+  return BitmapPlatformDevice::Create(NULL, width, height,
+                                      GetDefaultFlags(isOpaque));
 }
 
 }  // namespace skia