842986 - Implement MozFrameLoaderOwner on HTMLFrameElement

Status: RESOLVED FIXED

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

Description

[Boris Zbarsky [:bzbarsky]]

See bug 838343 comment 4.

We're going to need to backport this to Aurora...

Comment 1

[Andrea Marchesini [:baku]]

Attachment: patch

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 717156 [details] [diff] [review]
patch

Why not put this function on nsGenericHTMLFrameElement so iframe can just share it?

r=me with that changed.

Comment 4

[Andrea Marchesini [:baku]]

Attachment: patch

A quick review?

Comment 5

[Andrea Marchesini [:baku]]

https://tbpl.mozilla.org/?tree=Try&rev=b10a49b546e4

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 717532 [details] [diff] [review]
patch

r=me

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f5481770263d

Comment 8

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/f5481770263d

Comment 9

[Andrea Marchesini [:baku]]

Comment on attachment 717532 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 838343 
User impact if declined:mozFrameLoaderOwner interface will miss from HTMLFrameElement
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): no one. try server is green and all the existing tests are passed.
String or UUID changes made by this patch:

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/a4dc2d408983