Firing event to broker without worker thread.

ceee/ie/broker/broker.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // ICeeeBroker implementation
 
 #include "ceee/ie/broker/broker.h"
 
 #include "base/logging.h"
 #include "ceee/ie/broker/api_dispatcher.h"
@@ skipping 10 matching lines @@
   return S_OK;
 }
 
 STDMETHODIMP CeeeBroker::Execute(BSTR function, BSTR* response) {
   // This is DEPRECATED and we should use ChromePostman (see FireEvent).
   api_dispatcher_->HandleApiRequest(function, response);
   return S_OK;
 }
 
 STDMETHODIMP CeeeBroker::FireEvent(BSTR event_name, BSTR event_args) {
-  ChromePostman::GetInstance()->FireEvent(event_name, event_args);
+  ChromePostman::GetInstance()->FireEvent(CW2A(event_name), CW2A(event_args));

[Sigurður Ásgeirsson, 2010/11/19 18:29:30]

A NULL BSTR is synonymous with the empty string, so we have to pass through
com::ToString.
Also please use the chrome base string conversions, probably WideToUTF8 here.  

[Vitaly Buka corp, 2010/11/19 20:30:38]

Done.

   return S_OK;
 }
 
 STDMETHODIMP CeeeBroker::RegisterWindowExecutor(long thread_id,
                                                 IUnknown* executor) {
   // TODO(mad@chromium.org): Add security check here.
   return executors_manager_->RegisterWindowExecutor(thread_id, executor);
 }
 
 STDMETHODIMP CeeeBroker::UnregisterExecutor(long thread_id) {
   // TODO(mad@chromium.org): Add security check here.
   return executors_manager_->RemoveExecutor(thread_id);
 }
 
 STDMETHODIMP CeeeBroker::RegisterTabExecutor(long thread_id,
                                              IUnknown* executor) {
   // TODO(mad@chromium.org): Implement the proper manual/secure registration.
   return executors_manager_->RegisterTabExecutor(thread_id, executor);
 }
 
 STDMETHODIMP CeeeBroker::SetTabIdForHandle(long tab_id,
                                            CeeeWindowHandle handle) {
   // TODO(mad@chromium.org): Add security check here.
   DCHECK(tab_id != kInvalidChromeSessionId &&
       handle != reinterpret_cast<CeeeWindowHandle>(INVALID_HANDLE_VALUE));
   executors_manager_->SetTabIdForHandle(tab_id, reinterpret_cast<HWND>(handle));
   return S_OK;
 }