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..bb80a1bfb905a40280a6ca57b74320804b93c7dc 100644
--- a/chrome/test/webdriver/commands/create_session.cc
+++ b/chrome/test/webdriver/commands/create_session.cc
@@ -37,6 +37,12 @@ void CreateSession::ExecutePost(Response* const response) {
     return;
   }
 
+  bool screenshot_on_error;

[kkania, 2011/03/17 18:35:11]

default to false

[Joe, 2011/03/18 00:33:32]

Done.

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

[kkania, 2011/03/17 18:35:11]

I didn't mean for you to delete the part of
session->set_screenshot_on_error(screenshot_on_error). Otherwise this doesn't do
anything.

[Joe, 2011/03/18 00:33:32]

Sorry I mistook the comment as to remove the ability for now.
On 2011/03/17 18:35:11, kkania wrote:
Done.

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