diff options
| author | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-04 06:35:01 +0000 |
|---|---|---|
| committer | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-04 06:35:01 +0000 |
| commit | c82e4519d03a75c6568394b8c3b49de04c53ab0d (patch) | |
| tree | c5f467ffa1c06ce5d950be1d1e60429395ef66ec /chrome/browser/ime_input.h | |
| parent | 1283ba4fe600d3c8cb32b494cbba96fb1eb106e4 (diff) | |
| download | chromium_src-c82e4519d03a75c6568394b8c3b49de04c53ab0d.zip chromium_src-c82e4519d03a75c6568394b8c3b49de04c53ab0d.tar.gz chromium_src-c82e4519d03a75c6568394b8c3b49de04c53ab0d.tar.bz2 | |
A fix for Issue 2768 "IME: Current clause of Japanese IME doesn't appear to reflect its rage change."
This issue is caused by my code that cannot extract a target clause from a composition string when there is an input clause after a target clause.
when a user changes the range of a target clause, Japanese IMEs creates a composite clause which consists of a new target clause and an input clause. My IME code treats such composite clause as a target clause and cannot reflect the change.
To fix this problem, this code changes the algorithm which extracts a target clause.
Review URL: http://codereview.chromium.org/8970
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4569 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ime_input.h')
| -rw-r--r-- | chrome/browser/ime_input.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/chrome/browser/ime_input.h b/chrome/browser/ime_input.h index 1cf5344..4064f02 100644 --- a/chrome/browser/ime_input.h +++ b/chrome/browser/ime_input.h @@ -236,8 +236,7 @@ class ImeInput { // Determines whether or not the given attribute represents a target // (a.k.a. a selection). bool IsTargetAttribute(char attribute) const { - return (attribute == ATTR_INPUT || - attribute == ATTR_TARGET_CONVERTED || + return (attribute == ATTR_TARGET_CONVERTED || attribute == ATTR_TARGET_NOTCONVERTED); } |