Mark calls of visible{Start,End} with dirty layout deprecated

Mark calls of visible{Start,End} with dirty layout deprecated

This patch marks the current VisibleSelection::visible{Start,End}
deprecated because they updates layout synchronously, and introduces
the new desired versions that requires clean layout.

This patch also ensures that call of these two functions from outside
editing commands are done with clean layout, and make the calls in
editing commands switch to the deprecated versions.

BUG=647219
Committed: https://crrev.com/57eca1337e14e60d21b126d828c6c5e2845cd620
Cr-Commit-Position: refs/heads/master@{#421413}

[Xiaocheng, 2016-09-27 11:23:47]

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

[commit-bot: I haz the power, 2016-09-27 11:24:01]

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

[Xiaocheng, 2016-09-27 11:29:50]

Description was changed from

==========
Mark calls of visible{Start,End} with dirty layout deprecated

BUG=647219
==========

to

==========
Mark calls of visible{Start,End} with dirty layout deprecated

This patch marks the current VisibleSelection::visible{Start,End}
deprecated because they updates layout synchronously, and introduces
the new desired versions that requires clean layout.

This patch also ensures that call of these two functions from outside
editing commands are done with clean layout, and make the calls in
editing commands switch to the deprecated versions.

BUG=647219
==========

[Xiaocheng, 2016-09-27 12:27:56]

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

[Xiaocheng, 2016-09-27 12:27:57]

PTAL.

It doesn't appear to be one month's work :)

[commit-bot: I haz the power, 2016-09-27 12:46:33]

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

[commit-bot: I haz the power, 2016-09-27 12:46:34]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-09-28 01:10:48]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2016-09-28 01:10:48]

lgtm

[commit-bot: I haz the power, 2016-09-28 01:11:21]

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

[yosin_UTC9, 2016-09-28 01:11:48]

On 2016/09/27 at 12:27:57, xiaochengh wrote:
> PTAL.
> 
> It doesn't appear to be one month's work :)

Did you count getting rid of visible{Start,End}Deprecated? (^_^;)

[commit-bot: I haz the power, 2016-09-28 01:17:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-28 01:20:53]

Description was changed from

==========
Mark calls of visible{Start,End} with dirty layout deprecated

This patch marks the current VisibleSelection::visible{Start,End}
deprecated because they updates layout synchronously, and introduces
the new desired versions that requires clean layout.

This patch also ensures that call of these two functions from outside
editing commands are done with clean layout, and make the calls in
editing commands switch to the deprecated versions.

BUG=647219
==========

to

==========
Mark calls of visible{Start,End} with dirty layout deprecated

This patch marks the current VisibleSelection::visible{Start,End}
deprecated because they updates layout synchronously, and introduces
the new desired versions that requires clean layout.

This patch also ensures that call of these two functions from outside
editing commands are done with clean layout, and make the calls in
editing commands switch to the deprecated versions.

BUG=647219

Committed: https://crrev.com/57eca1337e14e60d21b126d828c6c5e2845cd620
Cr-Commit-Position: refs/heads/master@{#421413}
==========

[commit-bot: I haz the power, 2016-09-28 01:20:54]

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