RefCounted types should not have public destructors, webkit/

webkit/appcache/appcache_request_handler_unittest.cc

Index: webkit/appcache/appcache_request_handler_unittest.cc
diff --git a/webkit/appcache/appcache_request_handler_unittest.cc b/webkit/appcache/appcache_request_handler_unittest.cc
index 37b7bd1efab6f1c033f55b25243c339e8544c851..f91af0f00fe3d90a5e0a01e897cce58505e03d24 100644
--- a/webkit/appcache/appcache_request_handler_unittest.cc
+++ b/webkit/appcache/appcache_request_handler_unittest.cc
@@ -72,28 +72,34 @@ class AppCacheRequestHandlerTest : public testing::Test {
 
   class MockURLRequestJob : public net::URLRequestJob {
    public:
-    MockURLRequestJob(
-        net::URLRequest* request, int response_code)
+    MockURLRequestJob(net::URLRequest* request,
+                      int response_code)
         : net::URLRequestJob(request),
           response_code_(response_code),
           has_response_info_(false) {}
-    MockURLRequestJob(
-        net::URLRequest* request, const net::HttpResponseInfo& info)
+    MockURLRequestJob(net::URLRequest* request,
+                      const net::HttpResponseInfo& info)
         : net::URLRequestJob(request),
           response_code_(info.headers->response_code()),
           has_response_info_(true),
           response_info_(info) {}
-    virtual void Start() {
+
+  protected:
+    virtual ~MockURLRequestJob() {}
+    virtual void Start() OVERRIDE {
       NotifyHeadersComplete();
     }
-    virtual int GetResponseCode() const {
+    virtual int GetResponseCode() const OVERRIDE {
       return response_code_;
     }
-    virtual void GetResponseInfo(net::HttpResponseInfo* info) {
+    virtual void GetResponseInfo(
+        net::HttpResponseInfo* info) OVERRIDE {
       if (!has_response_info_)
         return;
       *info = response_info_;
     }
+
+  private:
     int response_code_;
     bool has_response_info_;
     net::HttpResponseInfo response_info_;