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

ui/gl/gl_context_wgl.cc

Index: ui/gl/gl_context_wgl.cc
diff --git a/ui/gl/gl_context_wgl.cc b/ui/gl/gl_context_wgl.cc
index fa7f7064772a1d789e9266ba288145a3a3eaae24..8fd29e60a8b354d1c1a2797876093235ac0c883c 100644
--- a/ui/gl/gl_context_wgl.cc
+++ b/ui/gl/gl_context_wgl.cc
@@ -15,25 +15,7 @@
 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(
@@ -142,4 +124,21 @@ void GLContextWGL::OnSetSwapInterval(int interval) {
   }
 }
 
+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