Following layout tests failed on LayoutNG due by whitespaces differences:
- accessibility/canvas-fallback-content-2.html
- accessibility/css-styles.html
Don't know if it should match though. I had reabaselined some before too, if reasonable, I think it's fine to change, but good to raise an attention that there are some more, and maybe nice to check if the changes are really reasonable.
Test runner relies on TextIterator to generate text dump. Many whitespace differences are due to hacky (and buggy in many cases) whitespace fixup, which are gone in the NG implementation.
So I think we can just rebaseline.
Owner: yosin@chromium.org Status: Started (was: Available)
It seems number of whitespace differences come from bug of Element#innerText.
I start making Element#innerText spec-compliant and to work as same as both
legacy layout and LayoutNG.
See https://goo.gl/VW9xxe
NG version of test expectations for following files are correct:
editing/pasteboard/copy-crash.html
fast/block/positioning/complex-percentage-height.html
fast/borders/border-color-visited.html
fast/dom/horizontal-scrollbar-in-rtl.html
fast/dom/left-overflow-in-ltr.html
fast/dom/right-overflow-in-rtl.html
fast/dom/set-outer-text.html
fast/forms/textarea/textarea-scrollbar-height.html
fast/lists/calc-width-with-space.html
fast/lists/item-not-in-list-line-wrapping.html
fast/overflow/scrollbar-restored-and-then-locked.html
fast/table/cell-in-row-before-misnested-text-crash.xhtml
fast/table/section-in-table-before-misnested-text-crash.xhtml
fast/text/whitespace/whitespace-and-margin-wrap-after-list-marker-crash.html
inspector-protocol/page/capture-snapshot.js
Comment 1 by yosin@chromium.org
, Jun 21 2018Cc: xiaoche...@chromium.org