mac: Fix a few memory leaks.

skia/ext/skia_utils_mac.mm

 // 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/skia_utils_mac.h"
 
 #import <AppKit/AppKit.h>
 
 #include "base/logging.h"
 #include "base/mac/scoped_cftyperef.h"
@@ skipping 345 matching lines @@
   useDeviceBits_ = deviceBits.getPixels();
   if (useDeviceBits_) {
     bitmap_ = deviceBits;
     bitmap_.lockPixels();
   } else {
     bitmap_.setConfig(
         SkBitmap::kARGB_8888_Config, deviceBits.width(), deviceBits.height());
     bitmap_.allocPixels();
     bitmap_.eraseColor(0);
   }
+  base::mac::ScopedCFTypeRef<CGColorSpaceRef> color_space(

[Robert Sesek, 2013/03/04 14:47:53]

It looks like the naming convention in skia is camelCase

[Nico, 2013/03/04 14:52:35]

Yes, but this file is chromium code, not skia code (skia proper is in
third_party/skia). The functions at the top use camel_case, only this class is
weird.

…but done.

+      CGColorSpaceCreateDeviceRGB());
   cgContext_ = CGBitmapContextCreate(bitmap_.getPixels(), bitmap_.width(),
-    bitmap_.height(), 8, bitmap_.rowBytes(), CGColorSpaceCreateDeviceRGB(), 
+    bitmap_.height(), 8, bitmap_.rowBytes(), color_space, 
     kCGBitmapByteOrder32Host | kCGImageAlphaPremultipliedFirst);
 
   // Apply device matrix.
   CGAffineTransform contentsTransform = CGAffineTransformMakeScale(1, -1);
   contentsTransform = CGAffineTransformTranslate(contentsTransform, 0,
       -device->height());
   CGContextConcatCTM(cgContext_, contentsTransform);
 
   const SkIPoint& pt = device->getOrigin();
   // Skip applying the clip when not writing directly to device.
@@ skipping 25 matching lines @@
   // Apply content matrix.
   SkMatrix skMatrix = canvas_->getTotalMatrix();
   skMatrix.postTranslate(-SkIntToScalar(pt.fX), -SkIntToScalar(pt.fY));
   CGAffineTransform affine = SkMatrixToCGAffineTransform(skMatrix);
   CGContextConcatCTM(cgContext_, affine);
   
   return cgContext_;
 }
 
 }  // namespace gfx