win: Ref-counted classes should have non-public destructors.

ui/gl/gl_context_wgl.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.
 
 // This file implements the GLContextWGL and PbufferGLContext classes.
 
 #include "ui/gl/gl_context_wgl.h"
 
 #include "base/logging.h"
 #include "base/trace_event/trace_event.h"
 #include "ui/gl/gl_bindings.h"
 #include "ui/gl/gl_implementation.h"
 #include "ui/gl/gl_surface_wgl.h"
 
 namespace gfx {
 
 GLContextWGL::GLContextWGL(GLShareGroup* share_group)
-    : GLContextReal(share_group),
-      context_(NULL) {
-}
-
-GLContextWGL::~GLContextWGL() {
-  Destroy();
-}
-
-std::string GLContextWGL::GetExtensions() {
-  const char* extensions = NULL;
-  if (g_driver_wgl.fn.wglGetExtensionsStringARBFn)
-    extensions = wglGetExtensionsStringARB(GLSurfaceWGL::GetDisplayDC());
-  else if (g_driver_wgl.fn.wglGetExtensionsStringEXTFn)
-    extensions = wglGetExtensionsStringEXT();
-
-  if (extensions)
-    return GLContext::GetExtensions() + " " + extensions;
-
-  return GLContext::GetExtensions();
+    : GLContextReal(share_group), context_(NULL) {
 }
 
 bool GLContextWGL::Initialize(
     GLSurface* compatible_surface, GpuPreference gpu_preference) {
   // Get the handle of another initialized context in the share group _before_
   // setting context_. Otherwise this context will be considered initialized
   // and could potentially be returned by GetHandle.
   HGLRC share_handle = static_cast<HGLRC>(share_group()->GetHandle());
 
   context_ = wglCreateContext(
@@ skipping 88 matching lines @@
   DCHECK(IsCurrent(NULL));
   if (gfx::g_driver_wgl.ext.b_WGL_EXT_swap_control) {
     wglSwapIntervalEXT(interval);
   } else {
       LOG(WARNING) <<
           "Could not disable vsync: driver does not "
           "support WGL_EXT_swap_control";
   }
 }
 
+std::string GLContextWGL::GetExtensions() {
+  const char* extensions = NULL;
+  if (g_driver_wgl.fn.wglGetExtensionsStringARBFn)
+    extensions = wglGetExtensionsStringARB(GLSurfaceWGL::GetDisplayDC());
+  else if (g_driver_wgl.fn.wglGetExtensionsStringEXTFn)
+    extensions = wglGetExtensionsStringEXT();
+
+  if (extensions)
+    return GLContext::GetExtensions() + " " + extensions;

[dcheng, 2015/05/02 01:15:08]

I wonder how much code bloat we have from string's operator+

+
+  return GLContext::GetExtensions();
+}
+
+GLContextWGL::~GLContextWGL() {
+  Destroy();
+}
+
 }  // namespace gfx