Closed
Bug 1360513
Opened 8 years ago
Closed 7 years ago
Jetpack test-leak-event-dom-closed-window.js fail with leaks when Screenshots is enabled as a system add-on (Legacy)
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: standard8, Unassigned)
References
Details
When we enable the WebExtension part of the screenshots system add-on, we see Jetpack test failures due to leaks:
https://treeherder.mozilla.org/logviewer.html#?job_id=94848421&repo=try&lineNumber=5056
[tINFO - TEST-PASS | jetpack-package/addon-sdk/source/test/leak/test-leak-event-dom-closed-window.js.test sdk/event/dom does not leak when attached to closed window | should see at least one new window
WARNING - TEST-UNEXPECTED-FAIL | jetpack-package/addon-sdk/source/test/leak/test-leak-event-dom-closed-window.js.test sdk/event/dom does not leak when attached to closed window | Component returned failure code: 0x8057000a (NS_ERROR_XPC_BAD_CONVERT_NATIVE) [xpcIJSWeakReference.get]
INFO - TEST-INFO | Traceback (most recent call last):
INFO - File "resource://gre/modules/Promise-backend.js", line 806, in walkerLoop
INFO - this.handlers.shift().process();
INFO - File "resource://gre/modules/Promise-backend.js", line 925, in process
INFO - nextValue = this.onReject.call(undefined, nextValue);
INFO - File "resource://gre/modules/commonjs/sdk/test/assert.js", line 88, in fail
INFO - this._log.fail(message);
INFO - File "resource://gre/modules/commonjs/sdk/deprecated/unit-test.js", line 104, in fail
INFO - this.console.testMessage(false, false, this.test.name, message);
INFO - File "resource://gre/modules/commonjs/sdk/test/harness.js", line 541, in testMessage
INFO - this.trace();
This is all platforms, both opt and debug.
It could also be related to bug 1360512.
Reporter | ||
Comment 1•7 years ago
|
||
I've re-done a try build, and this is no longer an issue. Probably due to the recent webextension improvements.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•