The following idl files forward-declare the interfaces they're defining for no apparent reason: xpcom/io/nsILineInputStream.idl toolkit/profile/nsIToolkitProfile.idl dom/interfaces/traversal/nsIDOMTreeWalker.idl dom/interfaces/traversal/nsIDOMNodeIterator.idl (to clarify, for example, see nsILineInputStream.idl: ... 6 7 #include "nsISupports.idl" 8 9 interface nsILineInputStream; 10 11 [scriptable, uuid(c97b466c-1e6e-4773-a4ab-2b2b3190a7a6)] 12 interface nsILineInputStream : nsISupports 13 { ... where nsILineInputStream is forward-declared for no reason that I see.)

This patch just removes the forward declarations (and some blank lines I didn't think were necessary). bsmedberg - would you be able to have a look?

Comment on attachment 791532 [details] [diff] [review] patch Justin - this is what I pinged you about earlier. I also found a few more instances of the same thing. If this all looks good, I'll need sign-off from a DOM peer, I assume. Thanks!

Here's a try run, just to be thorough: https://tbpl.mozilla.org/?tree=Try&rev=10c310c7b5bf

Comment on attachment 791532 [details] [diff] [review] patch sgtm if it compiles.

Thanks! https://hg.mozilla.org/integration/mozilla-inbound/rev/de5b7c7741e1