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

Unified Diff: ui/base/ime/chromeos/character_composer.cc

Issue 1284433002: Revise ui::DomKey to unify character and non-character codes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 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
Index: ui/base/ime/chromeos/character_composer.cc
diff --git a/ui/base/ime/chromeos/character_composer.cc b/ui/base/ime/chromeos/character_composer.cc
index 20b27ef9fb75e55b947fa0274ea04718de8da0fe..327bd4f48e504435a19bead771fdec8b4f9faff8 100644
--- a/ui/base/ime/chromeos/character_composer.cc
+++ b/ui/base/ime/chromeos/character_composer.cc
@@ -110,8 +110,7 @@ bool CharacterComposer::FilterKeyPress(const ui::KeyEvent& event) {
bool CharacterComposer::FilterKeyPressSequenceMode(const KeyEvent& event) {
DCHECK(composition_mode_ == KEY_SEQUENCE_MODE);
- compose_buffer_.push_back(
- KeystrokeMeaning(event.GetDomKey(), event.GetCharacter()));
+ compose_buffer_.push_back(event.GetDomKey());
// Check compose table.
uint32 composed_character_utf32 = 0;
@@ -214,17 +213,18 @@ ComposeChecker::CheckSequenceResult TreeComposeChecker::CheckSequence(
DCHECK(tree_index < data_.tree_entries);
// If we are looking up a dead key, skip over the character tables.
- if (keystroke.key == ui::DomKey::DEAD) {
+ if (keystroke.IsDead()) {
tree_index += 2 * data_.tree[tree_index] + 1; // internal unicode table
tree_index += 2 * data_.tree[tree_index] + 1; // leaf unicode table
- } else if (keystroke.key != ui::DomKey::CHARACTER) {
+ } else if (!keystroke.IsUnicode()) {
return CheckSequenceResult::NO_MATCH;
}
// Check the internal subtree table.
uint16_t result = 0;
uint16_t entries = data_.tree[tree_index++];
- if (entries && Find(tree_index, entries, keystroke.character, &result)) {
+ if (entries &&
+ Find(tree_index, entries, keystroke.GetCodePoint(), &result)) {
tree_index = result;
continue;
}
@@ -232,7 +232,8 @@ ComposeChecker::CheckSequenceResult TreeComposeChecker::CheckSequence(
// Skip over the internal subtree table and check the leaf table.
tree_index += 2 * entries;
entries = data_.tree[tree_index++];
- if (entries && Find(tree_index, entries, keystroke.character, &result)) {
+ if (entries &&
+ Find(tree_index, entries, keystroke.GetCodePoint(), &result)) {
*composed_character = result;
return CheckSequenceResult::FULL_MATCH;
}

Powered by Google App Engine
This is Rietveld 408576698