This test is failing:
third_party/blink/tools/run_web_tests.py --debug --additional-driver-flag=--enable-blink-gen-property-trees fast/block/positioning/vertical-rl/002.html
It looks like we are getting the vertical-rl scrollbar offset wrong.
I briefly looked into this. I don't know where the bug is, but this may help.
The BlinkGenPropertyTree scroll node is created via FragmentPaintPropertyTreeBuilder::UpdateScrollAndScrollTranslation.
The current scroll node is created via GraphicsLayers which are setup in CompositedLayerMapping::UpdateScrollingLayerGeometry.
Some of the offsets/positions used in these two functions look different to me. In particular the HasFlippedBlocksWritingMode check in FragmentPaintPropertyTreeBuilder.
The following test seems to prove that the bug is related to scrolling:
1. Open the test in content_shell with --enable-blink-gen-property-trees;
2. Shrink the window width until the horizontal scrollbar appears;
3. Try to scroll horizontally.
Expected:
2. The thumb of the horizontal scrollbar is on the right;
3. The contents are scrollable.
I'm not actively working on it (pdr@ mentioned you were looking into the RTL issues).
Background:
I added the expectations in https://chromium-review.googlesource.com/c/chromium/src/+/1173441 because I added a new clamp in CC that caused us to apply CC's scroll offset limits in some of these tests. The clamp itself is correct, but because CC's limits are wrong due to the RTL bug the tests were failing.
Comment 1 by wangxianzhu@chromium.org
, Aug 16Owner: wangxianzhu@chromium.org
Status: Assigned (was: Available)