Send screenshots back to the client for debugging

chrome/test/webdriver/commands/create_session.cc

Index: chrome/test/webdriver/commands/create_session.cc
diff --git a/chrome/test/webdriver/commands/create_session.cc b/chrome/test/webdriver/commands/create_session.cc
index b23158e02b18cc88c7d60e395ef09dbd92a235b7..8b37e87f4dce0f353fb2f8c4c940c7b9c83ba05c 100644
--- a/chrome/test/webdriver/commands/create_session.cc
+++ b/chrome/test/webdriver/commands/create_session.cc
@@ -37,6 +37,10 @@ void CreateSession::ExecutePost(Response* const response) {
     return;
   }
 
+  bool screenshot_on_error;

[kkania, 2011/03/16 18:19:10]

DictionaryValue* capabilities = NULL;
if (GetDictionaryParameter("desiredCapabilities", &capabilities)) {
  capabilities->GetBoolean("takeScreenshotOnError", &screenshot_on_error);
}

[Joe, 2011/03/17 00:15:25]

Shouldn't the bindings roll the dictionary as normal parameters to the url.

On 2011/03/16 18:19:10, kkania wrote:
Done.

+  GetBooleanParameter("screenshot_on_error", &screenshot_on_error);
+  session->set_screenshot_on_error(screenshot_on_error);
+
   VLOG(1) << "Created session " << session->id();
   std::ostringstream stream;
   stream << "http://" << session_manager->GetAddress() << "/session/"