Fix browsingContext.sys.mjs to assign to #contextCreatedHandled instead of contextCreatedHandled
| Tracking | Status | |
|---|---|---|
| firefox152 | --- | fixed |
| Reporter | |
Descriptionβ’2 months ago
|
constructor(messageHandler) {
super(messageHandler);
// Setup the LoadListener as early as possible.
this.#loadListener = new lazy.LoadListener(this.messageHandler.window);
this.#loadListener.on("DOMContentLoaded", this.#onDOMContentLoaded);
this.#loadListener.on("load", this.#onLoad);
// Set of event names which have active subscriptions.
this.#subscribedEvents = new Set();
this.contextCreatedHandled = false;
}
We should simply update contextCreatedHandled to be #contextCreatedHandled here. Completely straightforward, if anyone wants to use this to setup their environment, this is a good first bug.
| Reporter | |
Updatedβ’1 month ago
|
Comment 1β’1 month ago
|
Hi Julian, Iβm new to Firefox contribution and Iβd like to work on this bug.
I checked that it is unassigned, and Iβd like to take it if thatβs okay. Could I be assigned to it?
Thanks!
| Reporter | |
Comment 2β’1 month ago
|
(In reply to Amin Amir from comment #1)
Hi Julian, Iβm new to Firefox contribution and Iβd like to work on this bug.
I checked that it is unassigned, and Iβd like to take it if thatβs okay. Could I be assigned to it?
Thanks!
Hi Amin, the bug will automatically be assigned to you when you submit a patch on Phabricator.
Take a look at https://firefox-source-docs.mozilla.org/setup/index.html to setup your environment and let us know if you have any question!
| Assignee | |
Comment 3β’1 month ago
|
Updatedβ’1 month ago
|
Comment 4β’1 month ago
|
Comment 5β’1 month ago
|
|
| bugherder | |
Updatedβ’1 month ago
|
Updatedβ’8 days ago
|
