RefCounted types should not have public destructors

chrome/browser/net/connection_tester.h

 // 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.
 
 #ifndef CHROME_BROWSER_NET_CONNECTION_TESTER_H_
 #define CHROME_BROWSER_NET_CONNECTION_TESTER_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 85 matching lines @@
 
   typedef std::vector<Experiment> ExperimentList;
 
   // "Delegate" is an interface for receiving start and completion notification
   // of individual tests that are run by the ConnectionTester.
   //
   // NOTE: do not delete the ConnectionTester when executing within one of the
   // delegate methods.
   class Delegate {
    public:
-    virtual ~Delegate() {}
-
     // Called once the test suite is about to start.
     virtual void OnStartConnectionTestSuite() = 0;
 
     // Called when an individual experiment is about to be started.
     virtual void OnStartConnectionTestExperiment(
         const Experiment& experiment) = 0;
 
     // Called when an individual experiment has completed.
     //   |experiment| - the experiment that has completed.
     //   |result| - the net error that the experiment completed with
     //              (or net::OK if it was success).
     virtual void OnCompletedConnectionTestExperiment(
         const Experiment& experiment,
         int result) = 0;
 
     // Called once ALL tests have completed.
     virtual void OnCompletedConnectionTestSuite() = 0;
+
+   protected:
+    virtual ~Delegate() {}
   };
 
   // Constructs a ConnectionTester that notifies test progress to |delegate|.
   // |delegate| is owned by the caller, and must remain valid for the lifetime
   // of ConnectionTester.
   ConnectionTester(Delegate* delegate,
                    net::URLRequestContext* proxy_request_context);
 
   // Note that destruction cancels any in-progress tests.
   ~ConnectionTester();
@@ skipping 38 matching lines @@
   // The ordered list of experiments to try next. The experiment at the front
   // of the list is the one currently in progress.
   ExperimentList remaining_experiments_;
 
   net::URLRequestContext* const proxy_request_context_;
 
   DISALLOW_COPY_AND_ASSIGN(ConnectionTester);
 };
 
 #endif  // CHROME_BROWSER_NET_CONNECTION_TESTER_H_