First patch in making destructors of refcounted objects private.

skia/ext/bitmap_platform_device_linux.cc

 // Copyright (c) 2009 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_linux.h"
 
 #include <cairo/cairo.h>
 
 namespace skia {
 
@@ skipping 37 matching lines @@
 
   cairo_t* GetContext();
   cairo_surface_t* GetSurface();
 
   // Sets the transform and clip operations. This will not update the Cairo
   // surface, but will mark the config as dirty. The next call of LoadConfig
   // will pick up these changes.
   void SetMatrixClip(const SkMatrix& transform, const SkRegion& region);
 
  protected:
-  friend class base::RefCounted<BitmapPlatformDeviceData>;
-
-  ~BitmapPlatformDeviceData();
-
   void LoadConfig();
 
   // The Cairo surface inside this DC.
   cairo_t* context_;
   cairo_surface_t *const surface_;
 
   // True when there is a transform or clip that has not been set to the
   // surface.  The surface is retrieved for every text operation, and the
   // transform and clip do not change as much. We can save time by not loading
   // the clip and transform for every one.
   bool config_dirty_;
 
   // Translation assigned to the DC: we need to keep track of this separately
   // so it can be updated even if the DC isn't created yet.
   SkMatrix transform_;
 
   // The current clipping
   SkRegion clip_region_;
 
   // Disallow copy & assign.
   BitmapPlatformDeviceData(const BitmapPlatformDeviceData&);
   BitmapPlatformDeviceData& operator=(
       const BitmapPlatformDeviceData&);
+
+ private:
+  friend class base::RefCounted<BitmapPlatformDeviceData>;
+
+  ~BitmapPlatformDeviceData();
 };
 
 BitmapPlatformDevice::BitmapPlatformDeviceData::BitmapPlatformDeviceData(
     cairo_surface_t* surface)
     : surface_(surface),
       config_dirty_(true) {  // Want to load the config next time.
   context_ = cairo_create(surface);
 }
 
 BitmapPlatformDevice::BitmapPlatformDeviceData::~BitmapPlatformDeviceData() {
@@ skipping 106 matching lines @@
   data_->SetMatrixClip(transform, region);
 }
 
 BitmapPlatformDevice& BitmapPlatformDevice::operator=(
     const BitmapPlatformDevice& other) {
   data_ = other.data_;
   return *this;
 }
 
 }  // namespace skia