RefCounted types should not have public destructors, media/ and gpu/ edition

gpu/command_buffer/service/framebuffer_manager.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 "gpu/command_buffer/service/framebuffer_manager.h"
 #include "base/logging.h"
 #include "gpu/command_buffer/common/gles2_cmd_utils.h"
 
 namespace gpu {
 namespace gles2 {
 
 class RenderbufferAttachment
     : public FramebufferManager::FramebufferInfo::Attachment {
  public:
   explicit RenderbufferAttachment(
       RenderbufferManager::RenderbufferInfo* renderbuffer)
       : renderbuffer_(renderbuffer) {
   }
 
-  virtual ~RenderbufferAttachment() { }
-
   virtual GLsizei width() const {
     return renderbuffer_->width();
   }
 
   virtual GLsizei height() const {
     return renderbuffer_->height();
   }
 
   virtual GLenum internal_format() const {
     return renderbuffer_->internal_format();
@@ skipping 34 matching lines @@
     uint32 need = GLES2Util::GetChannelsNeededForAttachmentType(
         attachment_type);
     uint32 have = GLES2Util::GetChannelsForFormat(internal_format());
     return (need & have) != 0;
   }
 
   RenderbufferManager::RenderbufferInfo* renderbuffer() const {
     return renderbuffer_.get();
   }
 
+ protected:
+  virtual ~RenderbufferAttachment() { }
+
  private:
   RenderbufferManager::RenderbufferInfo::Ref renderbuffer_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderbufferAttachment);
 };
 
 class TextureAttachment
     : public FramebufferManager::FramebufferInfo::Attachment {
  public:
   TextureAttachment(
       TextureManager::TextureInfo* texture, GLenum target, GLint level)
       : texture_(texture),
         target_(target),
         level_(level) {
   }
 
-  virtual ~TextureAttachment() { }
-
   virtual GLsizei width() const {
     GLsizei temp_width = 0;
     GLsizei temp_height = 0;
     texture_->GetLevelSize(target_, level_, &temp_width, &temp_height);
     return temp_width;
   }
 
   virtual GLsizei height() const {
     GLsizei temp_width = 0;
     GLsizei temp_height = 0;
@@ skipping 48 matching lines @@
     GLenum internal_format = 0;
     if (!texture_->GetLevelType(target_, level_, &type, &internal_format)) {
       return false;
     }
     uint32 need = GLES2Util::GetChannelsNeededForAttachmentType(
         attachment_type);
     uint32 have = GLES2Util::GetChannelsForFormat(internal_format);
     return (need & have) != 0;
   }
 
+ protected:
+  virtual ~TextureAttachment() { }
+
  private:
   TextureManager::TextureInfo::Ref texture_;
   GLenum target_;
   GLint level_;
 
   DISALLOW_COPY_AND_ASSIGN(TextureAttachment);
 };
 
 FramebufferManager::FramebufferManager()
     : framebuffer_state_change_count_(1),
@@ skipping 283 matching lines @@
     FramebufferManager::FramebufferInfo* framebuffer) {
   DCHECK(framebuffer);
   return framebuffer->framebuffer_complete_state_count_id() ==
       framebuffer_state_change_count_;
 }
 
 }  // namespace gles2
 }  // namespace gpu