Bug 105173

Summary: [Qt][WK2] compositing/overflow/automatically-opt-into-composited-scrolling.html makes other tests fail
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: enne, hausmann, ossy, vollick
Priority: P2 Keywords: Qt, QtTriaged
Version: 420+   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 87008, 94743    

Description Csaba Osztrogonác 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.
Comment 1 Csaba Osztrogonác 2012-12-17 07:03:21 PST
I skipped the culprit - r137907. Let's see which tests are fixed.
Comment 2 Csaba Osztrogonác 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
Comment 3 Csaba Osztrogonác 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.
Comment 4 Csaba Osztrogonác 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
Comment 5 Csaba Osztrogonác 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
Comment 6 Jocelyn Turcotte 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.