RefCounted types should not have public destructors, printing/ and ui/

ui/gfx/gl/gl_surface_cgl.cc

 // Copyright (c) 2011 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 "ui/gfx/gl/gl_surface_cgl.h"
 
 #include <OpenGL/CGLRenderers.h>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/mac/mac_util.h"
 #include "ui/gfx/gl/gl_bindings.h"
 #include "ui/gfx/gl/gl_context.h"
 #include "ui/gfx/gl/gl_implementation.h"
 
 namespace gfx {
 
 namespace {
 CGLPixelFormatObj g_pixel_format;
 }
 
-GLSurfaceCGL::GLSurfaceCGL() {
-}
-
-GLSurfaceCGL::~GLSurfaceCGL() {
-}
+GLSurfaceCGL::GLSurfaceCGL() {}
 
 bool GLSurfaceCGL::InitializeOneOff() {
   static bool initialized = false;
   if (initialized)
     return true;
 
   // This is called from the sandbox warmup code on Mac OS X.
   // GPU-related stuff is very slow without this, probably because
   // the sandbox prevents loading graphics drivers or some such.
   std::vector<CGLPixelFormatAttribute> attribs;
@@ skipping 24 matching lines @@
   CGLReleasePixelFormat(format);
   DCHECK_NE(num_pixel_formats, 0);
   initialized = true;
   return true;
 }
 
 void* GLSurfaceCGL::GetPixelFormat() {
   return g_pixel_format;
 }
 
+GLSurfaceCGL::~GLSurfaceCGL() {}
+
 NoOpGLSurfaceCGL::NoOpGLSurfaceCGL(const gfx::Size& size)
   : size_(size) {
 }
 
-NoOpGLSurfaceCGL::~NoOpGLSurfaceCGL() {
-  Destroy();
-}
-
 bool NoOpGLSurfaceCGL::Initialize() {
   return true;
 }
 
 void NoOpGLSurfaceCGL::Destroy() {
 }
 
 bool NoOpGLSurfaceCGL::IsOffscreen() {
   return true;
 }
 
 bool NoOpGLSurfaceCGL::SwapBuffers() {
   NOTREACHED() << "Cannot call SwapBuffers on a NoOpGLSurfaceCGL.";
   return false;
 }
 
 gfx::Size NoOpGLSurfaceCGL::GetSize() {
   return size_;
 }
 
 void* NoOpGLSurfaceCGL::GetHandle() {
   return NULL;
 }
 
+NoOpGLSurfaceCGL::~NoOpGLSurfaceCGL() {
+  Destroy();
+}
+
 }  // namespace gfx