Bug 139224 – REGRESSION(r176687): Caused webkit2 test assertion failure crashes in WebKit::WebPage::willStartDrag()

• Component: WebKit2
• Status: RESOLVED FIXED

[Mark Lam]

The following tests started crashing with assertion failures after r176687:

fast/events/clear-drag-state.html
fast/css/user-drag-none.html
editing/pasteboard/dataTransfer-setData-getData.html
editing/pasteboard/drop-text-events-sideeffect-crash.html
editing/pasteboard/drag-drop-iframe-refresh-crash.html
editing/pasteboard/drop-text-events-sideeffect.html
editing/selection/user-drag-element-and-user-select-none.html

[Mark Lam]

Updated TestExpectations to expect crashes for these tests in r176723: <http://trac.webkit.org/r176723>.

[Alexey Proskuryakov]

I think that a Skip expectation would be more appropriate. Seven tests that crash every time make bots substantially slower, and I suspect that the high system load associated with creating crash logs may destabilize other tests too.

So, the better expectations would be (note how they can be limited to debug builds only):

webkit.org/b/139224 [ Debug ] fast/events/clear-drag-state.html [ Skip ]
...

[Alexey Proskuryakov]

Also, crashes make running tests locally very annoying.

[Mark Lam]

Updated TestExpectations to skip tests per Alexey's instructions in r176726: <http://trac.webkit.org/r176726>.

Created attachment 242503 [details]
Wait for the UI process to begin dragging if needed before handling a synthetic synchronous mouse event

Fixed in <http://trac.webkit.org/r176728>.