Index: ui/base/ime/input_method_auralinux_unittest.cc |
diff --git a/ui/base/ime/input_method_auralinux_unittest.cc b/ui/base/ime/input_method_auralinux_unittest.cc |
index 456b372bf8643b9cbd0384919f7f3b6c64211d31..5eed1a9c83989467535e7d425d730f91a510ea35 100644 |
--- a/ui/base/ime/input_method_auralinux_unittest.cc |
+++ b/ui/base/ime/input_method_auralinux_unittest.cc |
@@ -153,9 +153,10 @@ class InputMethodDelegateForTesting : public internal::InputMethodDelegate { |
InputMethodDelegateForTesting(){}; |
~InputMethodDelegateForTesting() override{}; |
- bool DispatchKeyEventPostIME(const ui::KeyEvent& key_event) override { |
+ ui::EventDispatchDetails DispatchKeyEventPostIME( |
+ ui::KeyEvent* key_event) override { |
std::string action; |
- switch (key_event.type()) { |
+ switch (key_event->type()) { |
case ET_KEY_PRESSED: |
action = "keydown:"; |
break; |
@@ -166,10 +167,10 @@ class InputMethodDelegateForTesting : public internal::InputMethodDelegate { |
break; |
} |
std::stringstream ss; |
- ss << key_event.key_code(); |
+ ss << key_event->key_code(); |
action += std::string(ss.str()); |
TestResult::GetInstance()->RecordAction(base::ASCIIToUTF16(action)); |
- return false; |
+ return ui::EventDispatchDetails(); |
} |
private: |
@@ -292,7 +293,7 @@ TEST_F(InputMethodAuraLinuxTest, BasicSyncModeTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:65"); |
test_result_->ExpectAction("keypress:97"); |
@@ -305,7 +306,7 @@ TEST_F(InputMethodAuraLinuxTest, BasicSyncModeTest) { |
input_method_auralinux_->SetFocusedTextInputClient(client.get()); |
input_method_auralinux_->OnTextInputTypeChanged(client.get()); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:65"); |
test_result_->ExpectAction("keypress:97"); |
@@ -322,7 +323,7 @@ TEST_F(InputMethodAuraLinuxTest, BasicAsyncModeTest) { |
input_method_auralinux_->OnTextInputTypeChanged(client.get()); |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
input_method_auralinux_->OnCommit(base::ASCIIToUTF16("a")); |
test_result_->ExpectAction("keydown:229"); |
@@ -336,7 +337,7 @@ TEST_F(InputMethodAuraLinuxTest, BasicAsyncModeTest) { |
input_method_auralinux_->SetFocusedTextInputClient(client.get()); |
input_method_auralinux_->OnTextInputTypeChanged(client.get()); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:65"); |
test_result_->ExpectAction("keypress:97"); |
@@ -353,7 +354,7 @@ TEST_F(InputMethodAuraLinuxTest, IBusUSTest) { |
input_method_auralinux_->OnTextInputTypeChanged(client.get()); |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
// IBus mutes the key down. |
test_result_->Verify(); |
@@ -361,7 +362,7 @@ TEST_F(InputMethodAuraLinuxTest, IBusUSTest) { |
// IBus simulates a faked key down and handle it in sync mode. |
context_->SetSyncMode(true); |
context_->AddCommitAction("a"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:65"); |
test_result_->ExpectAction("keypress:97"); |
@@ -369,8 +370,8 @@ TEST_F(InputMethodAuraLinuxTest, IBusUSTest) { |
// IBus does NOT handle the key up. |
context_->SetEatKey(false); |
- input_method_auralinux_->DispatchKeyEvent( |
- KeyEvent(ET_KEY_RELEASED, VKEY_A, 0)); |
+ KeyEvent key_up(ET_KEY_RELEASED, VKEY_A, 0); |
+ input_method_auralinux_->DispatchKeyEvent(&key_up); |
test_result_->ExpectAction("keyup:65"); |
test_result_->Verify(); |
@@ -386,7 +387,7 @@ TEST_F(InputMethodAuraLinuxTest, IBusPinyinTest) { |
input_method_auralinux_->OnTextInputTypeChanged(client.get()); |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
// IBus issues a standalone set_composition action. |
input_method_auralinux_->OnPreeditStart(); |
@@ -400,8 +401,8 @@ TEST_F(InputMethodAuraLinuxTest, IBusPinyinTest) { |
test_result_->Verify(); |
// IBus issues a commit text with composition after muting the space key down. |
- input_method_auralinux_->DispatchKeyEvent( |
- KeyEvent(ET_KEY_PRESSED, VKEY_SPACE, 0)); |
+ KeyEvent key_up(ET_KEY_RELEASED, VKEY_SPACE, 0); |
+ input_method_auralinux_->DispatchKeyEvent(&key_up); |
input_method_auralinux_->OnPreeditEnd(); |
input_method_auralinux_->OnCommit(base::ASCIIToUTF16("A")); |
@@ -425,7 +426,7 @@ TEST_F(InputMethodAuraLinuxTest, DeadKeyTest) { |
KeyEvent dead_key(ET_KEY_PRESSED, VKEY_OEM_7, 0); |
dead_key.set_character(L'\''); |
- input_method_auralinux_->DispatchKeyEvent(dead_key); |
+ input_method_auralinux_->DispatchKeyEvent(&dead_key); |
// The single quote key is muted. |
test_result_->ExpectAction("keydown:222"); |
@@ -434,7 +435,7 @@ TEST_F(InputMethodAuraLinuxTest, DeadKeyTest) { |
context_simple_->AddCommitAction("X"); |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
// The following A key generates the accent key: รก. |
test_result_->ExpectAction("keydown:65"); |
@@ -456,7 +457,7 @@ TEST_F(InputMethodAuraLinuxTest, MultiCommitsTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("textinput:abc"); |
@@ -479,7 +480,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedCompositionAndCommitTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("textinput:ac"); |
@@ -488,7 +489,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedCompositionAndCommitTest) { |
test_result_->Verify(); |
context_->AddCommitAction("e"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionend"); |
@@ -509,7 +510,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionEndWithoutCommitTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionstart"); |
@@ -517,7 +518,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionEndWithoutCommitTest) { |
test_result_->Verify(); |
context_->AddCompositionEndAction(); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionend"); |
@@ -537,7 +538,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionEndWithEmptyCommitTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionstart"); |
@@ -546,7 +547,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionEndWithEmptyCommitTest) { |
context_->AddCompositionEndAction(); |
context_->AddCommitAction(""); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionend"); |
@@ -566,7 +567,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionEndWithCommitTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionstart"); |
@@ -575,7 +576,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionEndWithCommitTest) { |
context_->AddCompositionEndAction(); |
context_->AddCommitAction("b"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
// Verifies single char commit under composition mode will call InsertText |
// intead of InsertChar. |
@@ -599,7 +600,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionUpdateWithCommitTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("textinput:b"); |
@@ -612,7 +613,7 @@ TEST_F(InputMethodAuraLinuxTest, CompositionUpdateWithCommitTest) { |
context_->SetEatKey(true); |
context_->AddCommitAction("c"); |
context_->AddCompositionUpdateAction(""); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionend"); |
@@ -631,7 +632,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedAsyncAndSyncTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
CompositionText comp; |
comp.text = base::ASCIIToUTF16("a"); |
input_method_auralinux_->OnPreeditChanged(comp); |
@@ -645,7 +646,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedAsyncAndSyncTest) { |
context_->AddCompositionEndAction(); |
context_->AddCommitAction("b"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionend"); |
@@ -666,7 +667,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedSyncAndAsyncTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'a'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionstart"); |
@@ -675,7 +676,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedSyncAndAsyncTest) { |
context_->SetSyncMode(false); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
input_method_auralinux_->OnCommit(base::ASCIIToUTF16("b")); |
test_result_->ExpectAction("keydown:229"); |
@@ -685,7 +686,7 @@ TEST_F(InputMethodAuraLinuxTest, MixedSyncAndAsyncTest) { |
context_->SetSyncMode(true); |
context_->AddCommitAction("c"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:65"); |
test_result_->ExpectAction("keypress:99"); |
@@ -704,7 +705,7 @@ TEST_F(InputMethodAuraLinuxTest, ReleaseKeyTest) { |
KeyEvent key(ET_KEY_PRESSED, VKEY_A, 0); |
key.set_character(L'A'); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("keydown:229"); |
test_result_->ExpectAction("compositionstart"); |
@@ -713,7 +714,7 @@ TEST_F(InputMethodAuraLinuxTest, ReleaseKeyTest) { |
context_->SetEatKey(false); |
context_->AddCommitAction("b"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("compositionend"); |
test_result_->ExpectAction("textinput:b"); |
@@ -722,7 +723,7 @@ TEST_F(InputMethodAuraLinuxTest, ReleaseKeyTest) { |
test_result_->Verify(); |
context_->AddCommitAction("c"); |
- input_method_auralinux_->DispatchKeyEvent(key); |
+ input_method_auralinux_->DispatchKeyEvent(&key); |
test_result_->ExpectAction("textinput:c"); |
test_result_->ExpectAction("keydown:65"); |