Remove aliased properties (JS_AliasProperty, JS_AliasElement, etc.)

RESOLVED FIXED in mozilla8

Status

()

Core
JavaScript Engine
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: jorendorff, Assigned: Waldo)

Tracking

Other Branch
mozilla8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
In bug 518633, comment 19, Brendan writes:

  Historical note: aliases were in JS from day 1, never made it into ES1. They
  were for document.forms[0] and document.forms.myFirstForm aliasing. We should
  try to get rid of them, I may have asked for a separate bug already.
Created attachment 550534 [details] [diff] [review]
1 - Remove the public-facing aliasing APIs

Yes, those are the only users now.
Assignee: general → jwalden+bmo
Status: NEW → ASSIGNED
Attachment #550534 - Flags: review?(jorendorff)
Created attachment 550535 [details] [diff] [review]
2 - Remove the aliasing implementation
Attachment #550535 - Flags: review?(jorendorff)
(Reporter)

Updated

6 years ago
Attachment #550534 - Flags: review?(jorendorff) → review+
(Reporter)

Comment 3

6 years ago
Comment on attachment 550535 [details] [diff] [review]
2 - Remove the aliasing implementation

Great.
Attachment #550535 - Flags: review?(jorendorff) → review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/409145fb4346
http://hg.mozilla.org/integration/mozilla-inbound/rev/6ef18ea1911a
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
http://hg.mozilla.org/mozilla-central/rev/409145fb4346
http://hg.mozilla.org/mozilla-central/rev/6ef18ea1911a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.