Fix crash bug of IME extension API.

chrome/browser/chromeos/input_method/input_method_engine_ibus.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/chromeos/input_method/input_method_engine_ibus.h"
 
 #include <map>
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 37 matching lines @@
       aux_text_visible_(false),
       observer_(NULL),
       preedit_text_(new ibus::IBusText()),
       preedit_cursor_(0),
       component_(new ibus::IBusComponent()),
       table_(new ibus::IBusLookupTable()),
       table_visible_(false),
       weak_ptr_factory_(this) {
 }
 
+InputMethodEngineIBus::~InputMethodEngineIBus() {
+  GetCurrentService()->UnsetEngine();
+  input_method::InputMethodManager::GetInstance()->
+      RemoveInputMethodExtension(ibus_id_);
+}
+
 void InputMethodEngineIBus::Initialize(
     InputMethodEngine::Observer* observer,
     const char* engine_name,
     const char* extension_id,
     const char* engine_id,
     const char* description,
     const char* language,
     const std::vector<std::string>& layouts,
     std::string* error) {
   DCHECK(observer) << "Observer must not be null.";
@@ skipping 35 matching lines @@
 
   component_->mutable_engine_description()->push_back(engine_desc);
   manager->AddInputMethodExtension(ibus_id_, engine_name, layouts, language,
                                    this);
   // If connection is avaiable, register component. If there are no connection
   // to ibus-daemon, OnConnected callback will register component instead.
   if (IsConnected())
     RegisterComponent();
 }
 
-InputMethodEngineIBus::~InputMethodEngineIBus() {
-}
-
 bool InputMethodEngineIBus::SetComposition(
     int context_id,
     const char* text,
     int selection_start,
     int selection_end,
     int cursor,
     const std::vector<SegmentInfo>& segments,
     std::string* error) {
   if (!active_) {
     *error = kErrorNotActive;
@@ skipping 438 matching lines @@
   // TODO(nona): Implement error handling.
 }
 
 void InputMethodEngineIBus::CreateEngineHandler(
     const IBusEngineFactoryService::CreateEngineResponseSender& sender) {
   DBusThreadManager::Get()->RemoveIBusEngineService(object_path_);
 
   current_object_path_++;
   object_path_ = dbus::ObjectPath(kObjectPathPrefix +
                                   base::IntToString(current_object_path_));
-  GetCurrentService()->Initialize(this);
+  GetCurrentService()->SetEngine(this);
   sender.Run(object_path_);
 }
 
 }  // namespace chromeos