Revert of Avoid inline capacity of WordMeasurements

Revert of Avoid inline capacity of WordMeasurements (patchset #1 id:1 of https://codereview.chromium.org/2765353004/ )

Reason for revert:
Reverting due to 10.2%-11.9% regression in blink_perf.layout,
reported in issue 706907.

Original issue's description:
> Avoid inline capacity of WordMeasurements
> 
> This patch disables the inline capacity of WordMeasurements but to
> reserve the heap once per line, in order to avoid stack overflow in
> LayoutBlockFlow::layoutRunsAndFloatsInRange() when stack is critical,
> such as when V8 initiates the layout.
> 
> Currently, LayoutBlockFlow::layoutRunsAndFloatsInRange() is the only
> place we instantiate WordMeasurements.
> 
> This change is expected to save stack space by ~2k.
> 
> BUG=704536
> 
> Review-Url: https://codereview.chromium.org/2765353004
> Cr-Commit-Position: refs/heads/master@{#459485}
> Committed: https://chromium.googlesource.com/chromium/src/+/704ca0cfcc046d199d31e6502639bc94b89fddb0

TBR=eae@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=704536, 706907
Review-Url: https://codereview.chromium.org/2789913002
Cr-Commit-Position: refs/heads/master@{#463172}
Committed: https://chromium.googlesource.com/chromium/src/+/165039831ad0a01bcc90b66d023c9d90dcc964d1

[kojii, 2017-03-31 19:24:52]

The CQ bit was checked by kojii@chromium.org

[kojii, 2017-03-31 19:24:52]

Created Revert of Avoid inline capacity of WordMeasurements

[commit-bot: I haz the power, 2017-03-31 19:25:41]

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

[dcheng, 2017-03-31 20:02:43]

lgtm

[kojii, 2017-03-31 20:39:02]

The CQ bit was unchecked by kojii@chromium.org

[eae, 2017-04-03 22:39:18]

LGTM

[eae, 2017-04-03 22:39:37]

(not sure we want to do this yet but if we do this is the way to do it)

[kojii, 2017-04-10 03:05:31]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-04-10 03:05:44]

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-04-10 03:07:33]

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

[commit-bot: I haz the power, 2017-04-10 03:07:34]

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-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[kojii, 2017-04-10 03:22:18]

Description was changed from

==========
Revert of Avoid inline capacity of WordMeasurements (patchset #1 id:1 of
https://codereview.chromium.org/2765353004/ )

Reason for revert:
Reverting due to 10.2%-11.9% regression in blink_perf.layout,
reported in issue 706907.

Original issue's description:
> Avoid inline capacity of WordMeasurements
> 
> This patch disables the inline capacity of WordMeasurements but to
> reserve the heap once per line, in order to avoid stack overflow in
> LayoutBlockFlow::layoutRunsAndFloatsInRange() when stack is critical,
> such as when V8 initiates the layout.
> 
> Currently, LayoutBlockFlow::layoutRunsAndFloatsInRange() is the only
> place we instantiate WordMeasurements.
> 
> This change is expected to save stack space by ~2k.
> 
> BUG=704536
> 
> Review-Url: https://codereview.chromium.org/2765353004
> Cr-Commit-Position: refs/heads/master@{#459485}
> Committed:
https://chromium.googlesource.com/chromium/src/+/704ca0cfcc046d199d31e6502639...

TBR=eae@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=704536
==========

to

==========
Revert of Avoid inline capacity of WordMeasurements (patchset #1 id:1 of
https://codereview.chromium.org/2765353004/ )

Reason for revert:
Reverting due to 10.2%-11.9% regression in blink_perf.layout,
reported in issue 706907.

Original issue's description:
> Avoid inline capacity of WordMeasurements
> 
> This patch disables the inline capacity of WordMeasurements but to
> reserve the heap once per line, in order to avoid stack overflow in
> LayoutBlockFlow::layoutRunsAndFloatsInRange() when stack is critical,
> such as when V8 initiates the layout.
> 
> Currently, LayoutBlockFlow::layoutRunsAndFloatsInRange() is the only
> place we instantiate WordMeasurements.
> 
> This change is expected to save stack space by ~2k.
> 
> BUG=704536
> 
> Review-Url: https://codereview.chromium.org/2765353004
> Cr-Commit-Position: refs/heads/master@{#459485}
> Committed:
https://chromium.googlesource.com/chromium/src/+/704ca0cfcc046d199d31e6502639...

TBR=eae@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=704536, 706907
==========

[kojii, 2017-04-10 03:22:24]

The CQ bit was checked by kojii@chromium.org

[kojii, 2017-04-10 03:22:25]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2789913002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2017-04-10 03:22:43]

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-04-10 03:24:29]

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

[commit-bot: I haz the power, 2017-04-10 03:24:31]

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-...)

[kojii, 2017-04-10 04:10:43]

The CQ bit was checked by kojii@chromium.org

[kojii, 2017-04-10 04:10:43]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2789913002/#ps80001
(title: "Rebase after the big rename")

[commit-bot: I haz the power, 2017-04-10 04:10: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-04-10 05:29:08]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1491797443216110,
"parent_rev": "adc8910776c64a876f0422454da618a37d01167b", "commit_rev":
"165039831ad0a01bcc90b66d023c9d90dcc964d1"}

[commit-bot: I haz the power, 2017-04-10 05:30:46]

Description was changed from

==========
Revert of Avoid inline capacity of WordMeasurements (patchset #1 id:1 of
https://codereview.chromium.org/2765353004/ )

Reason for revert:
Reverting due to 10.2%-11.9% regression in blink_perf.layout,
reported in issue 706907.

Original issue's description:
> Avoid inline capacity of WordMeasurements
> 
> This patch disables the inline capacity of WordMeasurements but to
> reserve the heap once per line, in order to avoid stack overflow in
> LayoutBlockFlow::layoutRunsAndFloatsInRange() when stack is critical,
> such as when V8 initiates the layout.
> 
> Currently, LayoutBlockFlow::layoutRunsAndFloatsInRange() is the only
> place we instantiate WordMeasurements.
> 
> This change is expected to save stack space by ~2k.
> 
> BUG=704536
> 
> Review-Url: https://codereview.chromium.org/2765353004
> Cr-Commit-Position: refs/heads/master@{#459485}
> Committed:
https://chromium.googlesource.com/chromium/src/+/704ca0cfcc046d199d31e6502639...

TBR=eae@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=704536, 706907
==========

to

==========
Revert of Avoid inline capacity of WordMeasurements (patchset #1 id:1 of
https://codereview.chromium.org/2765353004/ )

Reason for revert:
Reverting due to 10.2%-11.9% regression in blink_perf.layout,
reported in issue 706907.

Original issue's description:
> Avoid inline capacity of WordMeasurements
> 
> This patch disables the inline capacity of WordMeasurements but to
> reserve the heap once per line, in order to avoid stack overflow in
> LayoutBlockFlow::layoutRunsAndFloatsInRange() when stack is critical,
> such as when V8 initiates the layout.
> 
> Currently, LayoutBlockFlow::layoutRunsAndFloatsInRange() is the only
> place we instantiate WordMeasurements.
> 
> This change is expected to save stack space by ~2k.
> 
> BUG=704536
> 
> Review-Url: https://codereview.chromium.org/2765353004
> Cr-Commit-Position: refs/heads/master@{#459485}
> Committed:
https://chromium.googlesource.com/chromium/src/+/704ca0cfcc046d199d31e6502639...

TBR=eae@chromium.org,dcheng@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=704536, 706907

Review-Url: https://codereview.chromium.org/2789913002
Cr-Commit-Position: refs/heads/master@{#463172}
Committed:
https://chromium.googlesource.com/chromium/src/+/165039831ad0a01bcc90b66d023c...
==========

[commit-bot: I haz the power, 2017-04-10 05:30:47]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/165039831ad0a01bcc90b66d023c...