RefCounted types should not have public destructors, chrome/browser/ part 5

chrome/browser/safe_browsing/malware_details_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 <algorithm>
 
 #include "base/bind.h"
 #include "base/pickle.h"
 #include "base/time.h"
 #include "chrome/browser/history/history.h"
@@ skipping 120 matching lines @@
 class MalwareDetailsWrap : public MalwareDetails {
  public:
   MalwareDetailsWrap(SafeBrowsingService* sb_service,
                      WebContents* web_contents,
                      const SafeBrowsingService::UnsafeResource& unsafe_resource,
                      net::URLRequestContextGetter* request_context_getter)
       : MalwareDetails(sb_service, web_contents, unsafe_resource) {
     request_context_getter_ = request_context_getter;
   }
 
+ private:
   virtual ~MalwareDetailsWrap() {}
 };
 
 class MockSafeBrowsingService : public SafeBrowsingService {
  public:
   MockSafeBrowsingService() {}
-  virtual ~MockSafeBrowsingService() {}
 
   // When the MalwareDetails is done, this is called.
   virtual void SendSerializedMalwareDetails(const std::string& serialized) {
     DVLOG(1) << "SendSerializedMalwareDetails";
     // Notify WaitForSerializedReport.
     BrowserThread::PostTask(BrowserThread::IO,
                             FROM_HERE,
                             base::Bind(&QuitUIMessageLoop));
     serialized_ = serialized;
   }
 
   const std::string& GetSerialized() {
     return serialized_;
   }
 
  private:
+  virtual ~MockSafeBrowsingService() {}
+
   std::string serialized_;
   DISALLOW_COPY_AND_ASSIGN(MockSafeBrowsingService);
 };
 
 }  // namespace.
 
 class MalwareDetailsTest : public ChromeRenderViewHostTestHarness {
  public:
   MalwareDetailsTest()
       : ui_thread_(BrowserThread::UI, &message_loop_),
@@ skipping 505 matching lines @@
   pb_resource = expected.add_resources();
   pb_resource->set_id(2);
   pb_resource->set_parent_id(3);
   pb_resource->set_url(kSecondRedirectURL);
   pb_resource = expected.add_resources();
   pb_resource->set_id(3);
   pb_resource->set_url(kFirstRedirectURL);
 
   VerifyResults(actual, expected);
 }