RefCounted types should not have public destructors, content/browser part 1

content/browser/trace_message_filter.h

 // 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.
 
 #ifndef CONTENT_BROWSER_TRACE_MESSAGE_FILTER_H_
 #define CONTENT_BROWSER_TRACE_MESSAGE_FILTER_H_
 
 #include <string>
 #include <vector>
 
 #include "content/public/browser/browser_message_filter.h"
 
 // This class sends and receives trace messages on the browser process.
 // See also: trace_controller.h
 // See also: child_trace_message_filter.h
 class TraceMessageFilter : public content::BrowserMessageFilter {
  public:
   TraceMessageFilter();
-  virtual ~TraceMessageFilter();
 
   // content::BrowserMessageFilter override.
   virtual void OnFilterAdded(IPC::Channel* channel) OVERRIDE;
 
   // content::BrowserMessageFilter implementation.
   virtual void OnChannelClosing() OVERRIDE;
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok) OVERRIDE;
 
   void SendBeginTracing(const std::vector<std::string>& included_categories,
                         const std::vector<std::string>& excluded_categories);
   void SendEndTracing();
   void SendGetTraceBufferPercentFull();
 
+ protected:
+  virtual ~TraceMessageFilter();
+
  private:
   // Message handlers.
   void OnChildSupportsTracing();
   void OnEndTracingAck(const std::vector<std::string>& known_categories);
   void OnTraceBufferFull();
   void OnTraceBufferPercentFullReply(float percent_full);
   void OnTraceDataCollected(const std::string& data);
 
   // ChildTraceMessageFilter exists:
   bool has_child_;
 
   // Awaiting ack for previously sent SendEndTracing
   bool is_awaiting_end_ack_;
   bool is_awaiting_bpf_ack_;
 
   DISALLOW_COPY_AND_ASSIGN(TraceMessageFilter);
 };
 
 #endif  // CONTENT_BROWSER_TRACE_MESSAGE_FILTER_H_