Comment on attachment 746069 [details] [diff] [review] Move class into a header v1 r=me

Comment on attachment 746071 [details] [diff] [review] Switch to WebIDL v1 >+interface ImageDocument : HTMLDocument { [NoInterfaceObject], please. Might be worth a followup to make these all [ChromeOnly], or at least try to. r=me with that.

Comment on attachment 746072 [details] [diff] [review] Move PostCreateWrapper into nsIDocument::WrapObject v1 r=me This needs one more patch to drop the hasXPConnectImpls bits from HTMLDocument and Document in the conf file, right?

Comment on attachment 746069 [details] [diff] [review] Move class into a header v1 >--- a/content/html/document/src/ImageDocument.cpp >+++ b/content/html/document/src/ImageDocument.h >@@ -1,78 +1,27 @@ > /* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */ > /* This Source Code Form is subject to the terms of the Mozilla Public > * License, v. 2.0. If a copy of the MPL was not distributed with this > * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ >+#ifndef ImageDocument_h___ >+#define ImageDocument_h___ Please do not add a new symbol containing two (or more) consecutive underscores.

(In reply to Boris Zbarsky (:bz) from comment #5) > [NoInterfaceObject], please. > > Might be worth a followup to make these all [ChromeOnly], or at least try to. There's one place in chrome where we |instanceof nsIImageDocument|. It'd be nice to be able to remove that interface (it's unused in our code otherwise) but still allow chrome to do something like that. Maybe we should have ChromeOnly interface objects (and make that mean "all properties are ChromeOnly too")?

And a few places where we do |instanceof ImageDocument|, fwiw

Comment on attachment 746307 [details] [diff] [review] Drop hasXPConnectImpls from documents v1 r=me, yay!

Er, wait. Why is there a window.ImageDocument right now, if the interface is not named nsIDOMImageDocument? Is this more classinfo suck? :( If we already have a window.ImageDocument, I guess we're at least not making things worse... but yes, having a way to flag the entire interface as ChromeOnly would be nice.

https://hg.mozilla.org/integration/mozilla-inbound/rev/249001348a81 https://hg.mozilla.org/integration/mozilla-inbound/rev/1f162188fed4 https://hg.mozilla.org/integration/mozilla-inbound/rev/b87e943e0ab3 https://hg.mozilla.org/integration/mozilla-inbound/rev/9a86629a8fa2 (In reply to Boris Zbarsky (:bz) from comment #12) > Er, wait. Why is there a window.ImageDocument right now, if the interface > is not named nsIDOMImageDocument? Is this more classinfo suck? :( CutPrefix deals with both nsIDOM* and nsI* prefixes, and we apply it to all interfaces on a primary prototype chain I think.

https://hg.mozilla.org/mozilla-central/rev/249001348a81 https://hg.mozilla.org/mozilla-central/rev/1f162188fed4 https://hg.mozilla.org/mozilla-central/rev/b87e943e0ab3 https://hg.mozilla.org/mozilla-central/rev/9a86629a8fa2