Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/base/ime/input_method_auralinux_unittest.cc

Issue 1257603006: Refactoring for the InputMethod & InputMethodDelegate interfaces. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix test failures on windows. Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« ui/base/ime/input_method_auralinux.cc ('K') | « ui/base/ime/input_method_auralinux.cc ('k') | ui/base/ime/input_method_base.h » ('j') | ui/base/ime/input_method_chromeos.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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");
« ui/base/ime/input_method_auralinux.cc ('K') | « ui/base/ime/input_method_auralinux.cc ('k') | ui/base/ime/input_method_base.h » ('j') | ui/base/ime/input_method_chromeos.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698