Prune most of the deprecated functions from CompositeEditCommand

Prune most of the deprecated functions from CompositeEditCommand

BUG=647219, 651373
Committed: https://crrev.com/cc8c3088a0fb49c131d9a6e472d8020946f1a161
Cr-Commit-Position: refs/heads/master@{#424648}

[Xiaocheng, 2016-10-11 09:01:30]

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

[commit-bot: I haz the power, 2016-10-11 09:01:43]

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

[Xiaocheng, 2016-10-11 10:11:02]

xiaochengh@chromium.org changed reviewers:
+ yosin@chromium.org

[Xiaocheng, 2016-10-11 10:11:02]

PTAL.

[commit-bot: I haz the power, 2016-10-11 10:57:42]

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

[commit-bot: I haz the power, 2016-10-11 10:57:42]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-10-12 02:10:53]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2016-10-12 02:10:53]

lgtm

[commit-bot: I haz the power, 2016-10-12 02:11:08]

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

[yosin_UTC9, 2016-10-12 02:12:06]

https://codereview.chromium.org/2410813002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
(right):

https://codereview.chromium.org/2410813002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp:860:
VisiblePosition visibleUpstreamPos =
BTW, should we replace |VisiblePosition| to |const VisiblePositon&|?

[Xiaocheng, 2016-10-12 02:15:55]

https://codereview.chromium.org/2410813002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
(right):

https://codereview.chromium.org/2410813002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp:860:
VisiblePosition visibleUpstreamPos =
On 2016/10/12 at 02:12:06, Yosi_UTC9 wrote:
> BTW, should we replace |VisiblePosition| to |const VisiblePositon&|?

Do you mean a global s/VisiblePosition/const VisiblePosition&/?

[commit-bot: I haz the power, 2016-10-12 02:16:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-12 02:19:42]

Description was changed from

==========
Prune most of the deprecated functions from CompositeEditCommand

BUG=647219,651373
==========

to

==========
Prune most of the deprecated functions from CompositeEditCommand

BUG=647219,651373

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

[commit-bot: I haz the power, 2016-10-12 02:19:42]

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