Don't flip SVG text coordinates

Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, flipped writing-mode doesn't apply to SVG text because it doesn't
have multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
R=pdr@chromium.org

Committed: https://crrev.com/9e370a37157d26aa866cab25ff0649f162967638
Cr-Commit-Position: refs/heads/master@{#434797}

[Xianzhu, 2016-11-27 04:07:43]

Description was changed from

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, writing-mode doesn't apply to SVG text because it doesn't have
multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
==========

to

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, writing-mode doesn't apply to SVG text because it doesn't have
multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2016-11-27 04:07:50]

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

[commit-bot: I haz the power, 2016-11-27 04:07:56]

Dry run: 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-11-27 05:45:39]

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

[commit-bot: I haz the power, 2016-11-27 05:45:40]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Xianzhu, 2016-11-27 17:55:28]

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

[commit-bot: I haz the power, 2016-11-27 17:55:37]

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

[Xianzhu, 2016-11-27 18:01:27]

wangxianzhu@chromium.org changed reviewers:
+ pdr@chromium.org

[commit-bot: I haz the power, 2016-11-27 20:42:00]

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

[commit-bot: I haz the power, 2016-11-27 20:42:01]

Dry run: This issue passed the CQ dry run.

[pdr., 2016-11-28 03:35:50]

On 2016/11/27 at 18:01:28, wangxianzhu wrote:
> 

LGTM

[Xianzhu, 2016-11-28 17:56:49]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2016-11-28 17:57:34]

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

[Xianzhu, 2016-11-28 18:05:44]

Description was changed from

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, writing-mode doesn't apply to SVG text because it doesn't have
multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, flipped writing-mode doesn't apply to SVG text because it doesn't
have multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-11-28 20:43:15]

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

[commit-bot: I haz the power, 2016-11-28 20:43:16]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)

[Xianzhu, 2016-11-28 21:40:20]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2016-11-28 21:41:06]

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

[Xianzhu, 2016-11-29 00:28:24]

Description was changed from

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, flipped writing-mode doesn't apply to SVG text because it doesn't
have multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, flipped writing-mode doesn't apply to SVG text because it doesn't
have multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
R=pdr@chromium.org
==========

[Xianzhu, 2016-11-29 00:28:26]

Committed patchset #2 (id:20001) to pending queue manually as
eab4fb729f5e3e59ce4777e5979edf0e9b9b32bd (presubmit successful).

[commit-bot: I haz the power, 2016-11-29 00:35:16]

Description was changed from

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, flipped writing-mode doesn't apply to SVG text because it doesn't
have multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
R=pdr@chromium.org
==========

to

==========
Don't flip SVG text coordinates

This is a partial revert of https://codereview.chromium.org/2524113002/
which is incomplete about SVG text coordinate flipping. We could make it
complete, but before we think everything clear, I'd like to revert the
part first. At least all existing and new text cases about vertical SVG
text pass with this CL.

(Putting aside that we reuse InlineBox/LayoutBlockFlow code for SVG
text, flipped writing-mode doesn't apply to SVG text because it doesn't
have multi-line block direction layout. Vertical-rl and vertical-lr are
actually equivalent for SVG text.)

BUG=666416
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
R=pdr@chromium.org

Committed: https://crrev.com/9e370a37157d26aa866cab25ff0649f162967638
Cr-Commit-Position: refs/heads/master@{#434797}
==========

[commit-bot: I haz the power, 2016-11-29 00:35:17]

Patchset 2 (id:??) landed as
https://crrev.com/9e370a37157d26aa866cab25ff0649f162967638
Cr-Commit-Position: refs/heads/master@{#434797}