[android] Adding a check not to send the same update to IME.

[android] Adding a check not to send the same update to IME.

Adding a check for the previously sent value to avoid sending updates
that are duplicates. This could happen when we have batch events.

BUG=172845
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=191142

[aurimas (slooooooooow), 2013-03-27 00:28:22]

Hey Yusuf,

Please take a look at this CL.

Aurimas

[Yusuf, 2013-03-27 18:10:47]

lgtm with nit
 

On 2013/03/27 00:28:22, aurimas wrote:
> Hey Yusuf,
> 
> Please take a look at this CL.
> 
> Aurimas

[Yusuf, 2013-03-27 18:10:53]

https://codereview.chromium.org/13105005/diff/1/content/public/android/java/s...
File
content/public/android/java/src/org/chromium/content/browser/ImeAdapter.java
(right):

https://codereview.chromium.org/13105005/diff/1/content/public/android/java/s...
content/public/android/java/src/org/chromium/content/browser/ImeAdapter.java:559:
mLastUpdateCompositionEnd == compositionEnd)
You need curly braces for this.

[aurimas (slooooooooow), 2013-03-27 18:49:59]

https://codereview.chromium.org/13105005/diff/1/content/public/android/java/s...
File
content/public/android/java/src/org/chromium/content/browser/ImeAdapter.java
(right):

https://codereview.chromium.org/13105005/diff/1/content/public/android/java/s...
content/public/android/java/src/org/chromium/content/browser/ImeAdapter.java:559:
mLastUpdateCompositionEnd == compositionEnd)
On 2013/03/27 18:10:53, Yusuf wrote:
> You need curly braces for this.

Done.

[aurimas (slooooooooow), 2013-03-27 20:58:52]

Hey Nilesh,

Could you take a look at this CL?

Thanks!
Aurimas

[nilesh, 2013-03-27 22:36:01]

LGTM

[commit-bot: I haz the power, 2013-03-28 02:29:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/aurimas@chromium.org/13105005/4003

[commit-bot: I haz the power, 2013-03-28 13:01:25]

Change committed as 191142