Fix Handle above IME issue

Fix Handle above IME issue

Added flag to check if IME state is updated
This will prevent resetting of mFocusPreOSKViewportRect
on onTouchEvent.

So, if cancelRequestToScrollFocusedEditableNodeIntoView()
is triggerred from onTouchEvent()<ACTION_MOVE> and it
happens before onSizeChanged is triggerred, then the
input field will not be scrolled to view.

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

[AviD, 2014-07-15 14:51:21]

PTAL

[jdduke (slow), 2014-07-15 15:36:58]

https://codereview.chromium.org/392943002/diff/1/content/public/android/java/...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
(right):

https://codereview.chromium.org/392943002/diff/1/content/public/android/java/...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:1135:
cancelRequestToScrollFocusedEditableNodeIntoView();
I think we can achieve the same result by simply moving this command below
|final int eventAction = event.getActionMasked();|, and then calling it like:

if (eventAction == MotionEvent.ACTION_DOWN) {
    cancelRequestToScrollFocusedEditableNodeIntoView();
}

While it might result in the scroll after longpress being interrupted by the
focus call, it's somewhat simpler than the additional logic bit.

[AviD, 2014-07-15 15:58:02]

Uploaded new patch.

PTAL

https://codereview.chromium.org/392943002/diff/1/content/public/android/java/...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
(right):

https://codereview.chromium.org/392943002/diff/1/content/public/android/java/...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:1135:
cancelRequestToScrollFocusedEditableNodeIntoView();
Agree. 
It was my initial thought, but wasn't sure of the side effects it might cause.
Will change to this.

[jdduke (slow), 2014-07-15 16:01:02]

lgtm

[AviD, 2014-07-15 16:01:23]

The CQ bit was checked by avi.nitk@samsung.com

[commit-bot: I haz the power, 2014-07-15 16:03:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/avi.nitk@samsung.com/392943002/20001

[commit-bot: I haz the power, 2014-07-16 00:58:30]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-16 01:06:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-16 01:06:39]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[AviD, 2014-07-16 02:24:57]

The CQ bit was checked by avi.nitk@samsung.com

[commit-bot: I haz the power, 2014-07-16 02:28:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/avi.nitk@samsung.com/392943002/20001

[commit-bot: I haz the power, 2014-07-16 02:42:00]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-16 02:46:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-16 02:46:08]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[AviD, 2014-07-16 05:13:04]

The CQ bit was checked by avi.nitk@samsung.com

[commit-bot: I haz the power, 2014-07-16 05:17:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/avi.nitk@samsung.com/392943002/40001

[commit-bot: I haz the power, 2014-07-16 11:39:12]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[commit-bot: I haz the power, 2014-07-16 11:58:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-16 11:58:44]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[AviD, 2014-07-16 15:07:12]

The CQ bit was checked by avi.nitk@samsung.com

[commit-bot: I haz the power, 2014-07-16 15:09:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/avi.nitk@samsung.com/392943002/40001

[commit-bot: I haz the power, 2014-07-16 18:19:25]

Change committed as 283434