RefCounted types should not have public destructors, net/

net/websockets/websocket_job_spdy3_unittest.cc

Index: net/websockets/websocket_job_spdy3_unittest.cc
diff --git a/net/websockets/websocket_job_spdy3_unittest.cc b/net/websockets/websocket_job_spdy3_unittest.cc
index 4b7e2a0af4a6342b520d39cf9f83a6d5d988ad86..6fd47676c0a9179e0e3dca2a875fdb955a27242c 100644
--- a/net/websockets/websocket_job_spdy3_unittest.cc
+++ b/net/websockets/websocket_job_spdy3_unittest.cc
@@ -43,8 +43,8 @@ namespace {
 class MockSocketStream : public net::SocketStream {
  public:
   MockSocketStream(const GURL& url, net::SocketStream::Delegate* delegate)
-      : SocketStream(url, delegate) {}
-  virtual ~MockSocketStream() {}
+      : SocketStream(url, delegate) {
+  }
 
   virtual void Connect() OVERRIDE {}
   virtual bool SendData(const char* data, int len) OVERRIDE {
@@ -65,6 +65,9 @@ class MockSocketStream : public net::SocketStream {
     return sent_data_;
   }
 
+ protected:
+  virtual ~MockSocketStream() {}
+
  private:
   std::string sent_data_;
 };
@@ -72,7 +75,10 @@ class MockSocketStream : public net::SocketStream {
 class MockSocketStreamDelegate : public net::SocketStream::Delegate {
  public:
   MockSocketStreamDelegate()
-      : amount_sent_(0), allow_all_cookies_(true) {}
+      : amount_sent_(0),
+      allow_all_cookies_(true) {
+  }
+
   void set_allow_all_cookies(bool allow_all_cookies) {
     allow_all_cookies_ = allow_all_cookies;
   }
@@ -94,24 +100,26 @@ class MockSocketStreamDelegate : public net::SocketStream::Delegate {
     on_close_ = callback;
   }
 
-  virtual int OnStartOpenConnection(net::SocketStream* socket,
-                                    const net::CompletionCallback& callback) {
+  virtual int OnStartOpenConnection(
+      net::SocketStream* socket,
+      const net::CompletionCallback& callback) OVERRIDE {
     if (!on_start_open_connection_.is_null())
       on_start_open_connection_.Run();
     return net::OK;
   }
   virtual void OnConnected(net::SocketStream* socket,
-                           int max_pending_send_allowed) {
+                           int max_pending_send_allowed) OVERRIDE {
     if (!on_connected_.is_null())
       on_connected_.Run();
   }
-  virtual void OnSentData(net::SocketStream* socket, int amount_sent) {
+  virtual void OnSentData(net::SocketStream* socket,
+                          int amount_sent) OVERRIDE {
     amount_sent_ += amount_sent;
     if (!on_sent_data_.is_null())
       on_sent_data_.Run();
   }
   virtual void OnReceivedData(net::SocketStream* socket,
-                              const char* data, int len) {
+                              const char* data, int len) OVERRIDE {
     received_data_ += std::string(data, len);
     if (!on_received_data_.is_null())
       on_received_data_.Run();
@@ -120,13 +128,14 @@ class MockSocketStreamDelegate : public net::SocketStream::Delegate {
     if (!on_close_.is_null())
       on_close_.Run();
   }
-  virtual bool CanGetCookies(net::SocketStream* socket, const GURL& url) {
+  virtual bool CanGetCookies(net::SocketStream* socket,
+                             const GURL& url) OVERRIDE {
     return allow_all_cookies_;
   }
   virtual bool CanSetCookie(net::SocketStream* request,
                             const GURL& url,
                             const std::string& cookie_line,
-                            net::CookieOptions* options) {
+                            net::CookieOptions* options) OVERRIDE {
     return allow_all_cookies_;
   }
 
@@ -151,11 +160,12 @@ class MockCookieStore : public net::CookieStore {
     std::string cookie_line;
     net::CookieOptions options;
   };
+
   MockCookieStore() {}
 
-  virtual bool SetCookieWithOptions(const GURL& url,
-                                    const std::string& cookie_line,
-                                    const net::CookieOptions& options) {
+  bool SetCookieWithOptions(const GURL& url,
+                            const std::string& cookie_line,
+                            const net::CookieOptions& options) {
     Entry entry;
     entry.url = url;
     entry.cookie_line = cookie_line;
@@ -164,18 +174,8 @@ class MockCookieStore : public net::CookieStore {
     return true;
   }
 
-  virtual void SetCookieWithOptionsAsync(
-      const GURL& url,
-      const std::string& cookie_line,
-      const net::CookieOptions& options,
-      const SetCookiesCallback& callback) {
-    bool result = SetCookieWithOptions(url, cookie_line, options);
-    if (!callback.is_null())
-      callback.Run(result);
-  }
-  virtual std::string GetCookiesWithOptions(
-      const GURL& url,
-      const net::CookieOptions& options) {
+  std::string GetCookiesWithOptions(const GURL& url,
+                                    const net::CookieOptions& options) {
     std::string result;
     for (size_t i = 0; i < entries_.size(); i++) {
       Entry &entry = entries_[i];
@@ -188,44 +188,51 @@ class MockCookieStore : public net::CookieStore {
     }
     return result;
   }
-  virtual void GetCookiesWithOptionsAsync(
+
+  // CookieStore:
+  virtual void SetCookieWithOptionsAsync(
       const GURL& url,
+      const std::string& cookie_line,
       const net::CookieOptions& options,
-      const GetCookiesCallback& callback) {
-  if (!callback.is_null())
-    callback.Run(GetCookiesWithOptions(url, options));
+      const SetCookiesCallback& callback) OVERRIDE {
+    bool result = SetCookieWithOptions(url, cookie_line, options);
+    if (!callback.is_null())
+      callback.Run(result);
   }
-  virtual void GetCookiesWithInfo(const GURL& url,
-                                  const net::CookieOptions& options,
-                                  std::string* cookie_line,
-                                  std::vector<CookieInfo>* cookie_infos) {
-    ADD_FAILURE();
+
+  virtual void GetCookiesWithOptionsAsync(
+      const GURL& url,
+      const net::CookieOptions& options,
+      const GetCookiesCallback& callback) OVERRIDE {
+    if (!callback.is_null())
+      callback.Run(GetCookiesWithOptions(url, options));
   }
+
   virtual void GetCookiesWithInfoAsync(
       const GURL& url,
       const net::CookieOptions& options,
-      const GetCookieInfoCallback& callback) {
-    ADD_FAILURE();
-  }
-  virtual void DeleteCookie(const GURL& url,
-                            const std::string& cookie_name) {
+      const GetCookieInfoCallback& callback) OVERRIDE {
     ADD_FAILURE();
   }
+
   virtual void DeleteCookieAsync(const GURL& url,
                                  const std::string& cookie_name,
-                                 const base::Closure& callback) {
+                                 const base::Closure& callback) OVERRIDE {
     ADD_FAILURE();
   }
-  virtual void DeleteAllCreatedBetweenAsync(const base::Time& delete_begin,
-                                            const base::Time& delete_end,
-                                            const DeleteCallback& callback) {
+
+  virtual void DeleteAllCreatedBetweenAsync(
+      const base::Time& delete_begin,
+      const base::Time& delete_end,
+      const DeleteCallback& callback) OVERRIDE {
     ADD_FAILURE();
   }
-  virtual void DeleteSessionCookiesAsync(const DeleteCallback&) {
+
+  virtual void DeleteSessionCookiesAsync(const DeleteCallback&) OVERRIDE {
     ADD_FAILURE();
   }
 
-  virtual net::CookieMonster* GetCookieMonster() { return NULL; }
+  virtual net::CookieMonster* GetCookieMonster() OVERRIDE { return NULL; }
 
   const std::vector<Entry>& entries() const { return entries_; }
 
@@ -238,7 +245,10 @@ class MockCookieStore : public net::CookieStore {
 
 class MockSSLConfigService : public net::SSLConfigService {
  public:
-  virtual void GetSSLConfig(net::SSLConfig* config) {};
+  virtual void GetSSLConfig(net::SSLConfig* config) OVERRIDE {}
+
+ protected:
+  virtual ~MockSSLConfigService() {}
 };
 
 class MockURLRequestContext : public net::URLRequestContext {
@@ -252,10 +262,11 @@ class MockURLRequestContext : public net::URLRequestContext {
     transport_security_state_.EnableHost("upgrademe.com", state);
   }
 
- private:
+ protected:
   friend class base::RefCountedThreadSafe<MockURLRequestContext>;
   virtual ~MockURLRequestContext() {}
 
+ private:
   net::TransportSecurityState transport_security_state_;
 };