Restart of ceee_broker on crash.

ceee/ie/plugin/bho/browser_helper_object.cc

Index: ceee/ie/plugin/bho/browser_helper_object.cc
===================================================================
--- ceee/ie/plugin/bho/browser_helper_object.cc	(revision 67655)
+++ ceee/ie/plugin/bho/browser_helper_object.cc	(working copy)
@@ -102,6 +102,7 @@
       thread_id_(::GetCurrentThreadId()),
       full_tab_chrome_frame_(false),
       broker_client_queue_(this),
+      broker_rpc_(false),
       tab_events_funnel_(broker_client()) {
   TRACE_EVENT_BEGIN("ceee.bho", this, "");
 }
@@ -840,7 +841,7 @@
 
 HRESULT BrowserHelperObject::PostMessageImpl(int port_id,
                                              const std::string& message) {
-    return extension_port_manager_.PostMessage(port_id, message);
+  return extension_port_manager_.PostMessage(port_id, message);
 }
 
 HRESULT BrowserHelperObject::OnCfPrivateMessage(BSTR msg,
@@ -955,7 +956,7 @@
   HandleNavigateComplete(webbrowser, url_bstr);
 
   for (std::vector<Sink*>::iterator iter = sinks_.begin();
-    iter != sinks_.end(); ++iter) {
+       iter != sinks_.end(); ++iter) {
       (*iter)->OnNavigateComplete(webbrowser, url_bstr);

[Sigurður Ásgeirsson, 2010/12/01 14:23:54]

nit: indent -1

[MAD, 2010/12/01 16:35:23]

Actually, -1 for "iter !=" and -2 for "(*iter)->"

[Vitaly Buka corp, 2010/12/01 19:06:04]

That is strange. I didn't touch this code.
I'll fix for consistency -2. I didn't do -1 because this file has 7 loops in
this way.
On 2010/12/01 16:35:23, MAD wrote:

   }
 }
@@ -1379,7 +1380,7 @@
   READYSTATE old_state = lower_bound_ready_state_;
   READYSTATE new_state = new_lowest_ready_state;
   if (old_state == new_state)
-    return S_OK;
+  return S_OK;

[Sigurður Ásgeirsson, 2010/12/01 14:23:54]

nit: indent +2

[Vitaly Buka corp, 2010/12/01 19:06:04]

Same here.
On 2010/12/01 14:23:54, Ruðrugis wrote:

 
   // Remember the new lowest ready state as our current one.
   lower_bound_ready_state_ = new_state;