Code review changes. Incorporated all the suggestions from previous review.

One item not changed yet is to rename 'receiver' in ForwardMessageToExternalHost.
The idea is to invoke receiver("message") at the other end. So for example if the
args to ForwardMessageToExternalHost("hello", "world") then we will invoke a
script hello("world") on the other side.

'receiver' doesn't really describe the first argument here so if there is a 
better suggestion, I would be happy to change it :)


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1176 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 4 deletions

diff --git a/chrome/test/automation/tab_proxy.cc b/chrome/test/automation/tab_proxy.cc
index 6171ddb..7c82b5c 100644
--- a/chrome/test/automation/tab_proxy.cc
+++ b/chrome/test/automation/tab_proxy.cc
@@ -929,13 +929,15 @@ bool TabProxy::SavePage(const std::wstring& file_name,
   return succeeded;
 }
 
-void TabProxy::PostMessage(AutomationHandle handle,
-                           const std::string& target,
-                           const std::string& message) {
+void TabProxy::HandleMessageFromExternalHost(AutomationHandle handle,
+                                             const std::string& target,
+                                             const std::string& message) {
   if (!is_valid())
     return;
 
   bool succeeded =
-      sender_->Send(new AutomationMsg_PostMessage(0, handle, target, message));
+      sender_->Send(new AutomationMsg_HandleMessageFromExternalHost(0, handle,
+                                                                    target,
+                                                                    message));
   DCHECK(succeeded);
 }