RESOLVED FIXED Bug 82167
[Qt][WK2] REGRESSION (r111993): It made 3 layoutests fail and 30 test flakey, 3 API tests fail 
https://bugs.webkit.org/show_bug.cgi?id=82167
Summary [Qt][WK2] REGRESSION (r111993): It made 3 layoutests fail and 30 test flakey,...
Csaba Osztrogonác
Reported 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.
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2012-03-25 22:38:19 PDT
I skipped these tests, please unskip them with the proper fix.
Csaba Osztrogonác
Comment 2 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)]
Csaba Osztrogonác
Comment 3 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
Note You need to log in before you can comment on or make changes to this bug.