TestOverrideEncoding hanging is because TabProxy::WaitForNavigation can not g...

chrome/browser/automation/automation_provider.cc

Index: chrome/browser/automation/automation_provider.cc
===================================================================
--- chrome/browser/automation/automation_provider.cc	(revision 27337)
+++ chrome/browser/automation/automation_provider.cc	(working copy)
@@ -1346,6 +1346,7 @@
   }
 
   AutomationMsg_CloseTab::WriteReplyParams(reply_message, false);
+  Send(reply_message);
 }
 
 void AutomationProvider::CloseBrowser(int browser_handle,
@@ -1747,6 +1748,12 @@
       }
     }
   }
+
+  // This "!wait_for_navigation || !success condition" logic looks suspicious.
+  // It will send a failure message when success is true but
+  // |wait_for_navigation| is false.
+  // TODO(phajdan.jr): investgate whether the reply param (currently
+  // AUTOMATION_MSG_NAVIGATION_ERROR) should depend on success.
   if (!wait_for_navigation || !success)
     AutomationMsg_ClickSSLInfoBarLink::WriteReplyParams(
         reply_message, AUTOMATION_MSG_NAVIGATION_ERROR);
@@ -1768,6 +1775,7 @@
     AutomationMsg_WaitForNavigation::WriteReplyParams(reply_message,
         controller == NULL ? AUTOMATION_MSG_NAVIGATION_ERROR :
                              AUTOMATION_MSG_NAVIGATION_SUCCESS);
+    Send(reply_message);
     return;
   }