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

content/browser/trace_message_filter.cc

 // 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.
 
 #include "content/browser/trace_message_filter.h"
 
 #include "content/browser/trace_controller_impl.h"
 #include "content/common/child_process_messages.h"
 
 using content::BrowserMessageFilter;
 using content::BrowserThread;
 using content::TraceControllerImpl;
 
 TraceMessageFilter::TraceMessageFilter() :
     has_child_(false),
     is_awaiting_end_ack_(false),
     is_awaiting_bpf_ack_(false) {
 }
 
-TraceMessageFilter::~TraceMessageFilter() {
-}
-
 void TraceMessageFilter::OnFilterAdded(IPC::Channel* channel) {
   // Always on IO thread (BrowserMessageFilter guarantee).
   BrowserMessageFilter::OnFilterAdded(channel);
 }
 
 void TraceMessageFilter::OnChannelClosing() {
   // Always on IO thread (BrowserMessageFilter guarantee).
   BrowserMessageFilter::OnChannelClosing();
 
   if (has_child_) {
@@ skipping 41 matching lines @@
   Send(new ChildProcessMsg_EndTracing);
 }
 
 void TraceMessageFilter::SendGetTraceBufferPercentFull() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(!is_awaiting_bpf_ack_);
   is_awaiting_bpf_ack_ = true;
   Send(new ChildProcessMsg_GetTraceBufferPercentFull);
 }
 
+TraceMessageFilter::~TraceMessageFilter() {}
+
 void TraceMessageFilter::OnChildSupportsTracing() {
   has_child_ = true;
   TraceControllerImpl::GetInstance()->AddFilter(this);
 }
 
 void TraceMessageFilter::OnEndTracingAck(
     const std::vector<std::string>& known_categories) {
   // is_awaiting_end_ack_ should always be true here, but check in case the
   // child process is compromised.
   if (is_awaiting_end_ack_) {
@@ skipping 13 matching lines @@
 }
 
 void TraceMessageFilter::OnTraceBufferPercentFullReply(float percent_full) {
   if (is_awaiting_bpf_ack_) {
     is_awaiting_bpf_ack_ = false;
     TraceControllerImpl::GetInstance()->OnTraceBufferPercentFullReply(
         percent_full);
   }
 }