Bug 198220
| Summary: | REGRESSION (r245742) [iOS Sim] Layout Test compositing/rtl/rtl-overflow-scrolling.html is failing | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Shawn Roberts <sroberts> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | jlewis3, ryanhaddad, simon.fraser, tsavell, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=198208 | ||
Shawn Roberts
The following layout test is failing on iOS Simulator Release and Debug WK2
compositing/rtl/rtl-overflow-scrolling.html
Probable cause:
After changes in https://trac.webkit.org/changeset/245742/webkit the above test started failing on iOS Simulator
Verified locally, fails with r245742, passes in 245741
Unsure if it just a rebaseline?
Flakiness Dashboard:
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=compositing%2Frtl%2Frtl-overflow-scrolling.html
Diff:
--- /Volumes/Data/slave/ios-simulator-12-release-tests-wk2/build/layout-test-results/compositing/rtl/rtl-overflow-scrolling-expected.txt
+++ /Volumes/Data/slave/ios-simulator-12-release-tests-wk2/build/layout-test-results/compositing/rtl/rtl-overflow-scrolling-actual.txt
@@ -3,6 +3,6 @@
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x584
-layer at (8,8) size 300x300 scrollX -15 scrollWidth 1000 scrollHeight 1000
+layer at (8,8) size 300x300 scrollWidth 1000 scrollHeight 1000
RenderBlock {DIV} at (0,0) size 300x300
RenderBlock {DIV} at (-700,0) size 1000x1000 [bgcolor=#008000]
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/51114177>
Ryan Haddad
It looks like ios-sim EWS saw this on https://bugs.webkit.org/show_bug.cgi?id=198208, but it didn't get to confirm the result before the change was landed.
Simon Fraser (smfr)
Ah, this is a progression. scrollX should never be negative.
Simon Fraser (smfr)
https://trac.webkit.org/r245751