Revert of Fix endBatchEdit() state update order issue

Revert of Fix endBatchEdit() state update order issue (patchset #1 id:1 of https://codereview.chromium.org/2462583004/ )

Reason for revert:
As pointed out in https://bugs.chromium.org/p/chromium/issues/detail?id=659934#c23, keyboard may not show up when it has to with this patch.

Original issue's description:
> Fix endBatchEdit() state update order issue
>
> If we call endBatchEdit() and then call getExtractedText() immediately,
> then the order gets reversed.
>
> The reason is that getExtractedText() is already block-waiting for a new
> state, and endBatchEdit() was piggy-backing on FROM_IME to enforce a
> state update.
>
> The more natural change source for a state update at the end of a batch
> edit is actually FROM_NON_IME, which does not interfere with the
> block-wait mechanism of get* methods. The state update order in the above
> scenario is fixed because blockAndGetStateUpdate() handles it correctly
> when it's FROM_NON_IME.
>
> Also, now we bounce state update in RenderWidget instead of doing so in
> ThreadedInputConnection. This is another way to make sure that we get
> state update at the end of batch edit: the intermediate state update was
> already bounced off.
>
> Now there is no need to check isInBatchEdit() inside
> ThreadedInputConnection, and ThreadedInputConnectionTest needs to be
> updated, but let me handle them in a separate CL.
>
> BUG=659934
>
> Committed: https://crrev.com/72381b3189537b4e3c7e2dbff8a2ca398bd3b447
> Cr-Commit-Position: refs/heads/master@{#428602}

TBR=aelias@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659934
Committed: https://crrev.com/872c3c42beb58bbf5b6a2995db0be4fa14f67000
Cr-Commit-Position: refs/heads/master@{#428612}

[Changwan Ryu, 2016-10-29 16:31:52]

The CQ bit was checked by changwan@chromium.org

[Changwan Ryu, 2016-10-29 16:31:52]

Created Revert of Fix endBatchEdit() state update order issue

[commit-bot: I haz the power, 2016-10-29 16:32:02]

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

[commit-bot: I haz the power, 2016-10-29 16:32:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-29 17:01:13]

Description was changed from

==========
Revert of Fix endBatchEdit() state update order issue (patchset #1 id:1 of
https://codereview.chromium.org/2462583004/ )

Reason for revert:
As pointed out in
https://bugs.chromium.org/p/chromium/issues/detail?id=659934#c23, keyboard may
not show up when it has to with this patch.

Original issue's description:
> Fix endBatchEdit() state update order issue
> 
> If we call endBatchEdit() and then call getExtractedText() immediately,
> then the order gets reversed.
> 
> The reason is that getExtractedText() is already block-waiting for a new
> state, and endBatchEdit() was piggy-backing on FROM_IME to enforce a
> state update.
> 
> The more natural change source for a state update at the end of a batch
> edit is actually FROM_NON_IME, which does not interfere with the
> block-wait mechanism of get* methods. The state update order in the above
> scenario is fixed because blockAndGetStateUpdate() handles it correctly
> when it's FROM_NON_IME.
> 
> Also, now we bounce state update in RenderWidget instead of doing so in
> ThreadedInputConnection. This is another way to make sure that we get
> state update at the end of batch edit: the intermediate state update was
> already bounced off.
> 
> Now there is no need to check isInBatchEdit() inside
> ThreadedInputConnection, and ThreadedInputConnectionTest needs to be
> updated, but let me handle them in a separate CL.
> 
> BUG=659934
> 
> Committed: https://crrev.com/72381b3189537b4e3c7e2dbff8a2ca398bd3b447
> Cr-Commit-Position: refs/heads/master@{#428602}

TBR=aelias@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659934
==========

to

==========
Revert of Fix endBatchEdit() state update order issue (patchset #1 id:1 of
https://codereview.chromium.org/2462583004/ )

Reason for revert:
As pointed out in
https://bugs.chromium.org/p/chromium/issues/detail?id=659934#c23, keyboard may
not show up when it has to with this patch.

Original issue's description:
> Fix endBatchEdit() state update order issue
>
> If we call endBatchEdit() and then call getExtractedText() immediately,
> then the order gets reversed.
>
> The reason is that getExtractedText() is already block-waiting for a new
> state, and endBatchEdit() was piggy-backing on FROM_IME to enforce a
> state update.
>
> The more natural change source for a state update at the end of a batch
> edit is actually FROM_NON_IME, which does not interfere with the
> block-wait mechanism of get* methods. The state update order in the above
> scenario is fixed because blockAndGetStateUpdate() handles it correctly
> when it's FROM_NON_IME.
>
> Also, now we bounce state update in RenderWidget instead of doing so in
> ThreadedInputConnection. This is another way to make sure that we get
> state update at the end of batch edit: the intermediate state update was
> already bounced off.
>
> Now there is no need to check isInBatchEdit() inside
> ThreadedInputConnection, and ThreadedInputConnectionTest needs to be
> updated, but let me handle them in a separate CL.
>
> BUG=659934
>
> Committed: https://crrev.com/72381b3189537b4e3c7e2dbff8a2ca398bd3b447
> Cr-Commit-Position: refs/heads/master@{#428602}

TBR=aelias@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=659934

Committed: https://crrev.com/872c3c42beb58bbf5b6a2995db0be4fa14f67000
Cr-Commit-Position: refs/heads/master@{#428612}
==========

[commit-bot: I haz the power, 2016-10-29 17:01:14]

Patchset 1 (id:??) landed as
https://crrev.com/872c3c42beb58bbf5b6a2995db0be4fa14f67000
Cr-Commit-Position: refs/heads/master@{#428612}