Closed Bug 1464711 Opened 4 years ago Closed 4 years ago
Typo in Extension
Content Script .requires Cleanup()
47 bytes, text/x-phabricator-request
|Details | Review|
Typescript check found that property removeCss doesn't exist: https://searchfox.org/mozilla-central/rev/5a74471/toolkit/components/extensions/ExtensionContent.jsm#306
Hello, I'm an absolute beginner and would like to work on it.
Devika, thanks for your interest! Unfortunately, I don't think this is a particularly good first bug. In particular, removeCss looks like a mis-capitalization of removeCSS so it should probably be changed. But the fact that everything is working with that typo in place suggests we don't have very good test coverage here so ideally we would improve the test coverage at the same time. If you're interested in looking into that, feel free but since you say you're an absolute beginner, I would encourage you to look at bugs with the good-first-bug keyword.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/ca8f8514f60d Fix typo in ExtensionContent script.requiresCleanup getter. r=zombie
You need to log in before you can comment on or make changes to this bug.