Remove workaround for getOwnPropertyDescriptor from chrome code

RESOLVED FIXED in Firefox 8

Status

()

Firefox
Developer Tools
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: past, Assigned: past)

Tracking

Trunk
Firefox 8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

In bug 651501 I used the pattern scope.Object.getOwnPropertyDescriptor to work around a platform bug. Since bug 664689 was fixed we no longer need this, which would simplify our code a bit.
Assignee: nobody → past
Status: NEW → ASSIGNED
Created attachment 544273 [details] [diff] [review]
[in-fx-team] Patch v1
Attachment #544273 - Flags: review?(mihai.sucan)
Comment on attachment 544273 [details] [diff] [review]
[in-fx-team] Patch v1

This looks fine. All tests pass locally!
Attachment #544273 - Flags: review?(mihai.sucan) → review+
Comment on attachment 544273 [details] [diff] [review]
[in-fx-team] Patch v1

Thank you Mihai!
Dave can you review this cleanup of the patch you reviewed for bug 651501?
Attachment #544273 - Flags: review?(dtownsend)
Attachment #544273 - Flags: review?(dtownsend) → review+
Whiteboard: [land-in-fx-team]
Comment on attachment 544273 [details] [diff] [review]
[in-fx-team] Patch v1

Pushed to:

http://hg.mozilla.org/integration/fx-team/rev/bad869718019
Attachment #544273 - Attachment description: Patch v1 → [in-fx-team] Patch v1

Updated

6 years ago
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 8
http://hg.mozilla.org/mozilla-central/rev/bad869718019
You need to log in before you can comment on or make changes to this bug.