WebExtensions/Contribution Onramp
If this is your first contribution to the Firefox codebase, welcome! It can be scary to get started contributing to a large and complex codebase, but we’ve prepared some tips to help you fix a good-first-bug and land a patch in Firefox.
If we haven’t addressed your question in this document, please ask us by leaving a comment in the bug or by joining us on chat.mozilla.org in the #addons room (see Matrix). For a general overview of how to contribute to Firefox, please go here.
Contents
General Tips
- If you see a bug that looks interesting, please leave a comment saying that you would like to work on it, or upload your patch to phabricator and request review. Note: we will only assign the bug to you when a patch is uploaded.
- It's ok to ask questions in Bugzilla comments! It's best if you ask specific questions, like, "Can you help me find the line of code that needs changing," and not general questions like, "can you help me."
- It's normal to try things out and get stuck when you're working on your bug. When that happens, we encourage you to try a few different approaches before you ask for help. A good rule of thumb is to try to only needinfo your mentor once a day.
- It might take mentors a day or two to get back to you. If you haven't heard a response in more than two days, please feel free to ping them on chat.mozilla.org in the #addons room (see Matrix).
Getting Started
- If you haven't already done so, create an account on https://bugzilla.mozilla.org.
- Pick a bug you want to work on from the good-first-bugs list.
- Comment on the bug that you would like to work on it and proceed to set up your developer environment.
- Run unit tests to verify that your environment works as expected, preferably tests related to the bug that you want to work on.
Setting Up Developer Environment
For a detailed overview, see Getting Set Up To Work On The Firefox Codebase.
In short:
- Now, you need to set up your development environment by cloning the source with mercurial or with Git.
- Then you need to install dependencies with
mach bootstrap
for building Firefox. There are two ways to do this:- [recommended] Build Firefox using Firefox Artifact builds. If the issue only requires changes to privileged JavaScript code, we recommend you use this method. Artifact builds saves time because it downloads a pre-built version of Firefox sources and applies your local changes on it, instead of rebuilding all of the C++ and Rust pieces from scratch.
- To build Firefox for Android, see the GeckoView Contributor guide. For a faster local development experience, we recommend to use the Artifact builds and the Android x86 target (which is especially useful to run the tests locally).
- [default] Build Firefox from scratch, by running
mach build
without .mozconfig. Note: doing this can take some time.
- [recommended] Build Firefox using Firefox Artifact builds. If the issue only requires changes to privileged JavaScript code, we recommend you use this method. Artifact builds saves time because it downloads a pre-built version of Firefox sources and applies your local changes on it, instead of rebuilding all of the C++ and Rust pieces from scratch.
- Next, use Searchfox to find the code associated with your bug.
- Before making any changes, use
mach test [path to extension test]
to verify that you are able to run unit tests. Being able to run existing tests (and modified or new tests) is necessary, to see that your change works as expected.
WebExtensions APIs
You can learn more about how WebExtensions internals are organized and how to work on them by reading this internal API documentation and the WebExtensions Hacking wiki.
Fixing the Bug!
- You get to drive the actual bug fix. :)
- We encourage you to check in with the mentor to make sure you are on the right path, or ask for feedback on a patch in progress. You can attach your patch to a comment in Bugzilla and needinfo the mentor for review.
Testing
In order to land code in Firefox, you will need to create tests for your patches. For an overview of the different test systems Firefox uses, please see this overview of automated testing. In the WebExtension API code, most tests are either xpcshell tests or (browser-chrome) mochitests. Xpcshell tests are preferred because of the lower overhead. Browser tests need to be written when a test interfaces with browser UI, such as tabs or context menus. Browser tests are not supported on mobile; use mochitests if you need to run tests with a real browser on desktop and mobile.
The easiest way to get started with tests is to look at existing tests:
- Xpcshell tests:
- toolkit/components/extensions/test/xpcshell/ (Searchfox link)
- browser/components/extensions/test/xpcshell/ (Searchfox link)
- mobile/shared/components/extensions/test/xpcshell (Searchfox link)
- Browser-chrome tests:
- toolkit/components/extensions/test/browser/ (Searchfox link)
- browser/components/extensions/test/browser/ (Searchfox link)
- Mochitests:
- toolkit/components/extensions/test/mochitest/ (Searchfox link)
- Firefox for Android mochitests:
- mobile/shared/components/extensions/test/mochitest (Searchfox link)
Running tests
These tests can be run with the mach test
command. To run specific test(s), pass the name of the test file or directory to mach test
.
Sometimes mach test
has bugs (e.g. bug 1498636 and bug 1495311); in that case use mach xpcshell-test
or mach mochitest
instead of mach test
to run the test.
mach test
has minimal output by default. To debug test failures, increase the verbosity of the logs with the --verbose --log-mach-verbose
flags. The --sequential
flag is not required, but allows for easier debugging because only one test is run at a time (instead of multiple in parallel). Al together, the command to run tests and get maximum debug output is:
mach test --verbose --log-mach-verbose --sequential [paths to test files]
Adding new tests
Add your test to an existing file when it is small and fits in that test. Otherwise, create a new test file, and register the test file in one of the .toml files in the same directory as your test, usually browser.toml or xpcshell-common.toml.
After creating a new test file, run mach build
before running tests. It is not necessary to rebuild if you are changing an existing test file.
A test consists of multiple subtasks, which are defined in the test files via add_task(...). Most tests have the following minimal structure:
add_task(async function someShortAndSimpleDescriptionHere() { // Define a test extension. let extension = ExtensionTestUtils.loadExtension( .... ); // Loads the actual extension. await extension.startup(); // Tests here, often using extension.sendMessage and/or extension.awaitMessage // Unload the extension and remove the temporary files. await extension.unload(); });
ExtensionTestUtils.loadExtension allows you to quickly generate a test extension, and its input parameters are documented at: https://searchfox.org/mozilla-central/rev/aecb006bbb135d707ca4b8cea7572dd8abab6817/toolkit/components/extensions/ExtensionTestCommon.sys.mjs#483-507
Another common action in tests is to load a web page or extension page. In xpcshell-tests, use ExtensionTestUtils.loadContentPage.
In browser-chrome tests, the BrowserTestUtils namespace provides many useful test helpers, such as BrowserTestUtils.openNewForegroundTab to open a new tab. BrowserTestUtils is implemented and documented at https://searchfox.org/mozilla-central/source/testing/mochitest/BrowserTestUtils/BrowserTestUtils.sys.mjs
Running the tests (xpcshell tests and Firefox for Android mochitests) on the Android emulator requires some additional setup of the local development environment.
Linting
Firefox source code have a consistent formatting and code style, which is enforced through linting. Most extension code is written with JavaScript, which is linted via eslint. It is recommended to set up and run the linter locally (see eslint docs). When a patch has been submitted to Phabricator, static analysis will run and report style errors upon completion (which may take a while to run). The Try Server also offers linting (ES for eslint, per WebExtensions/Try Server).
You can run linters on individual files as follows:
./mach lint [path(s) to file(s) and/or directories]
You can also run the linter on all local changes:
./mach lint --outgoing
Submitting a Patch
For an overview of how to submit a patch, please see How to submit a patch. In short:
- Create a commit with a meaningful commit message, for example
Bug [bug ID here] - [short description of patch here] r=[reviewer name here]
.- Use
hg com --amend
orgit commit --amend
to modify the last commit if you want to change the commit title or content. - If you want to combine multiple commits into one:
- With mercurial, use
hg histedit
orhg rebase
. For more info, see Mercurial - Changing history - With git, use
git rebase -i
. For more info, see Git - Rewriting History.
- With mercurial, use
- Use
- When you are ready to have your patch reviewed, submit your patch to Phabricator:
- First, create an account and install the moz-phab tool.
- Then run moz-phab to upload your commit. Each commit will become a separe patch, so combine them into one if needed.
- Note that you may go through a few iterations of changes and review before your code is ready to go.
- If you need to make a change to a patch that has already been submitted to Phabricator, make sure you update the existing revision so the reviewers can check the interdiff and history of changes.
moz-phab
will automatically modify your commit to add the relevant link. - When working on unrelated changes at the same time, it is useful to work on them into separate hg bookmarks (or entire separate mozilla-central clones if you are not yet feeling confident enough with how the hg bookmarks works) or git branches.
- When the patch is approved, your mentor will use Lando to land the patch.
- Once your patch is accepted, you can expect to see your code in an upcoming version of Firefox!
Try server
To verify that the patch works as intended, your mentor may schedule a try job on the Try Server to run tests. Contributors with access to the try server can push try jobs to run tests themselves.
See WebExtensions/Try_Server for more information about using the try server to test WebExtension code.