865664 - Remove deprecated nsIDOMWindowUtils.getOuterWindowWithId

Status: RESOLVED FIXED

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

Description

[Colby Russell :crussell (no longer Mozilla-ing)]

See bug 861495.

Target sometime between Gecko 24 (next ESR) and Gecko 31 (the one after that)?

Comment 1

[Colby Russell :crussell (no longer Mozilla-ing)]

(In reply to Boris Zbarsky (:bz) from bug 861495, comment #3)
> https://mxr.mozilla.org/addons/
> search?string=getOuterWindowWithID&find=&findi=&filter=^[^\0]*%24&hitlimit=&t
> ree=addons says "Found 23 matching lines in 14 files".

Boris, can you paste the names of those add-ons here?

Comment 2

[Boris Zbarsky [:bzbarsky]]

Names are actually a pain to get reliably via that interface.  I'd have to go grovel through the source of each addon and hope...

As best as I can make it, we're talking: Firebug, "System Monitor", "Workspace for Thunderbird", "Testing d2c for platform-specific files" (?), Nulltxt, "Firefox OS Simulator".  Some of these have multiple uses.

Comment 3

[Masatoshi Kimura [:emk]]

Attachment: Bug 865664 - Remove deprecated nsIDOMWindowUtils.getOuterWindowWithId.

Review commit: https://reviewboard.mozilla.org/r/165316/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/165316/

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8894234 [details]
Bug 865664 - Remove deprecated nsIDOMWindowUtils.getOuterWindowWithId.

https://reviewboard.mozilla.org/r/165316/#review170920

r=me, but please hold off on landing this until legacy addons are disabled, so there is no weird breakage for nightly users.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc357f0e9f40
Remove deprecated nsIDOMWindowUtils.getOuterWindowWithId. r=bz

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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