JSDcov code coverage busted: ([Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [IJSDebugger.addClass]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"

RESOLVED FIXED in Firefox 57


a year ago
a year ago


(Reporter: aryx, Assigned: mccr8)




Firefox Tracking Flags

(firefox-esr52 unaffected, firefox55 unaffected, firefox56 unaffected, firefox57 fixed)


(Whiteboard: [stockwell fixed:other])


(1 attachment)

See bug 1381961 and bug 1400489 a shared global for JSMs.

First run with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=6c1b45e7e94dbf343813eda03f1fd0b3259b59b1&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable

Example log: https://treeherder.mozilla.org/logviewer.html#?job_id=131454517&repo=mozilla-central

[task 2017-09-16T10:56:13.244Z] 10:56:13    ERROR -  GECKO(1296) | TEST-UNEXPECTED-FAIL: setup.js | error parsing http://mochi.test:8888/tests.json ([Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [IJSDebugger.addClass]"  nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"  location: "JS frame :: resource://gre/modules/jsdebugger.jsm :: addDebuggerToGlobal :: line 23"  data: no])
[task 2017-09-16T10:56:13.246Z] 10:56:13     INFO -  GECKO(1296) | JavaScript error: resource://gre/modules/jsdebugger.jsm, line 23: NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [IJSDebugger.addClass]
[task 2017-09-16T11:12:53.277Z] 11:12:53     INFO - Automation Error: mozprocess timed out after 1000 seconds running ['/builds/worker/workspace/build/venv/bin/python', '-u', '/builds/worker/workspace/build/tests/mochitest/runtests.py', '--disable-e10s', '--total-chunks', '10', '--this-chunk', '8', '--jscov-dir-prefix=/builds/worker/workspace/build/blobber_upload_dir', '--appname=/builds/worker/workspace/build/application/firefox/firefox', '--utility-path=tests/bin', '--extra-profile-file=tests/bin/plugins', '--symbols-path=https://queue.taskcluster.net/v1/task/GzuKtY4NRAiee3sjJkwfpQ/artifacts/public/build/target.crashreporter-symbols.zip', '--certificate-path=tests/certs', '--setpref=webgl.force-enabled=true', '--quiet', '--log-raw=/builds/worker/workspace/build/blobber_upload_dir/plain-chunked-coverage_raw.log', '--log-errorsummary=/builds/worker/workspace/build/blobber_upload_dir/plain-chunked-coverage_errorsummary.log', '--use-test-media-devices', '--screenshot-on-fail', '--cleanup-crashes', '--marionette-startup-timeout=180', '--work-path=/builds/worker/workspace/build', '--chunk-by-dir=4', '--timeout=1200']

See https://dxr.mozilla.org/mozilla-central/rev/6be5c7d30d2def62a762ac187252eba626b23a92/js/ductwork/debugger/jsdebugger.jsm#23
> this.addDebuggerToGlobal = function addDebuggerToGlobal(global) {
>   init.addClass(global);
>   initPromiseDebugging(global);
> };
Flags: needinfo?(kmaglione+bmo)
Summary: Ccov code coverage busted: ([Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [IJSDebugger.addClass]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" → JSDcov code coverage busted: ([Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [IJSDebugger.addClass]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"
Looks like it needs to be updated to use getGlobalForObject
Flags: needinfo?(kmaglione+bmo) → needinfo?(continuation)


a year ago
Assignee: nobody → continuation
Component: Developer Tools: Debugger → Code Coverage
Flags: needinfo?(continuation)
Product: Firefox → Testing

Comment 2

a year ago
Are there any unit tests for CoverageUtils.jsm that I can run locally?

Comment 3

a year ago
I haven't tested this, but it looks reasonable to me...
Comment hidden (mozreview-request)

Comment 5

a year ago
Oh, I see, this isn't run by default, which is why my -p all -u all runs didn't find this.

Here's a try run to try to confirm that it works:

Comment 6

a year ago
This patch is pretty trivial, so if somebody gets impatient, feel free to land this with r=bustage...

Comment 7

a year ago
Comment on attachment 8909023 [details]
Bug 1400566 - Explicitly get the global of |this| in CoverageUtils.jsm.

Attachment #8909023 - Flags: review?(kmaglione+bmo) → review+

Comment 8

a year ago
I guess I didn't set up the try run correctly, so we'll just see what happens when I land this.

Comment 9

a year ago
Pushed by amccreight@mozilla.com:
Explicitly get the global of |this| in CoverageUtils.jsm. r=kmag

Comment 10

a year ago
Last Resolved: a year ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Thank you for taking care of this! :)

:mccr8, jsdcov and ccov only work with |mach try fuzzy -q 'linux64-jsdcov ...'| now after some changes from the windows taskcluster migration, IIRC. The documents haven't been updated to reflect this change yet, sorry about that.
188 failures in 1032 pushes (0.182 failures/push) were associated with this bug in the last 7 days. 

This is the #7 most frequent failure this week. 

** This failure happened more than 75 times this week! Resolving this bug is a very high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 1 week, the affected test(s) may be disabled. **   

Repository breakdown:
* mozilla-central: 188

Platform breakdown:
* linux64-jsdcov: 188

For more details, see:
Whiteboard: [stockwell fixed:other]
status-firefox55: --- → unaffected
status-firefox56: --- → unaffected
status-firefox-esr52: --- → unaffected
3 failures in 943 pushes (0.003 failures/push) were associated with this bug in the last 7 days.    

Repository breakdown:
* mozilla-central: 3

Platform breakdown:
* linux64-jsdcov: 3

For more details, see:
You need to log in before you can comment on or make changes to this bug.