RefCounted types should not have public destructors, chrome/browser/ui

chrome/browser/ui/webui/fileicon_source_unittest.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 "base/memory/ref_counted_memory.h"
 #include "base/message_loop.h"
 #include "chrome/browser/icon_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/fileicon_source.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 using content::BrowserThread;
 
 namespace {
 
 class TestFileIconSource : public FileIconSource {
  public:
   explicit TestFileIconSource() {}
 
   MOCK_METHOD3(FetchFileIcon, void(const FilePath& path,
                                    IconLoader::IconSize icon_size,
                                    int request_id));
+
+ private:
+  virtual ~TestFileIconSource() {}
 };
 
 class FileIconSourceTest : public testing::Test {
  public:
   FileIconSourceTest()
       : loop_(MessageLoop::TYPE_UI),
         ui_thread_(BrowserThread::UI, MessageLoop::current()),
         file_thread_(BrowserThread::FILE, MessageLoop::current()) {}
 
   static TestFileIconSource* CreateFileIconSource() {
@@ skipping 53 matching lines @@
 
 TEST_F(FileIconSourceTest, FileIconSource_Parse) {
   for (unsigned i = 0; i < arraysize(kBasicExpectations); i++) {
     scoped_refptr<TestFileIconSource> source(CreateFileIconSource());
     EXPECT_CALL(*source.get(),
                 FetchFileIcon(FilePath(kBasicExpectations[i].unescaped_path),
                               kBasicExpectations[i].size, i));
     source->StartDataRequest(kBasicExpectations[i].request_path, false, i);
   }
 }