Removing command_execution_timeout_ms in favor of action_max_timeout_ms.

chrome/test/automation/browser_proxy.h

-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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_BROWSER_PROXY_H_
 #define CHROME_TEST_AUTOMATION_BROWSER_PROXY_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
@@ skipping 206 matching lines @@
   //   2. Call an automation method that results in opening the popup menu.
   //   3. Call WaitForPopupMenuToOpen and check for success.
   // Both methods return true on success.
   bool StartTrackingPopupMenus() WARN_UNUSED_RESULT;
   bool WaitForPopupMenuToOpen() WARN_UNUSED_RESULT;
 
   // Generic pattern for sending automation requests.
   bool SendJSONRequest(const std::string& request,
                        std::string* response) WARN_UNUSED_RESULT;
 
+  bool SendJSONRequest(const std::string& request,

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

nit: Please remove the version that doesn't take a timeout.

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

Done.

+                       std::string* response,
+                       int timeout) WARN_UNUSED_RESULT;
+
   // Gets the load times for all tabs started from the command line.
   // Puts the time of the first tab to start loading into |min_start_time|,
   // the time when loading stopped into |max_stop_time| (should be similar to
   // the delay that WaitForInitialLoads waits for), and a list of all
   // finished timestamps into |stop_times|. Returns true on success.
   bool GetInitialLoadTimes(float* min_start_time, float* max_stop_time,
                            std::vector<float>* stop_times);
 
 
  protected:
   virtual ~BrowserProxy() {}
  private:
   DISALLOW_COPY_AND_ASSIGN(BrowserProxy);
 };
 
 #endif  // CHROME_TEST_AUTOMATION_BROWSER_PROXY_H_