RefCounted types should not have public destructors, net/ edition

net/base/net_log.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 NET_BASE_NET_LOG_H_
 #define NET_BASE_NET_LOG_H_
 #pragma once
 
 #include <string>
 
@@ skipping 103 matching lines @@
     // Constructs an observer that wants to see network events, with
     // the specified minimum event granularity.  A ThreadSafeObserver can only
     // observe a single NetLog at a time.
     //
     // Observers will be called on the same thread an entry is added on,
     // and are responsible for ensuring their own thread safety.
     //
     // Observers must stop watching a NetLog before either the Observer or the
     // NetLog is destroyed.
     ThreadSafeObserver();
-    virtual ~ThreadSafeObserver();
 
     // Returns the minimum log level for events this observer wants to
     // receive.  Must not be called when not watching a NetLog.
     LogLevel log_level() const;
 
     // Returns the NetLog we are currently watching, if any.  Returns NULL
     // otherwise.
     NetLog* net_log() const;
 
     // This method will be called on the thread that the event occurs on.  It
@@ skipping 11 matching lines @@
     // |params| - Optional (may be NULL) parameters for this event.
     //            The specific subclass of EventParameters is defined
     //            by the contract for events of this |type|.
     // TODO(eroman): Take a scoped_refptr<EventParameters> instead.
     virtual void OnAddEntry(EventType type,
                             const base::TimeTicks& time,
                             const Source& source,
                             EventPhase phase,
                             EventParameters* params) = 0;
 
+   protected:
+    virtual ~ThreadSafeObserver();
+
    private:
     friend class NetLog;
 
     // Both of these values are only modified by the NetLog.
     LogLevel log_level_;
     NetLog* net_log_;
 
     DISALLOW_COPY_AND_ASSIGN(ThreadSafeObserver);
   };
 
@@ skipping 238 matching lines @@
 
  private:
   BoundNetLog net_log_;
   const NetLog::EventType event_type_;
   scoped_refptr<NetLog::EventParameters> end_event_params_;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NET_LOG_H_