requestEvaluation's frame argument isn't used
Categories
(DevTools :: Console, enhancement, P3)
Tracking
(firefox70 fixed)
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
requestEvaluation
currently implements a frame
argument, but this doesn't seem to be used:
https://searchfox.org/mozilla-central/source/devtools/client/webconsole/webconsole-wrapper.js#177-179
This introduce some dead piece of code in this method:
https://searchfox.org/mozilla-central/source/devtools/client/webconsole/webconsole-wrapper.js#177-179
Comment 1•5 years ago
|
||
The file is going to change soon, see this permalink to see the unused code https://searchfox.org/mozilla-central/rev/4436573fa3d5c4311822090e188504c10c916c6f/devtools/client/webconsole/webconsole-wrapper.js#177-179
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Comment 3•5 years ago
|
||
Updated•5 years ago
|
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/adc394f833f9 Removed unused requestEvaluation's frame argument. r=nchevobbe
Comment 5•5 years ago
|
||
Backed out 2 changesets (Bug 1568203, Bug 1568185) for ES lint failure on eval-with-debugger.js
Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&fromchange=adc394f833f95918313bb71ed4249fc117d96524&tochange=73a5452606a89b96b833a4171fff407dad93dc3d&selectedJob=258059020
Backout link: https://hg.mozilla.org/integration/autoland/rev/73a5452606a89b96b833a4171fff407dad93dc3d
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=258059020&repo=autoland&lineNumber=222
[task 2019-07-24T07:57:55.441Z] x86_64-linux-gnu-gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-Bsymbolic-functions -Wl,-z,relro -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -Wdate-time -D_FORTIFY_SOURCE=2 -g -fstack-protector-strong -Wformat -Werror=format-security -Wl,-Bsymbolic-functions -Wl,-z,relro -Wdate-time -D_FORTIFY_SOURCE=2 -g -fstack-protector-strong -Wformat -Werror=format-security build/temp.linux-x86_64-2.7/psutil/_psutil_common.o build/temp.linux-x86_64-2.7/psutil/_psutil_posix.o -o build/lib.linux-x86_64-2.7/psutil/_psutil_posix.so
[task 2019-07-24T07:57:55.441Z] copying build/lib.linux-x86_64-2.7/psutil/_psutil_linux.so -> psutil
[task 2019-07-24T07:57:55.441Z] copying build/lib.linux-x86_64-2.7/psutil/_psutil_posix.so -> psutil
[task 2019-07-24T07:57:55.441Z]
[task 2019-07-24T07:57:55.441Z] Error processing command. Ignoring because optional. (optional:packages.txt:comm/build/virtualenv_packages.txt)
[task 2019-07-24T08:11:43.909Z] TEST-UNEXPECTED-ERROR | /builds/worker/checkouts/gecko/devtools/server/actors/webconsole/eval-with-debugger.js:10:9 | 'Cu' is assigned a value but never used. (no-unused-vars)
[taskcluster 2019-07-24 08:11:44.241Z] === Task Finished ===
[taskcluster 2019-07-24 08:11:45.057Z] Unsuccessful task run with exit code: 1 completed in 1152.148 seconds
Assignee | ||
Comment 6•5 years ago
|
||
Sorry, I missed that eslint failure... it should be fixed now.
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dc2fc7742cde Removed unused requestEvaluation's frame argument. r=nchevobbe
Comment 8•5 years ago
|
||
bugherder |
Description
•