diff options
| author | cjhopman@chromium.org <cjhopman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-13 01:10:35 +0000 |
|---|---|---|
| committer | cjhopman@chromium.org <cjhopman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-13 01:10:35 +0000 |
| commit | b5a35616ba4024bd2eaafe999c44f624b448e390 (patch) | |
| tree | 0a2f09825b7ed0c09fc5bd52e43c9cf902121315 /content | |
| parent | ebe123f9aeb87467c0cdf8145d8194811f1decc0 (diff) | |
| download | chromium_src-b5a35616ba4024bd2eaafe999c44f624b448e390.zip chromium_src-b5a35616ba4024bd2eaafe999c44f624b448e390.tar.gz chromium_src-b5a35616ba4024bd2eaafe999c44f624b448e390.tar.bz2 | |
Unselect text before rotating
When rotating, we receive events from the renderer in an unexpected
order (we get a SelectionBoundsChanged call with points in the new
window position before the CompositorFrameMessage and so we misinterpret
the SelectionBoundsChanged).
For now, we should simply unselect the text before rotating.
BUG=174665
Review URL: https://chromiumcodereview.appspot.com/12221150
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182074 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content')
| -rw-r--r-- | content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java index adba049..1e4046e 100644 --- a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java +++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java @@ -1300,6 +1300,12 @@ public class ContentViewCore implements MotionEventDelegate, NavigationClient { } if (mNeedUpdateOrientationChanged) { + // TODO(cjhopman): Once selection bounds are received from the renderer as absolute + // positions, we will no longer need to unselect before rotation (though we may decide + // it is the correct behavior). + // http://crbug.com/174665 + mImeAdapter.unselect(); + sendOrientationChangeEvent(); mNeedUpdateOrientationChanged = false; } |