Open
Bug 1459567
Opened 7 years ago
Updated 3 years ago
Remove unnecessary generators from devtools codebase
Categories
(DevTools :: General, enhancement, P3)
DevTools
General
Tracking
(Not tracked)
NEW
People
(Reporter: yulia, Unassigned)
References
Details
Since the Task.async refactoring, we have a number of places where we should re-evaluate the use of generators. Full list of generator usage: https://searchfox.org/mozilla-central/search?q=function*&case=true®exp=false&path=%5Edevtools
Areas we should focus on or check:
- Dispatch calls that use generators, such as https://searchfox.org/mozilla-central/source/devtools/client/memory/actions/filter.js#25
- Sourcemaps translation of async to generators -> perhaps we can avoid this transformation
- unnecessary generators such as https://searchfox.org/mozilla-central/source/devtools/server/tests/browser/storage-unsecured-iframe.html#19
- updates to docs such as https://searchfox.org/mozilla-central/source/devtools/docs/tests/writing-tests.md#46 and https://searchfox.org/mozilla-central/source/devtools/client/inspector/shared/test/head.js#42
- checking tests that might potentially be broken and failing silently such as https://searchfox.org/mozilla-central/source/devtools/shared/webconsole/test/test_commands_other.html#38 and https://searchfox.org/mozilla-central/source/devtools/shared/webconsole/test/common.js#228-242
| Reporter | ||
Updated•7 years ago
|
Priority: -- → P3
Updated•7 years ago
|
Product: Firefox → DevTools
| Reporter | ||
Updated•5 years ago
|
Assignee: ystartsev → nobody
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•