Send screenshots back to the client for debugging

chrome/test/automation/tab_proxy.cc

Index: chrome/test/automation/tab_proxy.cc
diff --git a/chrome/test/automation/tab_proxy.cc b/chrome/test/automation/tab_proxy.cc
index 3c5315a23203568c0813736f13a93d1b6981bdda..2eb8dd27a1fa5a70a131150fe829ba5c0dcc7033 100644
--- a/chrome/test/automation/tab_proxy.cc
+++ b/chrome/test/automation/tab_proxy.cc
@@ -12,6 +12,7 @@
 #include "base/utf_string_conversions.h"
 #include "chrome/common/automation_messages.h"
 #include "chrome/common/json_value_serializer.h"
+#include "chrome/test/automation/automation_json_requests.h"
 #include "chrome/test/automation/automation_proxy.h"
 #include "chrome/test/automation/browser_proxy.h"
 #include "googleurl/src/gurl.h"
@@ -746,10 +747,14 @@ bool TabProxy::CaptureEntirePageAsPNG(const FilePath& path) {
   if (!is_valid())
     return false;
 
-  bool succeeded = false;
-  sender_->Send(new AutomationMsg_CaptureEntirePageAsPNG(handle_, path,

[Paweł Hajdan Jr., 2011/03/23 18:01:57]

Could you mark that IPC message as deprecated - just add a comment in automation
messages header (we can't remove it because of hardcoded message IDs in the
reference builds), and remove the handler from the automation provider?

-                                                         &succeeded));
-  return succeeded;
+  int browser_index, tab_index;
+  if (!SendGetIndicesFromTabHandleJSONRequest(
+         sender_, handle_, &browser_index, &tab_index)) {
+    return false;
+  }
+
+  return SendCaptureEntirePageJSONRequest(sender_, browser_index,
+                                          tab_index, path);
 }
 
 #if defined(OS_WIN)