Remove the no-forward-declared-nsCOMPtr hack (bug 107291), r=bugmail@sicking.cc, sr=dbaron@dbaron.org

layout/forms/nsComboboxControlFrame.h

@@ -51,7 +51,6 @@
 //#define DO_NEW_REFLOW
 #endif
 
-#include "nsIDOMMouseListener.h"
 #include "nsAreaFrame.h"
 #include "nsIFormControlFrame.h"
 #include "nsIComboboxControlFrame.h"
@@ -65,6 +64,7 @@
 #include "nsIScrollableViewProvider.h"
 #include "nsIStatefulFrame.h"
 
+class nsIDOMMouseListener;
 class nsIView;
 class nsStyleContext;
 class nsIHTMLContent;

layout/html/forms/src/nsComboboxControlFrame.h

@@ -51,7 +51,6 @@
 //#define DO_NEW_REFLOW
 #endif
 
-#include "nsIDOMMouseListener.h"
 #include "nsAreaFrame.h"
 #include "nsIFormControlFrame.h"
 #include "nsIComboboxControlFrame.h"
@@ -65,6 +64,7 @@
 #include "nsIScrollableViewProvider.h"
 #include "nsIStatefulFrame.h"
 
+class nsIDOMMouseListener;
 class nsIView;
 class nsStyleContext;
 class nsIHTMLContent;

xpcom/glue/nsCOMPtr.h

@@ -449,17 +449,6 @@ class nsCOMPtr
     : private nsCOMPtr_base
 #endif
   {
-    enum { _force_even_compliant_compilers_to_fail_ = sizeof(T) };
-      /*
-        The declaration above exists specifically to make |nsCOMPtr<T>| _not_ compile with only
-        a forward declaration of |T|.  This should prevent Windows and Mac engineers from
-        breaking Solaris and other compilers that naturally have this behavior.  Thank
-        <law@netscape.com> for inventing this specific trick.
-
-        Of course, if you're using |nsCOMPtr| outside the scope of wanting to compile on
-        Solaris and old GCC, you probably want to remove the enum so you can exploit forward
-        declarations.
-      */
 
 #ifdef NSCAP_FEATURE_DEBUG_PTR_TYPES
     private: