Index: chromeos/ime/ibus_bridge.h |
diff --git a/chromeos/ime/ibus_bridge.h b/chromeos/ime/ibus_bridge.h |
index 57799ec1476bd5752cf4d24673446cbf55b998fa..2b6a3d3e58f51476f5fe545fe842379699bc178d 100644 |
--- a/chromeos/ime/ibus_bridge.h |
+++ b/chromeos/ime/ibus_bridge.h |
@@ -86,15 +86,23 @@ class CHROMEOS_EXPORT IBusEngineHandlerInterface { |
// Called when the IME is reset. |
virtual void Reset() = 0; |
- // Called when the key event is received. The |keycode| is raw layout |
- // independent keycode. The |keysym| is result of XLookupString function |
- // which translate |keycode| to keyboard layout dependent symbol value. |
- // Actual implementation must call |callback| after key event handling. |
- // For example: key press event for 'd' key on us layout and dvorak layout. |
+ // Called when the key event is received. The |keycode| is raw |
+ // layout independent keycode string. The |keysym| is a keyboard |
+ // layout dependent symbol string. Actual implementation must call |
+ // |callback| after key event handling. For example: key press |
+ // event for 'd' key on us layout and dvorak layout. |
// keyval keycode state |
- // us layout : 0x64 0x20 0x00 |
- // dvorak layout : 0x65 0x20 0x00 |
- virtual void ProcessKeyEvent(uint32 keysym, uint32 keycode, uint32 state, |
+ // us layout : "d" "KeyD" 0x00 |
+ // dvorak layout : "e" "KeyD" 0x00 |
+ // |
+ // TODO(komatsu): Use ui::KeyEvent after addressing the DEPS issue. |
+ virtual void ProcessKeyEvent(const std::string& keysym, |
+ const std::string& keycode, |
+ bool is_key_down, |
+ bool is_alt_down, |
+ bool is_ctrl_down, |
+ bool is_shift_down, |
+ bool is_caps_lock_down, |
const KeyEventDoneCallback& callback) = 0; |
// Called when the candidate in lookup table is clicked. The |index| is 0 |