Make it impossible to enter the native drag and drop loop on Windows in test automation
Categories
(Core :: DOM: Copy & Paste and Drag & Drop, enhancement, P5)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox68 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
Details
Attachments
(1 file)
|
Bug 1540150 - Make it impossible to enter the native drag-drop loop in test automation. r?NeilDeakin
47 bytes,
text/x-phabricator-request
|
Details | Review |
We have some mochitests that simulate drag and drop operations. This can be hazardous, at least on Windows, because if the widget code actually gets the directive to start an actual drag and drop operation, we'll enter this native loop here:
which puts us into this really weird state where Gecko events won't be processed properly until the drag and drop operation exits (which seems to only be possible via actual user input, because script doesn't appear to run when we're in this native loop).
Most tests get around this by using the EventUtils.synthesizeDrag* functions, which call stopPropagation() and preventDefault() on the drag events, which prevents the widget layer from entering the loop.
There are, however, other tests, that don't do this. dom/events/test/test_bug1264380.html is one of those tests.
Normally, that wouldn't be a problem, since in our CI on Windows, the mouse is positioned (I believe) in the very top-left of the screen, which up until now had been considered "outside of the window" to widget code, because that area is a moz-window-dragging region, which hittests to HT_CAPTION, which we were treating as "outside of the client area, and so therefore outside of the window". This meant that the native drag and drop loop would never be entered, because the widget code would go, "Oh, the mouse is outside of the browser window, so this doesn't make sense to start and drag and drop operation inside of it".
With bug 1534389, I'm making those moz-window-dragging regions behave as if they're still inside of the client area so that we can get things like :hover effects on them. This meant that in CI, test_bug1264380.html would enter the ::DoDragDrop native event loop, and the test (or the next test - there's a bit of a delay) would get stuck and eventually time out.
I propose we do the following:
-
Make test_bug1264380.html use the EventUtils.synthesizeDragStart helper to avoid the native drag and drop loop from being entered, and
-
Make it so that attempting to invoke the native drag and drop loop in automation causes us to crash.
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Updated•5 years ago
|
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 2•5 years ago
|
||
Hi Neil,
Do you think you'll get a chance to review the new revision of my patch today? I'm hoping to land it this week. Let me know if you're too busy and I'll redirect it.
Pushed by mconley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ba0820e380a3 Make it impossible to enter the native drag-drop loop in test automation. r=NeilDeakin
Backout by dluca@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/62d21e6df277 Backed out changeset ba0820e380a3 for Mochitest failures in layout/generic/test/test_bug496275.html. CLOSED TREE
Pushed by mconley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5dae6d1b597a Make it impossible to enter the native drag-drop loop in test automation. r=NeilDeakin
|
||
Comment 7•5 years ago
|
||
| bugherder | ||
|
||
Comment 8•5 years ago
|
||
Backed out changeset 5dae6d1b597a (Bug 1540150) for [@ libgdk-3.so.0.1800.9 + 0x60a09] crashes in h13 tests.
Backout: https://hg.mozilla.org/integration/autoland/rev/aa4c97d22712b946f70a4b88f9ea927ce969d611
Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=5dae6d1b597a0de158d61e156df1c7f479bbcfbe&selectedJob=238280202
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=238280202&repo=autoland&lineNumber=12411
Pushed by mconley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c523c4658e98 Make it impossible to enter the native drag-drop loop in test automation. r=NeilDeakin
|
||
Comment 10•5 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Updated•5 years ago
|
Description
•