Closed
Bug 1737482
Opened 3 years ago
Closed 3 years ago
Object inspector doesn't show properties and prototype of promises
Categories
(DevTools :: Object Inspector, defect)
Tracking
(firefox-esr78 unaffected, firefox-esr91 wontfix, firefox93 wontfix, firefox94 wontfix, firefox95 fixed)
RESOLVED
FIXED
95 Branch
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox-esr91 | --- | wontfix |
firefox93 | --- | wontfix |
firefox94 | --- | wontfix |
firefox95 | --- | fixed |
People
(Reporter: Oriol, Assigned: Oriol)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
What were you doing?
Enter this code into the console:
inspect(Object.assign(Promise.resolve(123), {foo: 1}))
What happened?
▼ Promise { <state>: "fulfilled", <value>: 123 }
│ <state>: "fulfilled"
│ <value>: 123
What should have happened?
▼ Promise { <state>: "fulfilled", <value>: 123 }
│ foo: 1
│ <state>: "fulfilled"
│ <value>: 123
│ <prototype>: Promise.prototype { … }
Anything else we should know?
I broke this in bug 1552648, I guess I just copied what was being done for proxies, which are not supposed to show properties nor prototype. And I didn't notice the change since I was testing promises with a null prototype (in order to nest them), and <prototype>
is not shown for a null prototype (bug 1612127).
Assignee | ||
Comment 1•3 years ago
|
||
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/32ab6b8790bb [devtools] Include properties and prototype when inspecting promise. r=nchevobbe
Comment 3•3 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox95:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
Updated•3 years ago
|
Keywords: regression
Comment 4•3 years ago
|
||
Set release status flags based on info from the regressing bug 1552648
status-firefox93:
--- → affected
status-firefox94:
--- → affected
status-firefox-esr78:
--- → unaffected
status-firefox-esr91:
--- → affected
Updated•3 years ago
|
Flags: in-testsuite+
Updated•3 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•