RefCounted types should not have public destructors, webkit/

webkit/plugins/npapi/plugin_lib_unittest.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 "webkit/plugins/npapi/plugin_lib.h"
 
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace webkit {
 namespace npapi {
 
 // Test the unloading of plugin libs. Bug http://crbug.com/46526 showed that
 // if UnloadAllPlugins() simply iterates through the g_loaded_libs global
 // variable, we can get a crash if no plugin libs were marked as always loaded.
 class PluginLibTest : public PluginLib {
  public:
-  PluginLibTest() : PluginLib(WebPluginInfo(), NULL) {
-  }
+  PluginLibTest() : PluginLib(WebPluginInfo(), NULL) {}
   using PluginLib::Unload;
+
+ protected:
+  virtual ~PluginLibTest() {}
 };
 
 TEST(PluginLibLoading, UnloadAllPlugins) {
   // For the creation of the g_loaded_libs global variable.
   ASSERT_EQ(static_cast<PluginLibTest*>(NULL),
       PluginLibTest::CreatePluginLib(FilePath()));
 
   // Try with a single plugin lib.
   scoped_refptr<PluginLibTest> plugin_lib1(new PluginLibTest());
   PluginLib::UnloadAllPlugins();
@@ skipping 138 matching lines @@
       "IcedTea NPR Web Browser Plugin "
       "(using IcedTea6 1.9.3 (fedora-49.1.9.3.fc14-i386))`",
       &info);
   EXPECT_EQ(ASCIIToUTF16("1.9.3"), info.version);
 }
 
 #endif  // defined(OS_POSIX) && !defined(OS_MACOSX)
 
 }  // namespace npapi
 }  // namespace webkit