commit | 9a5b5110105fca8cdf216bfcad69c495d139ad64 | [log] [tgz] |
---|---|---|
author | Fredrik Söderquist <fs@opera.com> | Thu Mar 14 13:15:09 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Mar 14 13:15:09 2019 |
tree | 30e586fd74c0ca731f50c1a9a4bba37c29793daa | |
parent | 001de7a75d0d08e4ccec191b1e5f2871e5f29abd [diff] |
Use the enclosing rect in SVGRootInlineBox::LayoutInlineBoxes The LayoutRect(const FloatRect&) constructor will round (floor in practice) the location/size independently. This can cause the resulting to be smaller than the argument - when the values a small (< 1) this effect is easily amplified. In cases like the one in this bug, where the bounding box of <text> is used to determine bounds for a <clipPath>, this can result in content being clipped when it shouldn't be. Use EnclosingLayoutRect instead of the "straight" conversion, to make sure the resulting bounds are not smaller than the original. Since this changes the bounding box of text, tests that depend on that directly or indirectly are affected. Mostly without noticeable visual differences. For svg/animations/animate-text-nested-transforms.html the location of the click (that starts the test) is nudged a bit to avoid timing out. Bug: 820955 Change-Id: I8811de81de0a5dc795218dcdf8a55a79b15e2863 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1520596 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#640717}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .