Restored missing BrokerRpcClient::Connect.

ceee/ie/plugin/bho/executor.cc

Index: ceee/ie/plugin/bho/executor.cc
===================================================================
--- ceee/ie/plugin/bho/executor.cc	(revision 68235)
+++ ceee/ie/plugin/bho/executor.cc	(working copy)
@@ -370,7 +370,9 @@
   return hr;
 }
 
-CeeeExecutor::CeeeExecutor() : hwnd_(NULL) {
+CeeeExecutor::CeeeExecutor()
+    : hwnd_(NULL),
+      broker_rpc_client_(false) {

[Jói, 2010/12/04 14:03:58]

Please add a comment explaining why allow_restarts needs to be false in this
case.

[Vitaly Buka (NO REVIEWS), 2010/12/04 18:23:22]

Done.

 }
 
 CeeeExecutor::~CeeeExecutor() {
@@ -426,9 +428,13 @@
   // Infobar. In any case, the construction below should have a reference to
   // a BHO and its EventSender so we don't create Infobars before the tab_id
   // is ready.
-  if (window_utils::GetTopLevelParent(hwnd_) != hwnd_)
+  if (window_utils::GetTopLevelParent(hwnd_) != hwnd_) {
+    hr = broker_rpc_client_.Connect(true);
+    if (FAILED(hr))
+      return hr;
     infobar_manager_.reset(
-        new infobar_api::InfobarManager(hwnd_, new BrokerRpcClient(false)));
+        new infobar_api::InfobarManager(hwnd_, &broker_rpc_client_));
+  }
 
   return S_OK;
 }