Revert of [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand| (3/3)

Revert of [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand| (3/3) (patchset #1 id:20001 of https://codereview.chromium.org/2579253002/ )

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

Original issue's description:
> [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand| (3/3)
> 
> This is the last sub-patch (3/3) of supporting |EditCommandSource| in
> |CompositeEditCommand| (https://crrev.com/2574793002).
> 
> Instead of adding |EditCommandSource| to constructor, this CL:
> 1. Pass source to |CompositeEditCommand::apply(EditCommandSource)|
> 2. Pass source to various static methods of |TypingCommand|
> 
> We don't have to store command source as attribute because we only need it to fire
> 'beforeinput' in the above locations.
> This approach also helped avoiding modifying all the |CompositeEditCommand|
> constructors.
> 
> 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=673789
> 
> Committed: https://crrev.com/acce62cb77947bf2f14d6c7ac74219d515a94ffb
> Cr-Commit-Position: refs/heads/master@{#439130}

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

[chongz, 2017-01-16 21:11:53]

Created Revert of [EditCommandSource] Pass source to |CompositEditCommand| and
|TypingCommand| (3/3)

[chongz, 2017-01-16 21:12:12]

Description was changed from

==========
Revert of [EditCommandSource] Pass source to |CompositEditCommand| and
|TypingCommand| (3/3) (patchset #1 id:20001 of
https://codereview.chromium.org/2579253002/ )

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

Original issue's description:
> [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand|
(3/3)
> 
> This is the last sub-patch (3/3) of supporting |EditCommandSource| in
> |CompositeEditCommand| (https://crrev.com/2574793002).
> 
> Instead of adding |EditCommandSource| to constructor, this CL:
> 1. Pass source to |CompositeEditCommand::apply(EditCommandSource)|
> 2. Pass source to various static methods of |TypingCommand|
> 
> We don't have to store command source as attribute because we only need it to
fire
> 'beforeinput' in the above locations.
> This approach also helped avoiding modifying all the |CompositeEditCommand|
> constructors.
> 
> 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=673789
> 
> Committed: https://crrev.com/acce62cb77947bf2f14d6c7ac74219d515a94ffb
> Cr-Commit-Position: refs/heads/master@{#439130}

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

to

==========
Revert of [EditCommandSource] Pass source to |CompositEditCommand| and
|TypingCommand| (3/3) (patchset #1 id:20001 of
https://codereview.chromium.org/2579253002/ )

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

Original issue's description:
> [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand|
(3/3)
> 
> This is the last sub-patch (3/3) of supporting |EditCommandSource| in
> |CompositeEditCommand| (https://crrev.com/2574793002).
> 
> Instead of adding |EditCommandSource| to constructor, this CL:
> 1. Pass source to |CompositeEditCommand::apply(EditCommandSource)|
> 2. Pass source to various static methods of |TypingCommand|
> 
> We don't have to store command source as attribute because we only need it to
fire
> 'beforeinput' in the above locations.
> This approach also helped avoiding modifying all the |CompositeEditCommand|
> constructors.
> 
> 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=673789
> 
> Committed: https://crrev.com/acce62cb77947bf2f14d6c7ac74219d515a94ffb
> Cr-Commit-Position: refs/heads/master@{#439130}

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

[chongz, 2017-01-16 21:12:25]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2017-01-16 21:12:34]

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:29:01]

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

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

Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[chongz, 2017-01-16 22:11:41]

The CQ bit was checked by chongz@chromium.org

[commit-bot: I haz the power, 2017-01-16 22:11:53]

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

[tkent, 2017-01-16 22:53:53]

lgtm

[commit-bot: I haz the power, 2017-01-16 23:28:39]

CQ is committing da patch.

Bot data: {"patchset_id": 270001, "attempt_start_ts": 1484604701512900,
"parent_rev": "550463c33bbb52a169b191a967d52afd438240e0", "commit_rev":
"e2dafbf4760e826f18b0c16b3976a667fcacf8a0"}

[commit-bot: I haz the power, 2017-01-16 23:29:06]

Description was changed from

==========
Revert of [EditCommandSource] Pass source to |CompositEditCommand| and
|TypingCommand| (3/3) (patchset #1 id:20001 of
https://codereview.chromium.org/2579253002/ )

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

Original issue's description:
> [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand|
(3/3)
> 
> This is the last sub-patch (3/3) of supporting |EditCommandSource| in
> |CompositeEditCommand| (https://crrev.com/2574793002).
> 
> Instead of adding |EditCommandSource| to constructor, this CL:
> 1. Pass source to |CompositeEditCommand::apply(EditCommandSource)|
> 2. Pass source to various static methods of |TypingCommand|
> 
> We don't have to store command source as attribute because we only need it to
fire
> 'beforeinput' in the above locations.
> This approach also helped avoiding modifying all the |CompositeEditCommand|
> constructors.
> 
> 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=673789
> 
> Committed: https://crrev.com/acce62cb77947bf2f14d6c7ac74219d515a94ffb
> Cr-Commit-Position: refs/heads/master@{#439130}

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

to

==========
Revert of [EditCommandSource] Pass source to |CompositEditCommand| and
|TypingCommand| (3/3) (patchset #1 id:20001 of
https://codereview.chromium.org/2579253002/ )

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

Original issue's description:
> [EditCommandSource] Pass source to |CompositEditCommand| and |TypingCommand|
(3/3)
> 
> This is the last sub-patch (3/3) of supporting |EditCommandSource| in
> |CompositeEditCommand| (https://crrev.com/2574793002).
> 
> Instead of adding |EditCommandSource| to constructor, this CL:
> 1. Pass source to |CompositeEditCommand::apply(EditCommandSource)|
> 2. Pass source to various static methods of |TypingCommand|
> 
> We don't have to store command source as attribute because we only need it to
fire
> 'beforeinput' in the above locations.
> This approach also helped avoiding modifying all the |CompositeEditCommand|
> constructors.
> 
> 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=673789
> 
> Committed: https://crrev.com/acce62cb77947bf2f14d6c7ac74219d515a94ffb
> Cr-Commit-Position: refs/heads/master@{#439130}

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

Review-Url: https://codereview.chromium.org/2636043002
Cr-Commit-Position: refs/heads/master@{#443960}
Committed:
https://chromium.googlesource.com/chromium/src/+/e2dafbf4760e826f18b0c16b3976...
==========

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

Committed patchset #2 (id:270001) as
https://chromium.googlesource.com/chromium/src/+/e2dafbf4760e826f18b0c16b3976...