commit | e9b36c2c7c55eeafcea1f71ee1217dff00248ede | [log] [tgz] |
---|---|---|
author | Koji Ishii <kojii@chromium.org> | Tue Oct 09 19:57:40 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Tue Oct 09 19:57:40 2018 |
tree | 64b4bfc5e513ff3965a5fcd199fa62708ea7679c | |
parent | c22429999a552e9400ce5fc66a08c2ac38a31cec [diff] |
[LayoutNG] Check box states in context matches to rebuilt one From the review comment in r597459 (crrev.com/c/1258810), this patch adds more complete (but not all) check for the box states in NGInlineChildLayoutContext matches to the one NGInlineLayoutAlgorithm rebuilds from scratch. * One minor failure for |has_start_edge| is fixed in this CL. The diff does not affect the layout result today because it is used only when |needs_box_fragment| is set today, but is better to match. * |needs_box_fragment| sometimes does not match. This needs larger fix that deferred to following patch. * Other failures were problems in the check code. Fixed in this CL. Bug: 636993 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I76d0c91c65e38c0135ec3fca91e716cac3937796 Reviewed-on: https://chromium-review.googlesource.com/c/1269727 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#598051}
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 .