Firing event to broker without worker thread.

ceee/ie/broker/chrome_postman.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.
 //
 // ChromePostman implementation.
 #include "ceee/ie/broker/chrome_postman.h"
 
 #include "base/string_util.h"
 #include "ceee/ie/broker/api_dispatcher.h"
 #include "ceee/ie/common/api_registration.h"
@@ skipping 50 matching lines @@
 
   virtual void Run() {
     ProductionApiDispatcher::get()->HandleApiRequest(message_, NULL);
   }
  private:
   CComBSTR message_;
 };
 
 class FireEventTask : public Task {
  public:
-  FireEventTask(BSTR event_name, BSTR event_args)
+  FireEventTask(const char* event_name, const char* event_args)
       : event_name_(event_name), event_args_(event_args) {}
 
   virtual void Run() {
-    ProductionApiDispatcher::get()->FireEvent(event_name_, event_args_);
+    ProductionApiDispatcher::get()->FireEvent(event_name_.c_str(),
+                                              event_args_.c_str());
   }
  private:
-  CComBSTR event_name_;
-  CComBSTR event_args_;
+  std::string event_name_;
+  std::string event_args_;
 };
 
 
 ChromePostman* ChromePostman::single_instance_ = NULL;
 ChromePostman::ChromePostman() {
   DCHECK(single_instance_ == NULL);
   single_instance_ = this;
   frame_reset_count_ = 0;
 }
 
@@ skipping 21 matching lines @@
   if (message_loop) {
     message_loop->PostTask(
         FROM_HERE, new ChromeFrameMessageTask(&chrome_postman_thread_,
                                               message, target));
   } else {
     LOG(ERROR) << "Trying to post a message before the postman thread is"
                   "completely initialized and ready.";
   }
 }
 
-void ChromePostman::FireEvent(BSTR event_name, BSTR event_args) {
+void ChromePostman::FireEvent(const char* event_name, const char* event_args) {
   MessageLoop* message_loop = api_worker_thread_.message_loop();
   if (message_loop) {
     message_loop->PostTask(FROM_HERE,

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

while you're in here, can I ask you to remove the task subclass and change this
to use NewRunnableMethod? 
Also fine to add a TODO and address in a separate CL if you're getting fed up
with this one :)

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

Done.

                            new FireEventTask(event_name, event_args));
   } else {
     LOG(ERROR) << "Trying to post a message before the API worker thread is"
                   "completely initialized and ready.";
   }
 }
 
 HRESULT ChromePostman::OnCfReadyStateChanged(LONG state) {
   if (state == READYSTATE_COMPLETE) {
     // If the page is fully loaded, we reset the count to 0 to ensure we restart
@@ skipping 156 matching lines @@
 void ChromePostman::ApiInvocationWorkerThread::Init() {
   ::CoInitializeEx(0, COINIT_MULTITHREADED);
   ProductionApiDispatcher::get()->SetApiInvocationThreadId(
       ::GetCurrentThreadId());
 }
 
 void ChromePostman::ApiInvocationWorkerThread::CleanUp() {
   ::CoUninitialize();
   ProductionApiDispatcher::get()->SetApiInvocationThreadId(0);
 }