RefCounted types should not have public destructors, chrome/ remaining parts

chrome/test/base/ui_test_utils.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/test/base/ui_test_utils.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
@@ skipping 189 matching lines @@
 
   virtual void FirstObjectAdded() {
     AddRef();
   }
 
   virtual void LastObjectRemoved() {
     Release();
   }
 
  private:
+  friend class base::RefCounted<InProcessJavaScriptExecutionController>;
+  virtual ~InProcessJavaScriptExecutionController() {}
+
   // Weak pointer to the associated RenderViewHost.
   RenderViewHost* render_view_host_;
 };
 
 // Specifying a prototype so that we can add the WARN_UNUSED_RESULT attribute.
 bool ExecuteJavaScriptHelper(RenderViewHost* render_view_host,
                              const std::wstring& frame_xpath,
                              const std::wstring& original_script,
                              scoped_ptr<Value>* result) WARN_UNUSED_RESULT;
 
@@ skipping 936 matching lines @@
                int state,
                const base::Closure& followup) {
   if (!followup.is_null())
     ui_controls::SendMouseEventsNotifyWhenDone(button, state, followup);
   else
     ui_controls::SendMouseEvents(button, state);
 }
 
 }  // namespace internal
 }  // namespace ui_test_utils