Some random code cleanup:

1)  Make SetContentSink and SetParserFilter not claim to return stuff
2)  Clean up nsIParser to use NS_IMETHOD_() where needed.  Sync up with nsParser
3)  Removed some never-used code

Bug 40149, r=harishd, sr=jst


git-svn-id: svn://10.0.0.236/trunk@140868 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/content/html/document/src/nsHTMLDocument.cpp

@@ -734,11 +734,8 @@ nsHTMLDocument::StartAutodetection(nsIDocShell *aDocShell, nsAString& aCharset,
           rv_detect = adp->Init(wss, cdet, this, mParser,
                                 PromiseFlatString(aCharset).get(), aCommand);
 
-          // The current implementation for SetParserFilter needs to
-          // be changed to be more XPCOM friendly. See bug #40149
           if (mParser)
-            nsCOMPtr<nsIParserFilter> oldFilter =
-              getter_AddRefs(mParser->SetParserFilter(cdetflt));
+            mParser->SetParserFilter(cdetflt);
         }
       }
     }