Revert of Remove InputMethodEventHandler.

Revert of Remove InputMethodEventHandler. (patchset #16 id:300001 of https://codereview.chromium.org/2872343003/ )

Reason for revert:
This CL makes TalkBack in Android window unusable. The issue is that after this CL, key event becomes to go through event rewriter in Chrome. TalkBack doesn't expect that events are rewritten by Chrome.

Original issue's description:
> Remove InputMethodEventHandler.
>
> This change removes InputMethodEventHandler and instead sends the events
> to IME in WindowEventDispatcher::PreDispatchEvent().
>
>
> This is a step towards unifying key event processing path in classic ash and
> mus+ash.
>
> BUG=719269
>
> Review-Url: https://codereview.chromium.org/2872343003
> Cr-Commit-Position: refs/heads/master@{#472941}
> Committed: https://chromium.googlesource.com/chromium/src/+/9f9dddf904a9eae86f20d0fa3f68c1881a360d80

TBR=sadrul@chromium.org,sky@chromium.org,reillyg@chromium.org,moshayedi@google.com,moshayedi@chromium.org
BUG=719269, 727179, 726642

[Hadi, 2017-05-29 14:23:39]

The CQ bit was checked by moshayedi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-29 14:23:57]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Hadi, 2017-05-29 14:24:16]

Description was changed from

==========
Revert of Remove InputMethodEventHandler. (patchset #16 id:300001 of
https://codereview.chromium.org/2872343003/ )

Reason for revert:
This CL makes TalkBack in Android window unusable. The issue is that after this
CL, key event becomes to go through event rewriter in Chrome. TalkBack doesn't
expect that events are rewritten by Chrome.

Original issue's description:
> Remove InputMethodEventHandler.
>
> This change removes InputMethodEventHandler and instead sends the events
> to IME in WindowEventDispatcher::PreDispatchEvent().
>
>
> This is a step towards unifying key event processing path in classic ash and
> mus+ash.
>
> BUG=719269
>
> Review-Url: https://codereview.chromium.org/2872343003
> Cr-Commit-Position: refs/heads/master@{#472941}
> Committed:
https://chromium.googlesource.com/chromium/src/+/9f9dddf904a9eae86f20d0fa3f68...

TBR=sadrul@chromium.org,sky@chromium.org,reillyg@chromium.org,moshayedi@googl...
BUG=719269
==========

to

==========
Revert of Remove InputMethodEventHandler. (patchset #16 id:300001 of
https://codereview.chromium.org/2872343003/ )

Reason for revert:
This CL makes TalkBack in Android window unusable. The issue is that after this
CL, key event becomes to go through event rewriter in Chrome. TalkBack doesn't
expect that events are rewritten by Chrome.

Original issue's description:
> Remove InputMethodEventHandler.
>
> This change removes InputMethodEventHandler and instead sends the events
> to IME in WindowEventDispatcher::PreDispatchEvent().
>
>
> This is a step towards unifying key event processing path in classic ash and
> mus+ash.
>
> BUG=719269
>
> Review-Url: https://codereview.chromium.org/2872343003
> Cr-Commit-Position: refs/heads/master@{#472941}
> Committed:
https://chromium.googlesource.com/chromium/src/+/9f9dddf904a9eae86f20d0fa3f68...

TBR=sadrul@chromium.org,sky@chromium.org,reillyg@chromium.org,moshayedi@googl...
BUG=719269
==========

[Hadi, 2017-05-29 14:24:16]

moshayedi@chromium.org changed reviewers:
+ yawano@chromium.org
- moshayedi@chromium.org, moshayedi@google.com

[Hadi, 2017-05-29 14:42:43]

Description was changed from

==========
Revert of Remove InputMethodEventHandler. (patchset #16 id:300001 of
https://codereview.chromium.org/2872343003/ )

Reason for revert:
This CL makes TalkBack in Android window unusable. The issue is that after this
CL, key event becomes to go through event rewriter in Chrome. TalkBack doesn't
expect that events are rewritten by Chrome.

Original issue's description:
> Remove InputMethodEventHandler.
>
> This change removes InputMethodEventHandler and instead sends the events
> to IME in WindowEventDispatcher::PreDispatchEvent().
>
>
> This is a step towards unifying key event processing path in classic ash and
> mus+ash.
>
> BUG=719269
>
> Review-Url: https://codereview.chromium.org/2872343003
> Cr-Commit-Position: refs/heads/master@{#472941}
> Committed:
https://chromium.googlesource.com/chromium/src/+/9f9dddf904a9eae86f20d0fa3f68...

TBR=sadrul@chromium.org,sky@chromium.org,reillyg@chromium.org,moshayedi@googl...
BUG=719269
==========

to

==========
Revert of Remove InputMethodEventHandler. (patchset #16 id:300001 of
https://codereview.chromium.org/2872343003/ )

Reason for revert:
This CL makes TalkBack in Android window unusable. The issue is that after this
CL, key event becomes to go through event rewriter in Chrome. TalkBack doesn't
expect that events are rewritten by Chrome.

Original issue's description:
> Remove InputMethodEventHandler.
>
> This change removes InputMethodEventHandler and instead sends the events
> to IME in WindowEventDispatcher::PreDispatchEvent().
>
>
> This is a step towards unifying key event processing path in classic ash and
> mus+ash.
>
> BUG=719269
>
> Review-Url: https://codereview.chromium.org/2872343003
> Cr-Commit-Position: refs/heads/master@{#472941}
> Committed:
https://chromium.googlesource.com/chromium/src/+/9f9dddf904a9eae86f20d0fa3f68...

TBR=sadrul@chromium.org,sky@chromium.org,reillyg@chromium.org,moshayedi@googl...
BUG=719269,727179,726642
==========

[Hadi, 2017-05-29 14:43:07]

PTAL.

[commit-bot: I haz the power, 2017-05-29 15:51:11]

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

[commit-bot: I haz the power, 2017-05-29 15:51:12]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[sky, 2017-05-30 16:35:10]

Rubber stamp LGTM

[Reilly Grant (use Gerrit), 2017-05-30 17:14:06]

rs extensions lgtm

[Hadi, 2017-05-31 19:36:53]

I am fixing the original issue at https://codereview.chromium.org/2916673002, so
closing this one without landing. I'll try to land the other CL once I write
some tests for it.

Thanks.

[David Tseng, 2017-06-01 22:06:27]

On 2017/05/31 19:36:53, Hadi wrote:
> I am fixing the original issue at https://codereview.chromium.org/2916673002,
so
> closing this one without landing. I'll try to land the other CL once I write
> some tests for it.
> 
> Thanks.

Could you revert this in m60? This is not just a Talkback issue, but also a
ChromeVox issue. It's actually worse than that as all event rewriters get
duplicate key down's.

[Hadi, 2017-06-01 22:41:05]

The CQ bit was checked by moshayedi@chromium.org

[commit-bot: I haz the power, 2017-06-01 22:42:19]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Hadi, 2017-06-01 22:47:53]

The CQ bit was unchecked by moshayedi@chromium.org