1522052 - Remove nsIDOMChromeWindow

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P2)

Description

[Peter Van der Beken [:peterv]]

Bug 1383059 comment 8 suggested marking nsIDOMChromeWindow as non-scriptable, but we can just remove it.

Comment 1

[Peter Van der Beken [:peterv]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=62342d1bc57c560dcb25b4307f73bcea79bf576f

Comment 2

[Boris Zbarsky [:bzbarsky]]

Jorg, there are some uses of nsIDOMChromeWindow in calendar and mail that you may need to deal with.

Comment 3

[Jorg K (CEST = GMT+2)]

Thanks for the heads-up. It's also used in mail/ and mailnews/:
https://dxr.mozilla.org/comm-central/search?q=nsIDOMChromeWindow+path%3Acomm%2F&redirect=false
Boris, could you comment on those mail/ and mailnews/ usages over in bug 1522158. I guess #include "nsIDOMChromeWindow.h" is unused an can be removed.

Comment 4

[Neha Kochar [:neha]]

Comment 3 has a ques for you, Boris.

Comment 5

[Boris Zbarsky [:bzbarsky]]

I commented in bug 1522158.. And yes, includes of nsIDOMChromeWindow.h can just be removed.

Comment 6

[Peter Van der Beken [:peterv]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed1c3f7616ae69c0cf6ab79e85a4504d0622d14e

Comment 7

[Peter Van der Beken [:peterv]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b35a1efde318e8233dd77a5f5c4fc6b16d3e538

Comment 8

[Peter Van der Beken [:peterv]]

Attachment: Bug 1522052 - remove nsIDOMChromeWindow. r?#dom-core!

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/801ba619d741
remove nsIDOMChromeWindow. r=dom-core,webdriver-reviewers,devtools-reviewers,whimboo,mccr8

Comment 10

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/801ba619d741