Revert of Change InputMethodController#setSelectionOffsets() to use NotUserTriggered parameter

Revert of Change InputMethodController#setSelectionOffsets() to use  NotUserTriggered parameter (patchset #3 id:40001 of https://codereview.chromium.org/2206923002/ )

Reason for revert:
ImeTest#testFinishComposingText failed.
We need to fix the test before relanding it.

Original issue's description:
> Change InputMethodController#setSelectionOffsets() to use
> NotUserTriggered parameter
>
> InputMethodController#setSelectionOffsets() should have no side
> effect other than changing the selection. But
> FrameSelection::CloseTyping will close typing, causing the failure of
> canceling the composing text when inserting line break. We should
> use NotUserTriggered parameter because it has no side effect.
>
> BUG=633840
>
> Committed: https://crrev.com/786070a33bcfebaff78b304401a4137e8dadccd8
> Cr-Commit-Position: refs/heads/master@{#410577}

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

[yabinh, 2016-08-10 00:47:50]

The CQ bit was checked by yabinh@chromium.org

[yabinh, 2016-08-10 00:47:50]

Created Revert of Change InputMethodController#setSelectionOffsets() to use 
NotUserTriggered parameter

[commit-bot: I haz the power, 2016-08-10 00:48:58]

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-08-10 00:48:58]

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

[commit-bot: I haz the power, 2016-08-10 00:48:59]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Changwan Ryu, 2016-08-10 00:49:47]

On 2016/08/10 00:48:59, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. 
> CQ run can only be started by full committers or once the patch has
> received an L-G-T-M from a full committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

lgtm

[Changwan Ryu, 2016-08-10 00:50:00]

The CQ bit was checked by changwan@chromium.org

[commit-bot: I haz the power, 2016-08-10 00:50:36]

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-08-10 00:52:44]

Description was changed from

==========
Revert of Change InputMethodController#setSelectionOffsets() to use 
NotUserTriggered parameter (patchset #3 id:40001 of
https://codereview.chromium.org/2206923002/ )

Reason for revert:
ImeTest#testFinishComposingText failed.
We need to fix the test before relanding it.

Original issue's description:
> Change InputMethodController#setSelectionOffsets() to use
> NotUserTriggered parameter
> 
> InputMethodController#setSelectionOffsets() should have no side
> effect other than changing the selection. But
> FrameSelection::CloseTyping will close typing, causing the failure of
> canceling the composing text when inserting line break. We should
> use NotUserTriggered parameter because it has no side effect.
> 
> BUG=633840
> 
> Committed: https://crrev.com/786070a33bcfebaff78b304401a4137e8dadccd8
> Cr-Commit-Position: refs/heads/master@{#410577}

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

to

==========
Revert of Change InputMethodController#setSelectionOffsets() to use 
NotUserTriggered parameter (patchset #3 id:40001 of
https://codereview.chromium.org/2206923002/ )

Reason for revert:
ImeTest#testFinishComposingText failed.
We need to fix the test before relanding it.

Original issue's description:
> Change InputMethodController#setSelectionOffsets() to use
> NotUserTriggered parameter
> 
> InputMethodController#setSelectionOffsets() should have no side
> effect other than changing the selection. But
> FrameSelection::CloseTyping will close typing, causing the failure of
> canceling the composing text when inserting line break. We should
> use NotUserTriggered parameter because it has no side effect.
> 
> BUG=633840
> 
> Committed: https://crrev.com/786070a33bcfebaff78b304401a4137e8dadccd8
> Cr-Commit-Position: refs/heads/master@{#410577}

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

[commit-bot: I haz the power, 2016-08-10 00:52:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 00:56:09]

Description was changed from

==========
Revert of Change InputMethodController#setSelectionOffsets() to use 
NotUserTriggered parameter (patchset #3 id:40001 of
https://codereview.chromium.org/2206923002/ )

Reason for revert:
ImeTest#testFinishComposingText failed.
We need to fix the test before relanding it.

Original issue's description:
> Change InputMethodController#setSelectionOffsets() to use
> NotUserTriggered parameter
> 
> InputMethodController#setSelectionOffsets() should have no side
> effect other than changing the selection. But
> FrameSelection::CloseTyping will close typing, causing the failure of
> canceling the composing text when inserting line break. We should
> use NotUserTriggered parameter because it has no side effect.
> 
> BUG=633840
> 
> Committed: https://crrev.com/786070a33bcfebaff78b304401a4137e8dadccd8
> Cr-Commit-Position: refs/heads/master@{#410577}

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

to

==========
Revert of Change InputMethodController#setSelectionOffsets() to use 
NotUserTriggered parameter (patchset #3 id:40001 of
https://codereview.chromium.org/2206923002/ )

Reason for revert:
ImeTest#testFinishComposingText failed.
We need to fix the test before relanding it.

Original issue's description:
> Change InputMethodController#setSelectionOffsets() to use
> NotUserTriggered parameter
>
> InputMethodController#setSelectionOffsets() should have no side
> effect other than changing the selection. But
> FrameSelection::CloseTyping will close typing, causing the failure of
> canceling the composing text when inserting line break. We should
> use NotUserTriggered parameter because it has no side effect.
>
> BUG=633840
>
> Committed: https://crrev.com/786070a33bcfebaff78b304401a4137e8dadccd8
> Cr-Commit-Position: refs/heads/master@{#410577}

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

Committed: https://crrev.com/dd122a409bd085efd4a566ea950755bdd60d2866
Cr-Commit-Position: refs/heads/master@{#410905}
==========

[commit-bot: I haz the power, 2016-08-10 00:56:09]

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