Clear the selection and insertion flags only upon getting the selection event

Clear the selection and insertion flags only upon getting the selection event

The mHasSelection and mHasInsertion flags are being cleared even before the
selection events are received and hence the popup is never hidden.
dismissTextHandles already triggers the selection events to clear the handles.
Hence we dont need to clear them outside the selection event handler.

BUG=520060
Committed: https://crrev.com/0f7316c4d8eeb601502549f7890aa8f34a873875
Cr-Commit-Position: refs/heads/master@{#343212}

[raghu, 2015-08-12 13:26:09]

r.ghatage@samsung.com changed reviewers:
+ ajith.v@chromium.org

[AKV, 2015-08-12 13:44:43]

minor change pointed.

https://codereview.chromium.org/1283193002/diff/1/content/public/android/java...
File
content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java
(right):

https://codereview.chromium.org/1283193002/diff/1/content/public/android/java...
content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java:944:
setComposingText("h", 1);
It would be good to check the insertion status after this line.

[AKV, 2015-08-13 05:50:56]

peer review looks good to me!

[raghu, 2015-08-13 05:56:41]

r.ghatage@samsung.com changed reviewers:
+ jdduke@chromium.org

[raghu, 2015-08-13 05:56:42]

jdduke@,
PTAL

[jdduke (slow), 2015-08-13 15:24:16]

lgtm, thanks.

[jdduke (slow), 2015-08-13 15:24:29]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2015-08-13 15:24:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1283193002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1283193002/20001

[commit-bot: I haz the power, 2015-08-13 16:32:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-13 16:33:08]

Patchset 2 (id:??) landed as
https://crrev.com/0f7316c4d8eeb601502549f7890aa8f34a873875
Cr-Commit-Position: refs/heads/master@{#343212}