Closed Bug 867414 Opened 7 years ago Closed 6 years ago

Rename SafeAutoJSContext to AutoSafeJSContext

Categories

(Core :: XPConnect, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: bholley, Assigned: sankha)

Details

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

Attachments

(1 file)

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.
Attached patch Patch v1Splinter Review
Can you review this patch?
Assignee: nobody → sankha93
Attachment #745652 - Flags: review?(bobbyholley+bmo)
Comment on attachment 745652 [details] [diff] [review]
Patch v1

Looks great! r=bholley assuming it compiles.
Attachment #745652 - Flags: review?(bobbyholley+bmo) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a0974c8e1e53
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.