RefCounted types should not have public destructors, delegate cleanup

chrome/test/automation/javascript_execution_controller.h

Index: chrome/test/automation/javascript_execution_controller.h
diff --git a/chrome/test/automation/javascript_execution_controller.h b/chrome/test/automation/javascript_execution_controller.h
index 7f2679b194254cf7a11255aaa0860056fba70058..8332285637772af0b432ce3f2db371d0d0a98d5a 100644
--- a/chrome/test/automation/javascript_execution_controller.h
+++ b/chrome/test/automation/javascript_execution_controller.h
@@ -23,7 +23,6 @@ class JavaScriptExecutionController
     : public base::SupportsWeakPtr<JavaScriptExecutionController> {
  public:
   JavaScriptExecutionController();
-  virtual ~JavaScriptExecutionController();
 
   // Executes |script| and parse the return value. Returns whether the
   // execution and parsing succeeded.
@@ -76,6 +75,8 @@ class JavaScriptExecutionController
   static void set_timeout(int timeout_ms) { timeout_ms_ = timeout_ms; }
 
  protected:
+  virtual ~JavaScriptExecutionController();
+
   // Executes |script| and sets the JSON response |json|. Returns true
   // on success.
   virtual bool ExecuteJavaScriptAndGetJSON(const std::string& script,