Even more test cleanup. Some fixes to non-test code.

chrome/test/webdriver/commands/execute_command.cc

 // Copyright (c) 2010 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 <string>
 
 #include "base/json/json_reader.h"
 
 #include "chrome/test/webdriver/commands/execute_command.h"
 
 namespace webdriver {
 
 const char kArgs[] = "args";
 const char kScript[] = "script";
 
+ExecuteCommand::ExecuteCommand(const std::vector<std::string>& path_segments,
+                               const DictionaryValue* const parameters)
+    : WebDriverCommand(path_segments, parameters) {}
+
+ExecuteCommand::~ExecuteCommand() {}
+
+
 bool ExecuteCommand::Init(Response* const response) {
   if (!WebDriverCommand::Init(response)) {
     SET_WEBDRIVER_ERROR(response, "Failure on Init for execute command",
                         kInternalServerError);
     return false;
   }
 
   if (!GetStringParameter(kScript, &script_)) {
     SET_WEBDRIVER_ERROR(response, "No script to execute specified",
                         kBadRequest);
     return false;
   }
 
   has_args_= GetStringASCIIParameter(kArgs, &args_);
   return true;
 }
 
+bool ExecuteCommand::DoesPost() {
+  return true;
+}
+
 void ExecuteCommand::ExecutePost(Response* const response) {
   int error_code = 0;
   std::string error_msg;
   Value* params = NULL;
   Value* result = NULL;
 
   if (has_args_) {
     params = base::JSONReader::ReadAndReturnError(args_, true,
                                                   &error_code, &error_msg);
     if (error_code != 0) {
@@ skipping 23 matching lines @@
   if (error != kSuccess) {
     SET_WEBDRIVER_ERROR(response, "Failed to execute script",
                         kInternalServerError);
     return;
   }
 
   response->set_value(result);
   response->set_status(kSuccess);
 }
 
+bool ExecuteCommand::RequiresValidTab() {
+  return true;
+}
+
 }  // namspace webdriver