Prune createVisibleSelectionDeprecated from svg

Prune createVisibleSelectionDeprecated from svg

SVGTextContentElement::selectSubString calls VisiblePosition::firstPositionInNode,
which updates layout and thus ensures clean layout for the later call of
createVisibleSelectionDeprecated. Hence, this patch safely prunes this call
of createVisibleSelectionDeprecated.

BUG=651373
Committed: https://crrev.com/666af71049d1b77a81ba9c15da70da466fdc6abd
Cr-Commit-Position: refs/heads/master@{#422389}

[Xiaocheng, 2016-10-03 05:53:04]

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

[commit-bot: I haz the power, 2016-10-03 05:53:13]

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-03 06:44:24]

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

[Xiaocheng, 2016-10-03 06:44:24]

PTAL.

[commit-bot: I haz the power, 2016-10-03 07:13:22]

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

[commit-bot: I haz the power, 2016-10-03 07:13:23]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-10-03 07:59:32]

Would you explain why we can prune it in the CL description please?

[Xiaocheng, 2016-10-03 08:05:34]

Description was changed from

==========
Prune createVisibleSelectionDeprecated from svg

BUG=651373
==========

to

==========
Prune createVisibleSelectionDeprecated from svg

SVGTextContentElement::selectSubString calls
VisiblePosition::firstPositionInNode,
which updates layout and thus ensures clean layout for the later call of
createVisibleSelectionDeprecated. Hence, this patch safely prunes this call
of createVisibleSelectionDeprecated.

BUG=651373
==========

[Xiaocheng, 2016-10-03 08:05:57]

On 2016/10/03 at 07:59:32, tkent wrote:
> Would you explain why we can prune it in the CL description please?

Done.

[tkent, 2016-10-03 08:08:49]

lgtm

[Xiaocheng, 2016-10-03 08:13:18]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2016-10-03 08:13:26]

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-03 08:18:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-03 08:20:45]

Description was changed from

==========
Prune createVisibleSelectionDeprecated from svg

SVGTextContentElement::selectSubString calls
VisiblePosition::firstPositionInNode,
which updates layout and thus ensures clean layout for the later call of
createVisibleSelectionDeprecated. Hence, this patch safely prunes this call
of createVisibleSelectionDeprecated.

BUG=651373
==========

to

==========
Prune createVisibleSelectionDeprecated from svg

SVGTextContentElement::selectSubString calls
VisiblePosition::firstPositionInNode,
which updates layout and thus ensures clean layout for the later call of
createVisibleSelectionDeprecated. Hence, this patch safely prunes this call
of createVisibleSelectionDeprecated.

BUG=651373

Committed: https://crrev.com/666af71049d1b77a81ba9c15da70da466fdc6abd
Cr-Commit-Position: refs/heads/master@{#422389}
==========

[commit-bot: I haz the power, 2016-10-03 08:20:46]

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