Bug 45411 – LayoutTest regressions due to http://trac.webkit.org/changeset/67001

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Qt

[Peter Kasting]

http://trac.webkit.org/changeset/67001 caused the following four tests to fail:

fast/events/continuous-platform-wheelevent-in-scrolling-div.html
fast/events/scroll-event-does-not-bubble.html
fast/events/spatial-navigation/snav-only-clipped-overflow-content.html
fast/events/wheelevent-in-scrolling-div.html

There seem to be a couple different issues.  One is that horizontal scrolling is scrolling the wrong distance.  I assumed at first that this was from my setValue() -> scroll() conversions in http://trac.webkit.org/changeset/67001/trunk/WebCore/platform/qt/ScrollbarQt.cpp , but not only do they not seem wrong, I now think they can't be at fault because they only handle context menu actions (right?).  The other problem has to do with event bubbling.

[Peter Kasting]

http://trac.webkit.org/changeset/67009 added these to the Qt Skipped list to keep the bot green.

[Antonio Gomes]

(In reply to comment #0)
> http://trac.webkit.org/changeset/67001 caused the following four tests to fail:
> fast/events/spatial-navigation/snav-only-clipped-overflow-content.html

it is likely due to bug 43030. I am going to look at it soon, when I get some time. Thank you skipping pkasting.

[Csaba Osztrogonác]

Skip more 4 tests, they fail intermittently:
fast/events/platform-wheelevent-in-scrolling-div.html 
fast/events/spatial-navigation/snav-iframe-with-offscreen-focusable-element.html 
platform/qt/fast/events/event-sender-keydown-frame.html 
fast/events/spatial-navigation/snav-unit-overflow-and-scroll-in-direction.html 

http://trac.webkit.org/changeset/67026

[Csaba Osztrogonác]

(In reply to comment #3)
> Skip more 4 tests, they fail intermittently:
> fast/events/platform-wheelevent-in-scrolling-div.html 
> fast/events/spatial-navigation/snav-iframe-with-offscreen-focusable-element.html 
> platform/qt/fast/events/event-sender-keydown-frame.html 
> fast/events/spatial-navigation/snav-unit-overflow-and-scroll-in-direction.html 
> 
> http://trac.webkit.org/changeset/67026

It might be a DRT sideeffect bug, a real bug revealed by this change,
using uninitialized variable, etc. I'll check it tomorrow.

[Peter Kasting]

I know the cause of at least the "scrolling the wrong distance" issues.  I failed to properly initialize some members.

Patched in http://trac.webkit.org/changeset/67052.  I didn't remove any tests from the Qt Skipped lists yet, but this may address the majority of the problems here.

[Csaba Osztrogonác]

I tested the skipped tests again, and it seems your fix works,
so I unskipped tests: http://trac.webkit.org/changeset/67061