RefCounted types should not have public destructors, delegate cleanup

content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.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_CLIENT_WEBGRAPHICSCONTEXT3D_COMMAND_BUFFER_IMPL_H_
 #define CONTENT_COMMON_GPU_CLIENT_WEBGRAPHICSCONTEXT3D_COMMAND_BUFFER_IMPL_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 40 matching lines @@
 using WebKit::WGC3Duint;
 using WebKit::WGC3Dfloat;
 using WebKit::WGC3Dclampf;
 using WebKit::WGC3Dintptr;
 using WebKit::WGC3Dsizeiptr;
 
 // TODO(piman): move this logic to the compositor and remove it from the
 // context...
 class WebGraphicsContext3DSwapBuffersClient {
  public:
-  virtual ~WebGraphicsContext3DSwapBuffersClient() { }
   virtual void OnViewContextSwapBuffersPosted() = 0;
   virtual void OnViewContextSwapBuffersComplete() = 0;
   virtual void OnViewContextSwapBuffersAborted() = 0;
+
+ protected:
+  virtual ~WebGraphicsContext3DSwapBuffersClient() {}
 };
 
 class WebGraphicsContext3DErrorMessageCallback;
 
 class WebGraphicsContext3DCommandBufferImpl
     : public WebKit::WebGraphicsContext3D,
       public base::SupportsWeakPtr<WebGraphicsContext3DCommandBufferImpl> {
  public:
   WebGraphicsContext3DCommandBufferImpl(
       int surface_id,
@@ skipping 620 matching lines @@
   CommandBufferProxy* command_buffer_;
   gpu::gles2::GLES2CmdHelper* gles2_helper_;
   gpu::TransferBuffer* transfer_buffer_;
   gpu::gles2::GLES2Implementation* gl_;
   Error last_error_;
   int frame_number_;
   bool bind_generates_resources_;
 };
 
 #endif  // CONTENT_COMMON_GPU_CLIENT_WEBGRAPHICSCONTEXT3D_COMMAND_BUFFER_IMPL_H_