Closed Bug 523438 Opened 10 years ago Closed 10 years ago

Make nsFrameLoader* available to content/layout

Categories

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

x86
Windows NT
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: benjamin, Assigned: benjamin)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

For e10s we're hanging additional methods off of nsFrameLoader. In my initial patches I was static_cast<nsFrameLoader*> from nsIFrameLoader*, but bz suggested that we should just have a getter and avoid the casts. This patch will hang a noscript nsFrameLoader* getter on nsIFrameLoaderOwner.
Attachment #407361 - Flags: review?(bzbarsky)
Attachment #407361 - Flags: review?(bzbarsky) → review+
Comment on attachment 407361 [details] [diff] [review]
native nsIFrameLoader.GetFrameLoader, rev. 1

I'd prefer we not NS_IF_ADDREF nsCOMPtrs.  Please .get() into a local before addreffing and returning that local?
http://hg.mozilla.org/mozilla-central/rev/ab4333859900
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.