Removing command_execution_timeout_ms in favor of action_max_timeout_ms.

chrome/test/automation/automation_json_requests.h

Index: chrome/test/automation/automation_json_requests.h
===================================================================
--- chrome/test/automation/automation_json_requests.h	(revision 79305)
+++ chrome/test/automation/automation_json_requests.h	(working copy)
@@ -40,6 +40,12 @@
                                std::string* reply,
                                bool* success) WARN_UNUSED_RESULT;
 
+bool SendAutomationJSONRequest(AutomationMessageSender* sender,

[Paweł Hajdan Jr., 2011/03/26 10:55:12]

nit: SendAutomationJSONRequest seems to be used in less than ~10 places. Could
you make the version with timeout argument mandatory (i.e. remove the version
above), and use some timeout from TestTimeouts in the callers?

[Huyen, 2011/03/29 00:11:39]

Done.

+                               const std::string& request,
+                               std::string* reply,
+                               bool* success,
+                               int timeout) WARN_UNUSED_RESULT;
+
 // Requests the current browser and tab indices for the given tab ID.
 // Returns true on success.
 bool SendGetIndicesFromTabIdJSONRequest(