Limit optimized LayoutBlockFlowLine width computation to 8bit

Limit optimized LayoutBlockFlowLine width computation to 8bit

Limit the use of the optimized LayoutBlockFlowLine width implementation
to 8-bit text as it isn't guaranteed to work correctly for 16-bit text.

BUG=692014
R=kojii@chromium.org

Review-Url: https://codereview.chromium.org/2706033009
Cr-Commit-Position: refs/heads/master@{#456271}
Committed: https://chromium.googlesource.com/chromium/src/+/8683b364d376b33bed4cbfe6b46d2c39411b05cd

[eae, 2017-02-21 23:34:06]

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

[commit-bot: I haz the power, 2017-02-21 23:35:23]

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-02-22 01:40:20]

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

[commit-bot: I haz the power, 2017-02-22 01:40:21]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)

[kojii, 2017-02-22 05:05:36]

lgtm

[kojii, 2017-02-22 05:06:05]

looks like some rebaselines needed.

[eae, 2017-03-08 07:08:42]

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

[commit-bot: I haz the power, 2017-03-08 07:09:08]

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

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

[commit-bot: I haz the power, 2017-03-08 08:14:26]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[eae, 2017-03-11 02:08:18]

The CQ bit was checked by eae@chromium.org

[eae, 2017-03-11 02:08:18]

The patchset sent to the CQ was uploaded after l-g-t-m from kojii@chromium.org
Link to the patchset: https://codereview.chromium.org/2706033009/#ps40001
(title: "w/Baselines")

[commit-bot: I haz the power, 2017-03-11 02:08:37]

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-03-11 03:28:18]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1489198098294910,
"parent_rev": "ac0b0fc3924e2586a072f8343376a1f295b3786a", "commit_rev":
"8683b364d376b33bed4cbfe6b46d2c39411b05cd"}

[commit-bot: I haz the power, 2017-03-11 03:28:59]

Description was changed from

==========
Limit optimized LayoutBlockFlowLine width computation to 8bit

Limit the use of the optimized LayoutBlockFlowLine width implementation
to 8-bit text as it isn't guaranteed to work correctly for 16-bit text.

BUG=692014
R=kojii@chromium.org
==========

to

==========
Limit optimized LayoutBlockFlowLine width computation to 8bit

Limit the use of the optimized LayoutBlockFlowLine width implementation
to 8-bit text as it isn't guaranteed to work correctly for 16-bit text.

BUG=692014
R=kojii@chromium.org

Review-Url: https://codereview.chromium.org/2706033009
Cr-Commit-Position: refs/heads/master@{#456271}
Committed:
https://chromium.googlesource.com/chromium/src/+/8683b364d376b33bed4cbfe6b46d...
==========

[commit-bot: I haz the power, 2017-03-11 03:29:00]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/8683b364d376b33bed4cbfe6b46d...

[klausw, 2017-03-11 06:06:12]

On 2017/03/11 03:29:00, commit-bot: I haz the power wrote:
> Committed patchset #3 (id:40001) as
>
https://chromium.googlesource.com/chromium/src/+/8683b364d376b33bed4cbfe6b46d...

FYI, I think this broke WebKit layout tests for Mac10.12:

https://luci-milo.appspot.com/buildbot/chromium.webkit/WebKit%20Mac10.12/91

unexpected_failures:
fast/writing-mode/japanese-rl-selection.html
fast/writing-mode/border-vertical-lr.html
fast/writing-mode/japanese-rl-text.html
fast/writing-mode/Kusa-Makura-background-canvas.html
fast/writing-mode/japanese-lr-text.html
fast/writing-mode/japanese-lr-selection.html

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Mac10_12/...

(I was keeping an eye on the queue due to my change being in the same batch.)