Typo in pref name: extension.webextensions.remote in `RemoteWorkerManager.cpp`
Categories
(WebExtensions :: Untriaged, task, P5)
Tracking
(firefox123 fixed)
| Tracking | Status | |
|---|---|---|
| firefox123 | --- | fixed |
People
(Reporter: byjxjr3usp, Assigned: eliotj12, NeedInfo)
Details
(Keywords: good-first-bug)
Attachments
(1 file)
Actual results:
Likely typo in the comment in dom/workers/remoteworkers/RemoteWorkerManager.cpp:
// when "extension.webextensions.remote" pref is false.
Must be referring to extensions.webextensions.remote.
Comment 1•2 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'WebExtensions::Untriaged' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Comment 3•2 years ago
|
||
I'd like to be assigned this bug, if possible. I already found and fixed the typo on my local build -- I would just need to get the change committed and reviewed. If anyone has advice on finding an appropriate reviewer, I'd appreciate it. (I've already taken a look at the new contributor guides, and I have a couple people in mind, but I'd hate to pester anyone with code that isn't their purview.)
| Assignee | ||
Comment 5•2 years ago
|
||
Updated•2 years ago
|
Comment 6•2 years ago
|
||
There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:eliotj12, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.
Comment 7•2 years ago
|
||
I just landed the patch to fix a typo.
Comment 9•2 years ago
|
||
| bugherder | ||
Description
•