RefCounted types should not have public destructors, webkit/

webkit/support/test_webmessageportchannel.cc

 // 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.
 
 #include "webkit/support/test_webmessageportchannel.h"
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMessagePortChannelClient.h"
@@ skipping 23 matching lines @@
   WebString data_;
   scoped_ptr<WebMessagePortChannelArray> ports_;
   DISALLOW_COPY_AND_ASSIGN(Message);
 };
 
 TestWebMessagePortChannel::TestWebMessagePortChannel()
     : client_(NULL) {
   AddRef();
 }
 
-TestWebMessagePortChannel::~TestWebMessagePortChannel() {
-}
-
 void TestWebMessagePortChannel::setClient(WebMessagePortChannelClient* client) {
   client_ = client;
 }
 
 void TestWebMessagePortChannel::destroy() {
   while (!message_queue_.empty()) {
     delete message_queue_.front();
     message_queue_.pop();
   }
   Release();
@@ skipping 18 matching lines @@
   if (message_queue_.empty())
     return false;
   scoped_ptr<Message> message(message_queue_.front());
   message_queue_.pop();
   *data = message->data();
   if (WebMessagePortChannelArray* message_ports = message->ports())
     ports.swap(*message_ports);
   return true;
 }
 
+TestWebMessagePortChannel::~TestWebMessagePortChannel() {}
+
 void TestWebMessagePortChannel::queueMessage(Message* message) {
   bool was_empty = message_queue_.empty();
   message_queue_.push(message);
   if (client_ && was_empty)
     client_->messageAvailable();
 }