Removing command_execution_timeout_ms in favor of action_max_timeout_ms.

chrome/test/automation/automation_json_requests.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_TEST_AUTOMATION_AUTOMATION_JSON_REQUESTS_H_
 #define CHROME_TEST_AUTOMATION_AUTOMATION_JSON_REQUESTS_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 22 matching lines @@
 
 // Sends a JSON request to the chrome automation provider. Returns true
 // if the JSON request was successfully sent and the reply was received.
 // If true, |success| will be set to whether the JSON request was
 // completed successfully by the automation provider.
 bool SendAutomationJSONRequest(AutomationMessageSender* sender,
                                const std::string& request,
                                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(
     AutomationMessageSender* sender,
     int tab_id,
     int* browser_index,
     int* tab_index) WARN_UNUSED_RESULT;
 
 // Requests the current browser and tab indices for the given |TabProxy|
 // handle. Returns true on success.
@@ skipping 135 matching lines @@
     AutomationMessageSender* sender,
     int browser_index,
     int tab_index,
     const WebKeyEvent& key_event) WARN_UNUSED_RESULT;
 
 // Requests to wait for all tabs to stop loading. Returns true on success.
 bool SendWaitForAllTabsToStopLoadingJSONRequest(
     AutomationMessageSender* sender) WARN_UNUSED_RESULT;
 
 #endif  // CHROME_TEST_AUTOMATION_AUTOMATION_JSON_REQUESTS_H_