Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for (1/3)

Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for 'beforeinput' (1/3) (patchset #2 id:40001 of https://codereview.chromium.org/2583993002/ )

Reason for revert:
We want to revert all CLs and start over the refactor process, see:
https://crbug.com/673789.

Original issue's description:
> [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for 'beforeinput' (1/3)
> 
> This is the first sub-patch (1/3) of unifying 'beforeinput' logic. (Original CL
> https://crrev.com/2558643003)
> 
> This CL:
> 1. Added attribute |m_inputType| to |TypingCommand|
> 2. Added calling path |CompositeEditCommand::apply()| =>
>     *|CompositeEditCommand::willApply()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 3. Added calling path |static TypingCommand::deleteSelection()/insertText()/...| =>
>     *|TypingCommand::willAddTypingToOpenCommand()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 
> * Marks new methods.
> 
> The next step is to move 'beforeinput' logic to
> |CompositeEditCommand::willApplyEditing()|.
> 
> This CL shouldn't have any behavior change.
> 
> To help reviewing, here is a simple doc describing the following plans:
> https://docs.google.com/a/chromium.org/document/d/1M9bQaLKdRpR1qJ9i7zZKN-hKwd4ZkgJs5GPde2ZsDy0/edit?usp=sharing
> 
> BUG=670035
> 
> Committed: https://crrev.com/4d0f52c1162dae46a7902527775f2a95fc972423
> Cr-Commit-Position: refs/heads/master@{#439633}

TBR=tkent@chromium.org,xiaochengh@chromium.org,yosin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=670035
Review-Url: https://codereview.chromium.org/2634633002
Cr-Commit-Position: refs/heads/master@{#443942}
Committed: https://chromium.googlesource.com/chromium/src/+/8fa239362956ce98ce4bd9f3b540050231a33179

[chongz, 2017-01-13 20:58:51]

Created Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()|
in prepare for 'beforeinput' (1/3)

[chongz, 2017-01-13 20:59:33]

Description was changed from

==========
Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in
prepare for 'beforeinput' (1/3) (patchset #2 id:40001 of
https://codereview.chromium.org/2583993002/ )

Reason for revert:
We want to revert all CLs and start over the refactor process, see:
https://crbug.com/673789.

Original issue's description:
> [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for
'beforeinput' (1/3)
> 
> This is the first sub-patch (1/3) of unifying 'beforeinput' logic. (Original
CL
> https://crrev.com/2558643003)
> 
> This CL:
> 1. Added attribute |m_inputType| to |TypingCommand|
> 2. Added calling path |CompositeEditCommand::apply()| =>
>     *|CompositeEditCommand::willApply()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 3. Added calling path |static
TypingCommand::deleteSelection()/insertText()/...| =>
>     *|TypingCommand::willAddTypingToOpenCommand()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 
> * Marks new methods.
> 
> The next step is to move 'beforeinput' logic to
> |CompositeEditCommand::willApplyEditing()|.
> 
> This CL shouldn't have any behavior change.
> 
> To help reviewing, here is a simple doc describing the following plans:
>
https://docs.google.com/a/chromium.org/document/d/1M9bQaLKdRpR1qJ9i7zZKN-hKwd...
> 
> BUG=670035
> 
> Committed: https://crrev.com/4d0f52c1162dae46a7902527775f2a95fc972423
> Cr-Commit-Position: refs/heads/master@{#439633}

TBR=tkent@chromium.org,xiaochengh@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=670035
==========

to

==========
Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in
prepare for 'beforeinput' (1/3) (patchset #2 id:40001 of
https://codereview.chromium.org/2583993002/ )

Reason for revert:
We want to revert all CLs and start over the refactor process, see:
https://crbug.com/673789.

Original issue's description:
> [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for
'beforeinput' (1/3)
> 
> This is the first sub-patch (1/3) of unifying 'beforeinput' logic. (Original
CL
> https://crrev.com/2558643003)
> 
> This CL:
> 1. Added attribute |m_inputType| to |TypingCommand|
> 2. Added calling path |CompositeEditCommand::apply()| =>
>     *|CompositeEditCommand::willApply()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 3. Added calling path |static
TypingCommand::deleteSelection()/insertText()/...| =>
>     *|TypingCommand::willAddTypingToOpenCommand()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 
> * Marks new methods.
> 
> The next step is to move 'beforeinput' logic to
> |CompositeEditCommand::willApplyEditing()|.
> 
> This CL shouldn't have any behavior change.
> 
> To help reviewing, here is a simple doc describing the following plans:
>
https://docs.google.com/a/chromium.org/document/d/1M9bQaLKdRpR1qJ9i7zZKN-hKwd...
> 
> BUG=670035
> 
> Committed: https://crrev.com/4d0f52c1162dae46a7902527775f2a95fc972423
> Cr-Commit-Position: refs/heads/master@{#439633}

TBR=tkent@chromium.org,xiaochengh@chromium.org,yosin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=670035
==========

[chongz, 2017-01-13 20:59:44]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2017-01-13 21:00:51]

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, 2017-01-13 21:03:17]

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

[commit-bot: I haz the power, 2017-01-13 21:03:18]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[chongz, 2017-01-16 19:39:58]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2017-01-16 19:40:08]

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, 2017-01-16 21:06:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484595598043590, "parent_rev":
"399369caffeee0144fd8fe38afc1d8790129a5c0", "commit_rev":
"8fa239362956ce98ce4bd9f3b540050231a33179"}

[commit-bot: I haz the power, 2017-01-16 21:07:29]

Description was changed from

==========
Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in
prepare for 'beforeinput' (1/3) (patchset #2 id:40001 of
https://codereview.chromium.org/2583993002/ )

Reason for revert:
We want to revert all CLs and start over the refactor process, see:
https://crbug.com/673789.

Original issue's description:
> [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for
'beforeinput' (1/3)
> 
> This is the first sub-patch (1/3) of unifying 'beforeinput' logic. (Original
CL
> https://crrev.com/2558643003)
> 
> This CL:
> 1. Added attribute |m_inputType| to |TypingCommand|
> 2. Added calling path |CompositeEditCommand::apply()| =>
>     *|CompositeEditCommand::willApply()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 3. Added calling path |static
TypingCommand::deleteSelection()/insertText()/...| =>
>     *|TypingCommand::willAddTypingToOpenCommand()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 
> * Marks new methods.
> 
> The next step is to move 'beforeinput' logic to
> |CompositeEditCommand::willApplyEditing()|.
> 
> This CL shouldn't have any behavior change.
> 
> To help reviewing, here is a simple doc describing the following plans:
>
https://docs.google.com/a/chromium.org/document/d/1M9bQaLKdRpR1qJ9i7zZKN-hKwd...
> 
> BUG=670035
> 
> Committed: https://crrev.com/4d0f52c1162dae46a7902527775f2a95fc972423
> Cr-Commit-Position: refs/heads/master@{#439633}

TBR=tkent@chromium.org,xiaochengh@chromium.org,yosin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=670035
==========

to

==========
Revert of [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in
prepare for 'beforeinput' (1/3) (patchset #2 id:40001 of
https://codereview.chromium.org/2583993002/ )

Reason for revert:
We want to revert all CLs and start over the refactor process, see:
https://crbug.com/673789.

Original issue's description:
> [Editing] Introduce |CompositeEditCommand::willApplyEditing()| in prepare for
'beforeinput' (1/3)
> 
> This is the first sub-patch (1/3) of unifying 'beforeinput' logic. (Original
CL
> https://crrev.com/2558643003)
> 
> This CL:
> 1. Added attribute |m_inputType| to |TypingCommand|
> 2. Added calling path |CompositeEditCommand::apply()| =>
>     *|CompositeEditCommand::willApply()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 3. Added calling path |static
TypingCommand::deleteSelection()/insertText()/...| =>
>     *|TypingCommand::willAddTypingToOpenCommand()| =>
>     *|CompositeEditCommand::willApplyEditing()|
> 
> * Marks new methods.
> 
> The next step is to move 'beforeinput' logic to
> |CompositeEditCommand::willApplyEditing()|.
> 
> This CL shouldn't have any behavior change.
> 
> To help reviewing, here is a simple doc describing the following plans:
>
https://docs.google.com/a/chromium.org/document/d/1M9bQaLKdRpR1qJ9i7zZKN-hKwd...
> 
> BUG=670035
> 
> Committed: https://crrev.com/4d0f52c1162dae46a7902527775f2a95fc972423
> Cr-Commit-Position: refs/heads/master@{#439633}

TBR=tkent@chromium.org,xiaochengh@chromium.org,yosin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=670035

Review-Url: https://codereview.chromium.org/2634633002
Cr-Commit-Position: refs/heads/master@{#443942}
Committed:
https://chromium.googlesource.com/chromium/src/+/8fa239362956ce98ce4bd9f3b540...
==========

[commit-bot: I haz the power, 2017-01-16 21:07:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8fa239362956ce98ce4bd9f3b540...