Rename SafeAutoJSContext to AutoSafeJSContext

RESOLVED FIXED in mozilla23

Status

()

Core
XPConnect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: bholley, Assigned: sankha)

Tracking

unspecified
mozilla23
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mentor=bholley][lang=c++])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Now that dom:workers::AutoSafeJSContext is gone, we can give mozilla::SafeAutoJSContext the name it truly deserves: AutoSafeJSContext.

This should be a simple s/SafeAutoJSContext/AutoSafeJSContext/g in the tree.
(Assignee)

Comment 1

5 years ago
Created attachment 745652 [details] [diff] [review]
Patch v1

Can you review this patch?
Assignee: nobody → sankha93
Attachment #745652 - Flags: review?(bobbyholley+bmo)
(Reporter)

Comment 2

5 years ago
Comment on attachment 745652 [details] [diff] [review]
Patch v1

Looks great! r=bholley assuming it compiles.
Attachment #745652 - Flags: review?(bobbyholley+bmo) → review+
(Reporter)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a0974c8e1e53
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.