RefCounted types should not have public destructors, net/

net/url_request/url_request_throttler_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 "net/url_request/url_request_throttler_manager.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/histogram.h"
 #include "base/pickle.h"
 #include "base/stringprintf.h"
@@ skipping 31 matching lines @@
       const TimeTicks& fake_now)
       : net::URLRequestThrottlerEntry(manager, ""),
         fake_time_now_(fake_now),
         mock_backoff_entry_(&backoff_policy_) {
     InitPolicy();
 
     mock_backoff_entry_.set_fake_now(fake_now);
     set_exponential_backoff_release_time(exponential_backoff_release_time);
     set_sliding_window_release_time(sliding_window_release_time);
   }
-  virtual ~MockURLRequestThrottlerEntry() {}
 
   void InitPolicy() {
     // Some tests become flaky if we have jitter.
     backoff_policy_.jitter_factor = 0.0;
 
     // This lets us avoid having to make multiple failures initially (this
     // logic is already tested in the BackoffEntry unit tests).
     backoff_policy_.num_errors_to_ignore = 0;
   }
 
@@ skipping 31 matching lines @@
   }
 
   void set_sliding_window_release_time(
       const base::TimeTicks& release_time) {
     URLRequestThrottlerEntry::set_sliding_window_release_time(
         release_time);
   }
 
   TimeTicks fake_time_now_;
   MockBackoffEntry mock_backoff_entry_;
+
+ private:
+  virtual ~MockURLRequestThrottlerEntry() {}
 };
 
 class MockURLRequestThrottlerManager : public URLRequestThrottlerManager {
  public:
   MockURLRequestThrottlerManager() : create_entry_index_(0) {}
 
   // Method to process the URL using URLRequestThrottlerManager protected
   // method.
   std::string DoGetUrlIdFromUrl(const GURL& url) { return GetIdFromUrl(url); }
 
@@ skipping 425 matching lines @@
         FAIL();
     }
 
     scoped_refptr<net::URLRequestThrottlerEntryInterface> entry_after =
         manager.RegisterRequestUrl(GURL("http://www.example.com/"));
     EXPECT_FALSE(entry_after->ShouldRejectRequest(0));
   }
 }
 
 }  // namespace net