Closed
Bug 1148759
Opened 10 years ago
Closed 10 years ago
Printing a promise result of generator ceases the execution
Categories
(DevTools :: Console, defect)
Tracking
(firefox36 affected, firefox37 ?, firefox38 ?, firefox39 ?, firefox40 fixed)
RESOLVED
FIXED
Firefox 40
People
(Reporter: czerny.jakub, Assigned: fitzgen)
References
Details
(Keywords: regression)
Attachments
(2 files)
758 bytes,
text/plain
|
Details | |
4.43 KB,
patch
|
past
:
review+
|
Details | Diff | Splinter Review |
First two block in attachment works ok. The last (third) block prints just
> "before"
but node.js (0.12 with --harmony flag) outputs
> before
> { value: {}, done: false }
> after
> { value: undefined, done: true }
as expected. I.e. line
> console.log(result);
ceases the execution.
Comment 1•10 years ago
|
||
Following error is shown in browser console.
TypeError: this.objectActor.preview is undefined
> let { ownProperties, safeGetterValues } = this.objectActor.preview;
Last good revision: 77b00f3cc7c7
First bad revision: 2add539cd09a
Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=77b00f3cc7c7&tochange=2add539cd09a
maybe a regression from bug 1033153? it adds the code.
https://hg.mozilla.org/mozilla-central/rev/2548cc70c191
Flags: needinfo?(nfitzgerald)
Comment 2•10 years ago
|
||
Simpler testcase:
console.log({ x: Promise.resolve() });
So basically the problem is in prettyprinting an object with a property value that's a Promise.
Blocks: 1033153
Component: JavaScript Engine → Developer Tools: Console
Keywords: regression
Product: Core → Firefox
Comment 3•10 years ago
|
||
Or even simpler, open the console and type:
({ x: Promise.resolve() })
Assignee: nobody → nfitzgerald
Assignee | ||
Updated•10 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(nfitzgerald)
Reg range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=8230834302c9&tochange=a255a234946e
status-firefox36:
--- → affected
status-firefox37:
--- → ?
status-firefox38:
--- → ?
status-firefox39:
--- → ?
Version: 39 Branch → 36 Branch
Assignee | ||
Comment 5•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Attachment #8588925 -
Flags: review?(past)
Updated•10 years ago
|
Attachment #8588925 -
Flags: review?(past) → review+
Assignee | ||
Comment 6•10 years ago
|
||
Comment 7•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox40:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•