Bug 82167

Summary: [Qt][WK2] REGRESSION (r111993): It made 3 layoutests fail and 30 test flakey, 3 API tests fail
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Tools / TestsAssignee: Jesus Sanchez-Palencia <jesus>
Status: RESOLVED FIXED    
Severity: Critical CC: jesus, kenneth, ossy
Priority: P1 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 70236, 79666, 81143    

Description Csaba Osztrogonác 2012-03-25 22:21:51 PDT
New failing tests:
-------------------
--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/css/hover-update-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/css/hover-update-actual.txt 
@@ -4,5 +4,5 @@
 
 Only green now
 Hover color before test: rgb(255, 255, 0)
-PASS window.getComputedStyle(document.getElementById('b'), null).backgroundColor is "rgb(0, 128, 0)"
+FAIL window.getComputedStyle(document.getElementById('b'), null).backgroundColor should be rgb(0, 128, 0). Was rgb(255, 255, 0).
 

--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/css/nested-layers-with-hover-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/css/nested-layers-with-hover-actual.txt 
@@ -7,10 +7,10 @@
         RenderTable at (0,0) size 784x42
           RenderTableSection (anonymous) at (0,0) size 784x42
       RenderBlock {DIV} at (0,42) size 784x21
-        RenderText {#text} at (0,0) size 239x21
-          text run at (0,0) width 239: "PASSED: The hover effect works!"
+        RenderText {#text} at (0,0) size 526x21
+          text run at (0,0) width 526: "FAILED: The hover effect did not correctly propagate down the layer chain."
 layer at (8,8) size 784x42
-  RenderTableRow {DIV} at (0,0) size 784x42 [bgcolor=#0000FF]
+  RenderTableRow {DIV} at (0,0) size 784x42
     RenderTableCell (anonymous) at (0,0) size 784x42 [r=0 c=0 rs=1 cs=1]
       RenderText {#text} at (0,0) size 739x42
         text run at (0,0) width 739: "To test this manually, first make sure you can see this text at all, and then hover over it and make sure the"


--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/events/mouseout-on-window-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/events/mouseout-on-window-actual.txt 
@@ -1,2 +1,2 @@
 This test ensures that mouse out events are sent to the window.
-PASS: Received mouseout event.
+FAIL: No mouseout was sent.

New flakey tests: (Fail first, and pass after re-run):
editing/pasteboard/copy-crash.html
fast/block/positioning/absolute-appended-to-inline.html
fast/css/hover-affects-child.html
fast/css/pseudo-any.html
fast/dom/shadow/shadow-boundary-crossing.html
fast/dom/shadow/shadow-contents-event.html
fast/events/dont-loose-last-event.html
fast/events/event-sender-mouse-moved.html
fast/events/iframe-onmousemove.html
fast/events/mousemove-after-drag-over-scrollbar.html
fast/events/mouse-moved-remove-frame-crash.html
fast/events/mouseout-dead-node.html
fast/events/mouseout-dead-subframe.html
fast/events/mouseover-mouseout2.html
fast/events/mouseover-mouseout.html
fast/events/touch/touch-before-pressing-spin-button.html
fast/forms/autofocus-input-css-style-change.html
fast/forms/input-step-as-double.html
fast/forms/number/spin-button-events.html
fast/forms/number/spin-button-state.html
fast/forms/number/spin-in-multi-column.html
fast/forms/textfield-overflow.html
fast/overflow/overflow-focus-ring.html
fast/replaced/image-map-2.html
scrollbars/overflow-custom-scrollbar-crash.html
scrollbars/scrollbar-iframe-click-does-not-blur-content.html
svg/custom/foreign-object-skew.svg
svg/custom/foreign-object-skew.svg
svg/custom/js-update-container2.svg
svg/custom/pointer-events-invalid-fill.svg
svg/custom/use-instanceRoot-as-event-target.xhtml

See http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r111993%20%2822025%29/results.html for details.
Comment 1 Csaba Osztrogonác 2012-03-25 22:38:19 PDT
I skipped these tests, please unskip them with the proper fix.
Comment 2 Csaba Osztrogonác 2012-03-25 23:26:37 PDT
Additionally it broke 3 Qt-WK2 API tests:

FAIL!  : qmltests::DesktopWebViewLinkHovered::test_linkHovered() 'wait for signal linkHovered' returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/Source/WebKit2/UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_linkHovered.qml(49)]
FAIL!  : qmltests::DesktopWebViewLinkHovered::test_linkHoveredDoesntEmitRepeated() 'wait for signal linkHovered' returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/Source/WebKit2/UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_linkHovered.qml(68)]
FAIL!  : qmltests::DesktopWebViewLoadHtml::test_baseUrlAfterLoadHtml() 'wait for signal linkHovered' returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/Source/WebKit2/UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_loadHtml.qml(48)]
Comment 3 Csaba Osztrogonác 2012-03-26 08:20:15 PDT
The original patch was rolled out, so I rolled out skipping tests too - http://trac.webkit.org/changeset/112098