RESOLVED INVALID 105173
[Qt][WK2] compositing/overflow/automatically-opt-into-composited-scrolling.html makes other tests fail 
https://bugs.webkit.org/show_bug.cgi?id=105173
Summary [Qt][WK2] compositing/overflow/automatically-opt-into-composited-scrolling.ht...
Csaba Osztrogonác
Reported 2012-12-17 06:59:53 PST
compositing/overflow/automatically-opt-into-composited-scrolling.html introduced in https://trac.webkit.org/changeset/137828 made some tests fail on Qt-WK2. I'll add detailed results to this bug soon.
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2012-12-17 07:03:21 PST
I skipped the culprit - r137907. Let's see which tests are fixed.
Csaba Osztrogonác
Comment 2 2012-12-17 08:27:32 PST
(In reply to comment #1) > I skipped the culprit - r137907. Let's see which tests are fixed. Skipping made the following tests pass again: - fast/multicol/overflow-unsplittable.html - fast/overflow/scrollbar-click-retains-focus.html - fast/spatial-navigation/snav-clipped-overflowed-content.html - fast/spatial-navigation/snav-only-clipped-overflow-content.html - fast/spatial-navigation/snav-simple-content-overflow.html
Csaba Osztrogonác
Comment 3 2012-12-17 08:29:33 PST
Diffs with enabled compositing/overflow/automatically-opt-into-composited-scrolling.html: --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/multicol/overflow-unsplittable-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/multicol/overflow-unsplittable-actual.txt @@ -52,7 +52,7 @@ text run at (0,608) width 158: "eu ipsum sit amet ligula mattis" text run at (0,632) width 158: "porttitor sit amet id felis. Integer" text run at (0,656) width 155: "volutpat aliquam odio et feugiat." -layer at (11,510) size 158x304 backgroundClip at (11,510) size 158x90 clip at (13,512) size 139x88 outlineClip at (0,0) size 800x600 scrollHeight 456 +layer at (181,11) size 158x304 clip at (183,13) size 139x300 scrollHeight 456 RenderBlock {DIV} at (3,502) size 158x304 [border: (2px solid #000000)] RenderText {#text} at (2,7) size 139x446 text run at (2,7) width 139: "Aenean diam enim, suscipit" --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/overflow/scrollbar-click-retains-focus-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/overflow/scrollbar-click-retains-focus-actual.txt @@ -5,7 +5,7 @@ Focus should remain in the input -PASS document.activeElement.tagName is "INPUT" +FAIL document.activeElement.tagName should be INPUT. Was BODY. Focus should move if ancestor is mouse focusable PASS document.activeElement.tagName is "DIV" --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/spatial-navigation/snav-clipped-overflowed-content-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/spatial-navigation/snav-clipped-overflowed-content-actual.txt @@ -4,13 +4,13 @@ . PASS gFocusedDocument.activeElement.getAttribute("id") is "1" -PASS gFocusedDocument.activeElement.getAttribute("id") is "1" -PASS gFocusedDocument.activeElement.getAttribute("id") is "2" -PASS gFocusedDocument.activeElement.getAttribute("id") is "2" +FAIL gFocusedDocument.activeElement.getAttribute("id") should be 1. Was 2. +FAIL gFocusedDocument.activeElement.getAttribute("id") should be 2. Was 3. +FAIL gFocusedDocument.activeElement.getAttribute("id") should be 2. Was 3. PASS gFocusedDocument.activeElement.getAttribute("id") is "3" PASS gFocusedDocument.activeElement.getAttribute("id") is "2" -PASS gFocusedDocument.activeElement.getAttribute("id") is "2" -PASS gFocusedDocument.activeElement.getAttribute("id") is "1" -PASS gFocusedDocument.activeElement.getAttribute("id") is "1" +FAIL gFocusedDocument.activeElement.getAttribute("id") should be 2. Was 1. +FAIL gFocusedDocument.activeElement.getAttribute("id") should be 1. Was start. +FAIL gFocusedDocument.activeElement.getAttribute("id") should be 1. Was start. PASS gFocusedDocument.activeElement.getAttribute("id") is "start" This test tests that a div with css overflow:auto will scroll to reveal its focusable elements --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/spatial-navigation/snav-only-clipped-overflow-content-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/spatial-navigation/snav-only-clipped-overflow-content-actual.txt @@ -14,6 +14,6 @@ And this is another normal link_2. -PASS gFocusedDocument.activeElement.getAttribute("id") is "start" +FAIL gFocusedDocument.activeElement.getAttribute("id") should be start. Was 1. PASS true is true --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/spatial-navigation/snav-simple-content-overflow-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/spatial-navigation/snav-simple-content-overflow-actual.txt @@ -20,5 +20,5 @@ This link should NOT get focused. -PASS gFocusedDocument.activeElement.getAttribute("id") is "start" +FAIL gFocusedDocument.activeElement.getAttribute("id") should be start. Was 1.
Csaba Osztrogonác
Comment 4 2012-12-17 08:29:55 PST
The culprit test fails in itself too: --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/compositing/overflow/automatically-opt-into-composited-scrolling-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/compositing/overflow/automatically-opt-into-composited-scrolling-actual.txt @@ -14,30 +14,30 @@ Iteration 6, layer tree Iteration 7: Passed Iteration 7, layer tree -Iteration 8: Passed -Iteration 8, no layer tree -Iteration 9: Passed -Iteration 9, no layer tree +Iteration 8: FAILED +Iteration 8, layer tree +Iteration 9: FAILED +Iteration 9, layer tree Iteration 10: Passed Iteration 10, layer tree Iteration 11: Passed Iteration 11, layer tree -Iteration 12: Passed -Iteration 12, no layer tree -Iteration 13: Passed -Iteration 13, no layer tree -Iteration 14: Passed -Iteration 14, no layer tree -Iteration 15: Passed -Iteration 15, no layer tree -Iteration 16: Passed -Iteration 16, no layer tree -Iteration 17: Passed -Iteration 17, no layer tree -Iteration 18: Passed -Iteration 18, no layer tree -Iteration 19: Passed -Iteration 19, no layer tree +Iteration 12: FAILED +Iteration 12, layer tree +Iteration 13: FAILED +Iteration 13, layer tree +Iteration 14: FAILED +Iteration 14, layer tree +Iteration 15: FAILED +Iteration 15, layer tree +Iteration 16: FAILED +Iteration 16, layer tree +Iteration 17: FAILED +Iteration 17, layer tree +Iteration 18: FAILED +Iteration 18, layer tree +Iteration 19: FAILED +Iteration 19, layer tree Iteration 20: Passed Iteration 20, layer tree Iteration 21: Passed @@ -66,18 +66,18 @@ Iteration 32, layer tree Iteration 33: Passed Iteration 33, layer tree -Iteration 34: Passed -Iteration 34, no layer tree -Iteration 35: Passed -Iteration 35, no layer tree +Iteration 34: FAILED +Iteration 34, layer tree +Iteration 35: FAILED +Iteration 35, layer tree Iteration 36: Passed Iteration 36, layer tree Iteration 37: Passed Iteration 37, layer tree -Iteration 38: Passed -Iteration 38, no layer tree -Iteration 39: Passed -Iteration 39, no layer tree +Iteration 38: FAILED +Iteration 38, layer tree +Iteration 39: FAILED +Iteration 39, layer tree Iteration 40: Passed Iteration 40, layer tree Iteration 41: Passed @@ -94,10 +94,10 @@ Iteration 46, layer tree Iteration 47: Passed Iteration 47, layer tree -Iteration 48: Passed -Iteration 48, no layer tree -Iteration 49: Passed -Iteration 49, no layer tree +Iteration 48: FAILED +Iteration 48, layer tree +Iteration 49: FAILED +Iteration 49, layer tree Iteration 50: Passed Iteration 50, layer tree Iteration 51: Passed @@ -106,22 +106,22 @@ Iteration 52, layer tree Iteration 53: Passed Iteration 53, layer tree -Iteration 54: Passed -Iteration 54, no layer tree -Iteration 55: Passed -Iteration 55, no layer tree -Iteration 56: Passed -Iteration 56, no layer tree -Iteration 57: Passed -Iteration 57, no layer tree -Iteration 58: Passed -Iteration 58, no layer tree -Iteration 59: Passed -Iteration 59, no layer tree +Iteration 54: FAILED +Iteration 54, layer tree +Iteration 55: FAILED +Iteration 55, layer tree +Iteration 56: FAILED +Iteration 56, layer tree +Iteration 57: FAILED +Iteration 57, layer tree +Iteration 58: FAILED +Iteration 58, layer tree +Iteration 59: FAILED +Iteration 59, layer tree Iteration 60: Passed Iteration 60, layer tree -Iteration 61: Passed -Iteration 61, no layer tree +Iteration 61: FAILED +Iteration 61, layer tree Iteration 62: Passed Iteration 62, layer tree
Csaba Osztrogonác
Comment 5 2012-12-17 08:31:14 PST
If you are interested in debugging this bug, you can easily reproduce the fails with $Tools/Scripts/run-webkit-tests compositing/overflow/automatically-opt-into-composited-scrolling.html ONE_FROM_THE_FAILING_LIST
Jocelyn Turcotte
Comment 6 2014-02-03 03:23:58 PST
=== Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.
Note You need to log in before you can comment on or make changes to this bug.