RefCounted types should not have public destructors, chrome/browser/ part 6

chrome/browser/importer/firefox_importer_unittest_utils_mac.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 "chrome/browser/importer/firefox_importer_unittest_utils.h"
 
 #include "base/bind.h"
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
@@ skipping 148 matching lines @@
 // A message_loop task that quits the message loop when invoked, setting cancel
 // causes the task to do nothing when invoked.
 class CancellableQuitMsgLoop : public base::RefCounted<CancellableQuitMsgLoop> {
  public:
   CancellableQuitMsgLoop() : cancelled_(false) {}
   void QuitNow() {
     if (!cancelled_)
       MessageLoop::current()->Quit();
   }
   bool cancelled_;
+
+ private:
+  friend class base::RefCounted<CancellableQuitMsgLoop>;
+  ~CancellableQuitMsgLoop() {}
 };
 
 // Spin until either a client response arrives or a timeout occurs.
 bool FFUnitTestDecryptorProxy::WaitForClientResponse() {
   // What we're trying to do here is to wait for an RPC message to go over the
   // wire and the client to reply.  If the client does not reply by a given
   // timeout we kill the message loop.
   // The way we do this is to post a CancellableQuitMsgLoop for 3 seconds in
   // the future and cancel it if an RPC message comes back earlier.
   // This relies on the IPC listener class to quit the message loop itself when
@@ skipping 86 matching lines @@
 
   IPC::Channel channel(kTestChannelID, IPC::Channel::MODE_CLIENT, &listener);
   CHECK(channel.Connect());
   listener.SetSender(&channel);
 
   // run message loop
   MessageLoop::current()->Run();
 
   return 0;
 }