Rubber-banding in overflow:scroll regions does not work correctly with direction:rtl
Created attachment 237004 [details] Test case Test case that shows the bug.
Created attachment 237006 [details] Patch
Comment on attachment 237006 [details] Patch no tests?
(In reply to comment #3) > (From update of attachment 237006 [details]) > no tests? I'm not sure that I can write a test that will actually behave differently with this change, but I will give it a shot.
(In reply to comment #3) > (From update of attachment 237006 [details]) > no tests? I spent some time today trying to make a test for this, and I still have not figured out how to make a reliable test. The momentum events that Brent added are cool, but they don't seem to be 100% consistent for replicating rubber-banding.
http://trac.webkit.org/changeset/172945
Comment on attachment 237006 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=237006&action=review > Source/WebCore/rendering/RenderLayer.cpp:2645 > + return IntRect(IntPoint(scrollPosition().x(), scrollPosition().y()), Why not just IntRect(scrollPosition(), ...)? No need for all this fun.
(In reply to comment #7) > (From update of attachment 237006 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=237006&action=review > > > Source/WebCore/rendering/RenderLayer.cpp:2645 > > + return IntRect(IntPoint(scrollPosition().x(), scrollPosition().y()), > > Why not just IntRect(scrollPosition(), ...)? No need for all this fun. r=you to fix, Tim?
Absolutely!
(In reply to comment #9) > Absolutely! r172965 :D