Closed Bug 1123112 Opened 6 years ago Closed 6 years ago

Mark AutoEntryScript as stack class, and clarify the ownership of its members; r=smaug

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla38

People

(Reporter: ehsan, Assigned: ehsan)

Details

Attachments

(1 file)

No description provided.
Assignee: nobody → ehsan
Comment on attachment 8550947 [details] [diff] [review]
Mark AutoEntryScript as stack class, and clarify the ownership of its members

This should be fine, since mDocShellForJSRunToCompletion is set only when
gRunToCompletionListeners > 0. And the code in such case is already addref/release heavy.
Attachment #8550947 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/953aecc38f56
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.