Chromium Code Reviews| Index: chrome/browser/chromeos/input_method/input_method_engine_ibus.cc |
| diff --git a/chrome/browser/chromeos/input_method/input_method_engine_ibus.cc b/chrome/browser/chromeos/input_method/input_method_engine_ibus.cc |
| index 0e36d43325b661edfcd1096a05ab7735d4085d50..f8e17dac1754e99b08f2624160ccc38d52f588d6 100644 |
| --- a/chrome/browser/chromeos/input_method/input_method_engine_ibus.cc |
| +++ b/chrome/browser/chromeos/input_method/input_method_engine_ibus.cc |
| @@ -61,9 +61,13 @@ InputMethodEngineIBus::InputMethodEngineIBus() |
| } |
| InputMethodEngineIBus::~InputMethodEngineIBus() { |
| - if (object_path_.IsValid()) |
| - GetCurrentService()->UnsetEngine(); |
| input_method::GetInputMethodManager()->RemoveInputMethodExtension(ibus_id_); |
| + |
| + // Do not unset engine before removing input method extension, above function |
| + // may call reset function of engine object. |
| + // TODO(nona): Call Reset manually here and remove relevant code from |
| + // InputMethodManager once ibus-daemon is gone. |
|
satorux1
2013/04/23 03:28:30
Add a bug URL?
Seigo Nonaka
2013/04/23 07:13:35
Done.
|
| + GetCurrentService()->UnsetEngine(this); |
| } |
| void InputMethodEngineIBus::Initialize( |