remove JSScript::isAboutToBeFinalized

RESOLVED FIXED in mozilla9

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Igor Bukanov, Assigned: Igor Bukanov)

Tracking

Trunk
mozilla9
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
We should remove JSScrit::isAboutToBeFinalized. With the bug 674251 fixed the generic IsAboutToBeFinalized should work for JSScript instances.
Bug 663138 removes this and will be landing soon, so an option is to just wait for that bug.
Depends on: 663138
(Assignee)

Comment 2

6 years ago
Created attachment 557616 [details] [diff] [review]
v1

Here is a straightforward removal of JSScript::isAboutToBeFinalized().
Assignee: general → igor
Attachment #557616 - Flags: review?(bhackett1024)
Attachment #557616 - Flags: review?(bhackett1024) → review+
(Assignee)

Comment 3

6 years ago
(In reply to Brian Hackett from comment #1)
> Bug 663138 removes this and will be landing soon, so an option is to just
> wait for that bug.

I need this to eliminate JSScript::u.object that should simplify the debugger implementation.
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/30ffa45f9a63
http://hg.mozilla.org/mozilla-central/rev/30ffa45f9a63
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9

Updated

6 years ago
Summary: remove JSScrit::isAboutToBeFinalized → remove JSScript::isAboutToBeFinalized
You need to log in before you can comment on or make changes to this bug.