when Console is running in the main thread the existence of mWindow should always be ensured

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
4 months ago

People

(Reporter: rpl, Assigned: baku)

Tracking

unspecified
mozilla48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

(Whiteboard: btpp-active)

Attachments

(1 attachment)

No description provided.
Assignee: nobody → amarchesini
Are you working on this, baku?
Flags: needinfo?(amarchesini)
Yes, I asked Luca to open this bug and to assign it to me.
Flags: needinfo?(amarchesini)
Posted patch console.patchSplinter Review
Attachment #8739727 - Flags: review?(bugs)
Comment on attachment 8739727 [details] [diff] [review]
console.patch

Do we not ever use those ctors in main thread without window? Like, we don't expose Console to JS components?
Comment on attachment 8739727 [details] [diff] [review]
console.patch

based on the webidl that is the case. Only window context has Console.
Attachment #8739727 - Flags: review?(bugs) → review+
> based on the webidl that is the case. Only window context has Console.

And workers, but they don't run on the main-thread.
Whiteboard: btpp-active
https://hg.mozilla.org/mozilla-central/rev/474f2f3c1059
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.