[LayoutNG] Implement LayoutBR/preserved newlines and tabs

[LayoutNG] Implement LayoutBR/preserved newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
Review-Url: https://codereview.chromium.org/2888643004
Cr-Commit-Position: refs/heads/master@{#474573}
Committed: https://chromium.googlesource.com/chromium/src/+/6f0296f4db2b403980ea251a5c720c99d7261e6c

[kojii, 2017-05-17 06:02:41]

Description was changed from

==========
ngctrl

BUG=
==========

to

==========
ngctrl

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[kojii, 2017-05-17 20:04:30]

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

[commit-bot: I haz the power, 2017-05-17 20:05:59]

Dry run: CQ is trying da patch.

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

[kojii, 2017-05-17 20:07:39]

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

[kojii, 2017-05-17 20:07:46]

Patchset #3 (id:40001) has been deleted

[commit-bot: I haz the power, 2017-05-17 20:07:56]

Dry run: CQ is trying da patch.

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

[kojii, 2017-05-17 20:18:20]

Description was changed from

==========
ngctrl

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Implement LayoutBR/newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[kojii, 2017-05-17 20:19:02]

Description was changed from

==========
[LayoutNG] Implement LayoutBR/newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Implement LayoutBR/newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[kojii, 2017-05-17 20:19:33]

kojii@chromium.org changed reviewers:
+ eae@chromium.org

[kojii, 2017-05-17 20:19:33]

PTAL.

[kojii, 2017-05-17 20:20:23]

Description was changed from

==========
[LayoutNG] Implement LayoutBR/newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Implement LayoutBR/preserved newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[commit-bot: I haz the power, 2017-05-17 22:11:31]

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

[commit-bot: I haz the power, 2017-05-17 22:11:32]

Dry run: Try jobs failed on following builders:
  linux_layout_tests_layout_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build has not started yet; builder either lacks capacity or does
not exist (misspelled?))

[kojii, 2017-05-18 15:19:02]

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

[commit-bot: I haz the power, 2017-05-18 15:20:00]

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, 2017-05-18 16:47:59]

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

[commit-bot: I haz the power, 2017-05-18 16:48:00]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[kojii, 2017-05-20 15:09:33]

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

[commit-bot: I haz the power, 2017-05-20 15:09:43]

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, 2017-05-20 15:11:34]

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

[commit-bot: I haz the power, 2017-05-20 15:11:35]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[kojii, 2017-05-20 15:45:02]

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

[commit-bot: I haz the power, 2017-05-20 15:45:15]

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, 2017-05-20 18:56:32]

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

[commit-bot: I haz the power, 2017-05-20 18:56:33]

Dry run: This issue passed the CQ dry run.

[eae, 2017-05-24 16:44:47]

LGTM!

[kojii, 2017-05-24 16:46:27]

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

[commit-bot: I haz the power, 2017-05-24 16:47:03]

Dry run: CQ is trying da patch.

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

[kojii, 2017-05-24 16:47:30]

Patchset #7 (id:140001) has been deleted

[kojii, 2017-05-24 16:47:49]

The CQ bit was unchecked by kojii@chromium.org

[kojii, 2017-05-24 16:48:13]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-05-24 16:48:57]

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-05-25 00:55:09]

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

[commit-bot: I haz the power, 2017-05-25 00:55:11]

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

[kojii, 2017-05-25 05:45:46]

Description was changed from

==========
[LayoutNG] Implement LayoutBR/preserved newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Implement LayoutBR/preserved newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
==========

[kojii, 2017-05-25 05:45:50]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-05-25 05:46:12]

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-05-25 05:51:08]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1495691150460550,
"parent_rev": "68c8fa131af2f78c5fd930b8469c8ab985d63c04", "commit_rev":
"6f0296f4db2b403980ea251a5c720c99d7261e6c"}

[commit-bot: I haz the power, 2017-05-25 05:51:22]

Description was changed from

==========
[LayoutNG] Implement LayoutBR/preserved newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993
==========

to

==========
[LayoutNG] Implement LayoutBR/preserved newlines and tabs

This patch implements laying out preserved newlines and tabs.

With the control NGInlineItem[1], LayoutBR is also supported.

6 existing tests pass, but 3 of them dumps text, and the text fails to
match because innerText ignores blank lines, probably because LayoutNG
creates InlineBoxes differently. This is to be investigated.

[1] https://codereview.chromium.org/2870413005

BUG=636993

Review-Url: https://codereview.chromium.org/2888643004
Cr-Commit-Position: refs/heads/master@{#474573}
Committed:
https://chromium.googlesource.com/chromium/src/+/6f0296f4db2b403980ea251a5c72...
==========

[commit-bot: I haz the power, 2017-05-25 05:51:23]

Committed patchset #6 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/6f0296f4db2b403980ea251a5c72...