RefCounted types should not have public destructors, delegate cleanup

content/common/gpu/image_transport_surface.h

 // 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.
 
 #ifndef CONTENT_COMMON_GPU_IMAGE_TRANSPORT_SURFACE_H_
 #define CONTENT_COMMON_GPU_IMAGE_TRANSPORT_SURFACE_H_
 #pragma once
 
 #if defined(ENABLE_GPU)
 
@@ skipping 41 matching lines @@
 // should be responded to. The factory returns an offscreen surface that looks
 // a lot like an onscreen surface to the GPU process.
 //
 // The ImageTransportSurfaceHelper provides some glue to the outside world:
 // making sure outside events reach the ImageTransportSurface and
 // allowing the ImageTransportSurface to send events to the outside world.
 
 class ImageTransportSurface {
  public:
   ImageTransportSurface();
-  virtual ~ImageTransportSurface();
 
   virtual void OnNewSurfaceACK(
       uint64 surface_id, TransportDIB::Handle surface_handle) = 0;
   virtual void OnBuffersSwappedACK() = 0;
   virtual void OnPostSubBufferACK() = 0;
   virtual void OnResizeViewACK() = 0;
   virtual void OnResize(gfx::Size size) = 0;
 
   // Creates the appropriate surface depending on the GL implementation.
   static scoped_refptr<gfx::GLSurface>
       CreateSurface(GpuChannelManager* manager,
                     GpuCommandBufferStub* stub,
                     const gfx::GLSurfaceHandle& handle);
  protected:
   // Used by certain implements of PostSubBuffer to determine
   // how much needs to be copied between frames.
   void GetRegionsToCopy(const gfx::Rect& previous_damage_rect,
                         const gfx::Rect& new_damage_rect,
                         std::vector<gfx::Rect>* regions);
 
+ protected:
+  virtual ~ImageTransportSurface();
+
  private:
   DISALLOW_COPY_AND_ASSIGN(ImageTransportSurface);
 };
 
 class ImageTransportHelper : public IPC::Channel::Listener {
  public:
   // Takes weak pointers to objects that outlive the helper.
   ImageTransportHelper(ImageTransportSurface* surface,
                        GpuChannelManager* manager,
                        GpuCommandBufferStub* stub,
@@ skipping 89 matching lines @@
   gfx::Size new_size_;
   bool transport_;
   bool did_set_swap_interval_;
 
   DISALLOW_COPY_AND_ASSIGN(PassThroughImageTransportSurface);
 };
 
 #endif  // defined(ENABLE_GPU)
 
 #endif  // CONTENT_COMMON_GPU_IMAGE_TRANSPORT_SURFACE_H_