RefCounted types should not have public destructors, net/

net/url_request/url_request_job_factory_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 "net/url_request/url_request_job_factory.h"
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
 #include "net/url_request/url_request_job.h"
 #include "net/url_request/url_request_test_util.h"
@@ skipping 13 matching lines @@
   virtual void Start() {
     // Start reading asynchronously so that all error reporting and data
     // callbacks happen as they would for network requests.
     MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&MockURLRequestJob::StartAsync,
                    weak_factory_.GetWeakPtr()));
   }
 
  private:
+  virtual ~MockURLRequestJob() {}
+
   void StartAsync() {
     SetStatus(status_);
     NotifyHeadersComplete();
   }
 
   URLRequestStatus status_;
   base::WeakPtrFactory<MockURLRequestJob> weak_factory_;
 };
 
 class DummyProtocolHandler : public URLRequestJobFactory::ProtocolHandler {
@@ skipping 158 matching lines @@
   EXPECT_FALSE(interceptor->WillHandleProtocol("anything"));
   EXPECT_FALSE(job_factory.IsHandledProtocol("anything"));
   interceptor->handle_all_protocols_ = true;
   EXPECT_TRUE(interceptor->WillHandleProtocol("anything"));
   EXPECT_TRUE(job_factory.IsHandledProtocol("anything"));
 }
 
 }  // namespace
 
 }  // namespace net