Open
Bug 1421150
Opened 8 years ago
Updated 3 years ago
Intermittent dom/events/test/test_DataTransferItemList.html | TypeError: _EU_maybeWrap(...) is null - Should not throw any errors
Categories
(Core :: DOM: Events, defect, P2)
Core
DOM: Events
Tracking
()
REOPENED
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: intermittent-failure, leave-open, Whiteboard: [stockwell disabled])
Attachments
(1 file, 1 obsolete file)
|
474 bytes,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 7•8 years ago
|
||
From what I can tell, this is caused by images decoding asynchronously. Usually it comes back soon enough, but sometimes it doesn't and we end up trying to scroll the full frame, rather than the underlying image. We can fix this easily if bug 1416328 was implemented. In turn, I have an incentive to see this fixed so I can land bug 1436247 which was backed out due to slightly changing the timing for this bug and permafailing it.
Assignee: nobody → aosmond
Blocks: 1436247
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Priority: -- → P2
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 13•8 years ago
|
||
Test is failing frequently on Linux. (~127 failures in the last 7days)
Here is a recent log:
https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=164083783&lineNumber=2684
[task 2018-02-24T02:07:42.371Z] 02:07:42 INFO - TEST-START | dom/events/test/test_DataTransferItemList.html
[task 2018-02-24T02:07:43.409Z] 02:07:43 INFO - TEST-INFO | started process screentopng
[task 2018-02-24T02:07:43.842Z] 02:07:43 INFO - TEST-INFO | screentopng: exit 0
[task 2018-02-24T02:07:43.842Z] 02:07:43 INFO - Buffered messages logged at 02:07:43
[task 2018-02-24T02:07:43.842Z] 02:07:43 INFO - SpawnTask.js | Entering test
[task 2018-02-24T02:07:43.842Z] 02:07:43 INFO - Buffered messages finished
[task 2018-02-24T02:07:43.844Z] 02:07:43 INFO - TEST-UNEXPECTED-FAIL | dom/events/test/test_DataTransferItemList.html | TypeError: _EU_maybeWrap(...) is null - Should not throw any errors
[task 2018-02-24T02:07:43.845Z] 02:07:43 INFO - createDragEventObject@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:2428:38
[task 2018-02-24T02:07:43.846Z] 02:07:43 INFO - synthesizeDragOver@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:2517:15
[task 2018-02-24T02:07:43.847Z] 02:07:43 INFO - synthesizeDrop@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:2596:34
[task 2018-02-24T02:07:43.848Z] 02:07:43 INFO - @chrome://mochitests/content/chrome/dom/events/test/test_DataTransferItemList.html:204:5
[task 2018-02-24T02:07:43.849Z] 02:07:43 INFO - async*add_task/</<@chrome://mochikit/content/tests/SimpleTest/SpawnTask.js:298:21
[task 2018-02-24T02:07:43.850Z] 02:07:43 INFO - onFulfilled@chrome://mochikit/content/tests/SimpleTest/SpawnTask.js:69:15
[task 2018-02-24T02:07:43.851Z] 02:07:43 INFO - co/<@chrome://mochikit/content/tests/SimpleTest/SpawnTask.js:58:5
[task 2018-02-24T02:07:43.852Z] 02:07:43 INFO - co@chrome://mochikit/content/tests/SimpleTest/SpawnTask.js:54:10
[task 2018-02-24T02:07:43.853Z] 02:07:43 INFO - setTimeout handler*SimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:676:12
[task 2018-02-24T02:07:43.854Z] 02:07:43 INFO - add_task@chrome://mochikit/content/tests/SimpleTest/SpawnTask.js:271:7
[task 2018-02-24T02:07:43.854Z] 02:07:43 INFO - @chrome://mochitests/content/chrome/dom/events/test/test_DataTransferItemList.html:27:3
[task 2018-02-24T02:07:43.855Z] 02:07:43 INFO - GECKO(1981) | MEMORY STAT vsizeMaxContiguous not supported in this build configuration.
[task 2018-02-24T02:07:43.857Z] 02:07:43 INFO - GECKO(1981) | MEMORY STAT | vsize 825MB | residentFast 257MB | heapAllocated 102MB
[task 2018-02-24T02:07:43.858Z] 02:07:43 INFO - TEST-OK | dom/events/test/test_DataTransferItemList.html | took 1054ms
:overholt, can you please take a look?
Flags: needinfo?(overholt)
| Comment hidden (Intermittent Failures Robot) |
Comment 15•8 years ago
|
||
Hello! Joel, can you please review this patch? I didn't disabled it on debug because the failures I saw in Orange factor were only on try and the one on inbound was missclassified.
There was also a file named chrome.ini where there was this test name in it. Should I have edited that ini file or it's ok if I created one with this test name? And also what happens with the new file we created? It gets added to the remote repo when the patch is landed? Thanks.
Attachment #8953915 -
Flags: review?(jmaher)
Comment 16•8 years ago
|
||
Comment on attachment 8953915 [details] [diff] [review]
Disable on Linux !debug
Review of attachment 8953915 [details] [diff] [review]:
-----------------------------------------------------------------
sorry for the confusion here- this would work great if this were a web-platform-test (testing/web-platform/tests/...), but in this case it is a mochitest chrome (hence the chrome.ini), so you can modify the existing chrome.ini file. In general when you see mochitest.ini, chrome.ini, browser.ini, or xpcshell.ini- you modify in the existing file (make note that we have a lot of other variations like browser2.ini, or mochitest-oop.ini, etc.)
Attachment #8953915 -
Flags: review?(jmaher) → review-
Comment 17•8 years ago
|
||
Oh, initially that was what I thought but eventually did it another .ini. No worries, I've now modified chrome.ini with those changes. Thanks.
Attachment #8953915 -
Attachment is obsolete: true
Attachment #8953976 -
Flags: review?(jmaher)
Comment 18•8 years ago
|
||
Comment on attachment 8953976 [details] [diff] [review]
Disable on Linux !debug
Review of attachment 8953976 [details] [diff] [review]:
-----------------------------------------------------------------
looks great
Attachment #8953976 -
Flags: review?(jmaher) → review+
Updated•8 years ago
|
Keywords: checkin-needed
Comment 19•8 years ago
|
||
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5684c810fb4b
Disable dom/events/test/test_DataTransferItemList.html for frequent failures. r=jmaher
Keywords: checkin-needed
Updated•8 years ago
|
Keywords: leave-open
Comment 20•8 years ago
|
||
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/723b390519de
Fix the syntax for disabling dom/events/test/test_DataTransferItemList.html on a CLOSED TREE
Comment 21•8 years ago
|
||
I think Andrew's working on this given comment 7.
Flags: needinfo?(overholt) → needinfo?(aosmond)
Comment 22•8 years ago
|
||
| bugherder | ||
Updated•8 years ago
|
Whiteboard: [stockwell disable-recommended] → [stockwell disabled]
| Comment hidden (Intermittent Failures Robot) |
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(aosmond)
Resolution: --- → FIXED
Updated•6 years ago
|
Assignee: aosmond → nobody
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•