Commit 03127f38 authored by Rob Buis's avatar Rob Buis Committed by Commit Bot

[mathml] Simplify fraction centering calculation

Simplify fraction centering calculation by not computing
content_inline_size, since this is equal to child_available_size.inline_size.

Bug: 6606
Change-Id: If2c734f0ea644922d49e1527b1ac599df17bc7bd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2134300Reviewed-by: default avatarIan Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Rob Buis <rbuis@igalia.com>
Cr-Commit-Position: refs/heads/master@{#756357}
parent 2e0b5c5c
......@@ -192,10 +192,6 @@ scoped_refptr<const NGLayoutResult> NGMathFractionLayoutAlgorithm::Layout() {
ConstraintSpace().GetWritingMode(), ConstraintSpace().Direction(),
To<NGPhysicalBoxFragment>(denominator_layout_result->PhysicalFragment()));
LayoutUnit content_inline_size = std::max(
numerator_fragment.InlineSize() + numerator_margins.InlineSum(),
denominator_fragment.InlineSize() + denominator_margins.InlineSum());
LayoutUnit numerator_ascent =
numerator_margins.block_start +
numerator_fragment.Baseline().value_or(numerator_fragment.BlockSize());
......@@ -252,13 +248,13 @@ scoped_refptr<const NGLayoutResult> NGMathFractionLayoutAlgorithm::Layout() {
LogicalOffset denominator_offset;
numerator_offset.inline_offset =
border_scrollbar_padding_.inline_start + numerator_margins.inline_start +
(content_inline_size -
(child_available_size.inline_size -
(numerator_fragment.InlineSize() + numerator_margins.InlineSum())) /
2;
denominator_offset.inline_offset =
border_scrollbar_padding_.inline_start +
denominator_margins.inline_start +
(content_inline_size -
(child_available_size.inline_size -
(denominator_fragment.InlineSize() + denominator_margins.InlineSum())) /
2;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment