RefCounted types should not have public destructors, delegate cleanup

chrome/test/automation/javascript_execution_controller.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/automation/javascript_execution_controller.h"
 
 #include "base/json/json_string_value_serializer.h"
 #include "chrome/test/automation/javascript_message_utils.h"
 
 using javascript_utils::JavaScriptPrintf;
 
 // Initialize this timeout to an invalid value. Each test framework or test
 // must set an appropriate timeout using set_timeout, or the
 // JavaScriptExecutionController will complain.
 int JavaScriptExecutionController::timeout_ms_ = -1;
 
 JavaScriptExecutionController::JavaScriptExecutionController() {}
 
-JavaScriptExecutionController::~JavaScriptExecutionController() {}
-
 bool JavaScriptExecutionController::ExecuteJavaScript(
     const std::string& script) {
   scoped_ptr<Value> return_value;
   return ExecuteAndParseHelper(WrapJavaScript(script), &return_value);
 }
 
 bool JavaScriptExecutionController::ExecuteAsyncJavaScript(
     const std::string& script) {
   scoped_ptr<Value> return_value;
   return ExecuteAndParseHelper(WrapAsyncJavaScript(script), &return_value);
 }
 
+JavaScriptExecutionController::~JavaScriptExecutionController() {}
+
 void JavaScriptExecutionController::Remove(int handle) {
   handle_to_object_.erase(handle);
   if (handle_to_object_.empty())
     LastObjectRemoved();
 }
 
 std::string JavaScriptExecutionController::WrapJavaScript(
     const std::string& original_script) {
   const char* script =
       "domAutomationController.setAutomationId(0);"
@@ skipping 58 matching lines @@
     return false;
   }
   if (!success) {
     LOG(WARNING) << "JavaScript evaluation did not complete successfully: "
                  << evaluation_error;
     return false;
   }
   result->reset(evaluation_result_value);
   return true;
 }