diff --git a/mozilla/htmlparser/robot/nsRobotSink.cpp b/mozilla/htmlparser/robot/nsRobotSink.cpp index 779bf9c68f3..5e73fc466e6 100644 --- a/mozilla/htmlparser/robot/nsRobotSink.cpp +++ b/mozilla/htmlparser/robot/nsRobotSink.cpp @@ -34,7 +34,7 @@ class nsIDocument; // - LAYER SRC static NS_DEFINE_IID(kISupportsIID, NS_ISUPPORTS_IID); -static NS_DEFINE_IID(kIHTMLContentSinkIID, NS_IHTMLCONTENTSINK_IID); +static NS_DEFINE_IID(kIHTMLContentSinkIID, NS_IHTML_CONTENT_SINK_IID); static NS_DEFINE_IID(kIRobotSinkIID, NS_IROBOTSINK_IID); class RobotSink : public nsIRobotSink { @@ -52,25 +52,28 @@ public: NS_DECL_ISUPPORTS // nsIHTMLContentSink - virtual PRInt32 SetTitle(const nsString& aValue); - virtual PRInt32 OpenHTML(const nsIParserNode& aNode); - virtual PRInt32 CloseHTML(const nsIParserNode& aNode); - virtual PRInt32 OpenHead(const nsIParserNode& aNode); - virtual PRInt32 CloseHead(const nsIParserNode& aNode); - virtual PRInt32 OpenBody(const nsIParserNode& aNode); - virtual PRInt32 CloseBody(const nsIParserNode& aNode); - virtual PRInt32 OpenForm(const nsIParserNode& aNode); - virtual PRInt32 CloseForm(const nsIParserNode& aNode); - virtual PRInt32 OpenFrameset(const nsIParserNode& aNode); - virtual PRInt32 CloseFrameset(const nsIParserNode& aNode); - virtual PRInt32 OpenContainer(const nsIParserNode& aNode); - virtual PRInt32 CloseContainer(const nsIParserNode& aNode); - virtual PRInt32 CloseTopmostContainer(); - virtual PRInt32 AddLeaf(const nsIParserNode& aNode); - virtual void WillBuildModel(void) { } - virtual void DidBuildModel(PRInt32 aQualityLevel) { } - virtual void WillInterrupt(void) { } - virtual void WillResume(void) { } + NS_IMETHOD PushMark(); + NS_IMETHOD SetTitle(const nsString& aValue); + NS_IMETHOD OpenHTML(const nsIParserNode& aNode); + NS_IMETHOD CloseHTML(const nsIParserNode& aNode); + NS_IMETHOD OpenHead(const nsIParserNode& aNode); + NS_IMETHOD CloseHead(const nsIParserNode& aNode); + NS_IMETHOD OpenBody(const nsIParserNode& aNode); + NS_IMETHOD CloseBody(const nsIParserNode& aNode); + NS_IMETHOD OpenForm(const nsIParserNode& aNode); + NS_IMETHOD CloseForm(const nsIParserNode& aNode); + NS_IMETHOD OpenMap(const nsIParserNode& aNode); + NS_IMETHOD CloseMap(const nsIParserNode& aNode); + NS_IMETHOD OpenFrameset(const nsIParserNode& aNode); + NS_IMETHOD CloseFrameset(const nsIParserNode& aNode); + NS_IMETHOD OpenContainer(const nsIParserNode& aNode); + NS_IMETHOD CloseContainer(const nsIParserNode& aNode); + NS_IMETHOD CloseTopmostContainer(); + NS_IMETHOD AddLeaf(const nsIParserNode& aNode); + NS_IMETHOD WillBuildModel(void) { return NS_OK; } + NS_IMETHOD DidBuildModel(PRInt32 aQualityLevel) { return NS_OK; } + NS_IMETHOD WillInterrupt(void) { return NS_OK; } + NS_IMETHOD WillResume(void) { return NS_OK; } // nsIRobotSink NS_IMETHOD Init(nsIURL* aDocumentURL); @@ -131,62 +134,77 @@ NS_IMETHODIMP RobotSink::QueryInterface(REFNSIID aIID, void** aInstancePtr) return NS_NOINTERFACE; } -PRInt32 RobotSink::SetTitle(const nsString& aValue) +NS_IMETHODIMP RobotSink::PushMark() { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::OpenHTML(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::SetTitle(const nsString& aValue) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseHTML(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::OpenHTML(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::OpenHead(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::CloseHTML(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseHead(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::OpenHead(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::OpenBody(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::CloseHead(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseBody(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::OpenBody(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::OpenForm(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::CloseBody(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseForm(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::OpenForm(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::OpenFrameset(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::CloseForm(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseFrameset(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::OpenMap(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::OpenContainer(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::CloseMap(const nsIParserNode& aNode) +{ + return NS_OK; +} + +NS_IMETHODIMP RobotSink::OpenFrameset(const nsIParserNode& aNode) +{ + return NS_OK; +} + +NS_IMETHODIMP RobotSink::CloseFrameset(const nsIParserNode& aNode) +{ + return NS_OK; +} + +NS_IMETHODIMP RobotSink::OpenContainer(const nsIParserNode& aNode) { nsAutoString tmp(aNode.GetText()); tmp.ToUpperCase(); @@ -219,22 +237,22 @@ PRInt32 RobotSink::OpenContainer(const nsIParserNode& aNode) } } } - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseContainer(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::CloseContainer(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::CloseTopmostContainer() +NS_IMETHODIMP RobotSink::CloseTopmostContainer() { - return kNoError; + return NS_OK; } -PRInt32 RobotSink::AddLeaf(const nsIParserNode& aNode) +NS_IMETHODIMP RobotSink::AddLeaf(const nsIParserNode& aNode) { - return kNoError; + return NS_OK; } NS_IMETHODIMP RobotSink::Init(nsIURL* aDocumentURL) diff --git a/mozilla/htmlparser/src/CNavDTD.cpp b/mozilla/htmlparser/src/CNavDTD.cpp index 44af472a73a..defc1241944 100644 --- a/mozilla/htmlparser/src/CNavDTD.cpp +++ b/mozilla/htmlparser/src/CNavDTD.cpp @@ -36,7 +36,7 @@ #include "nsHTMLTokens.h" #include "nsCRT.h" #include "nsParser.h" -#include "nsHTMLContentSink.h" +#include "nsIHTMLContentSink.h" #include "nsScanner.h" #include "nsParserTypes.h" #include "nsVoidArray.h" @@ -420,13 +420,14 @@ eAutoDetectResult CNavDTD::AutoDetectContentType(nsString& aBuffer,nsString& aTy * @param * @return */ -PRInt32 CNavDTD::WillBuildModel(nsString& aFilename){ - PRInt32 result=0; +nsresult CNavDTD::WillBuildModel(nsString& aFilename){ + nsresult result=NS_OK; mFilename=aFilename; - if(mSink) - mSink->WillBuildModel(); + if(mSink) { + result = mSink->WillBuildModel(); + } return result; } @@ -437,14 +438,14 @@ PRInt32 CNavDTD::WillBuildModel(nsString& aFilename){ * @param * @return */ -PRInt32 CNavDTD::DidBuildModel(PRInt32 anErrorCode){ - PRInt32 result=0; +nsresult CNavDTD::DidBuildModel(PRInt32 anErrorCode){ + nsresult result= NS_OK; if((kNoError==anErrorCode) && (mContextStack.mCount>0)) { - CloseContainersTo(0,eHTMLTag_unknown,PR_FALSE); + result = CloseContainersTo(0,eHTMLTag_unknown,PR_FALSE); } if(mSink) { - mSink->DidBuildModel(1); + result = mSink->DidBuildModel(1); } return result; @@ -462,8 +463,8 @@ PRInt32 CNavDTD::DidBuildModel(PRInt32 anErrorCode){ * @param aParser * @return */ -PRInt32 CNavDTD::HandleToken(CToken* aToken){ - PRInt32 result=0; +nsresult CNavDTD::HandleToken(CToken* aToken){ + nsresult result=NS_OK; if(aToken) { CHTMLToken* theToken= (CHTMLToken*)(aToken); @@ -496,16 +497,16 @@ PRInt32 CNavDTD::HandleToken(CToken* aToken){ * @param aNode -- CParserNode representing this start token * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleDefaultStartToken(CToken* aToken,eHTMLTags aChildTag,nsIParserNode& aNode) { +nsresult CNavDTD::HandleDefaultStartToken(CToken* aToken,eHTMLTags aChildTag,nsIParserNode& aNode) { NS_PRECONDITION(0!=aToken,kNullToken); eHTMLTags parentTag=GetTopNode(); - PRInt32 result=kNoError; + nsresult result=NS_OK; PRBool contains=CanContain(parentTag,aChildTag); if(PR_FALSE==contains){ result=CreateContextStackFor(aChildTag); - if(kNoError!=result) { + if(NS_OK!=result) { //if you're here, then the new topmost container can't contain aToken. //You must determine what container hierarchy you need to hold aToken, //and create that on the parsestack. @@ -546,7 +547,7 @@ PRInt32 CNavDTD::HandleDefaultStartToken(CToken* aToken,eHTMLTags aChildTag,nsIP * @param aNode -- CParserNode representing this start token * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { +nsresult CNavDTD::HandleStartToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); CStartToken* st= (CStartToken*)(aToken); @@ -556,9 +557,11 @@ PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { nsCParserNode attrNode((CHTMLToken*)aToken); PRInt16 attrCount=aToken->GetAttributeCount(); PRInt32 theCount; - PRInt32 result=(0==attrCount) ? kNoError : mParser->CollectAttributes(attrNode,attrCount); + nsresult result=(0==attrCount) + ? NS_OK + : mParser->CollectAttributes(attrNode,attrCount); - if(kNoError==result) { + if(NS_OK==result) { //now check to see if this token should be omitted... if(PR_FALSE==CanOmit(GetTopNode(),tokenTagType)) { @@ -571,7 +574,7 @@ PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { { nsCParserNode theNode(st); result=OpenHead(theNode); //open the head... - if(kNoError==result) { + if(NS_OK==result) { result=mParser->CollectSkippedContent(attrNode,theCount); mSink->SetTitle(attrNode.GetSkippedContent()); result=CloseHead(theNode); //close the head... @@ -595,9 +598,9 @@ PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { { nsCParserNode theNode((CHTMLToken*)aToken); result=OpenHead(theNode); - if(kNoError==result) + if(NS_OK==result) result=AddLeaf(attrNode); - if(kNoError==result) + if(NS_OK==result) result=CloseHead(theNode); } break; @@ -606,11 +609,11 @@ PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { { nsCParserNode theNode((CHTMLToken*)aToken); result=OpenHead(theNode); - if(kNoError==result) { + if(NS_OK==result) { mParser->CollectSkippedContent(attrNode,theCount); - if(kNoError==result) { + if(NS_OK==result) { result=AddLeaf(attrNode); - if(kNoError==result) + if(NS_OK==result) result=CloseHead(theNode); } } @@ -625,9 +628,9 @@ PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { case eHTMLTag_base: result=OpenHead(attrNode); - if(kNoError==result) { + if(NS_OK==result) { result=AddLeaf(attrNode); - if(kNoError==result) + if(NS_OK==result) result=CloseHead(attrNode); } break; @@ -661,10 +664,10 @@ PRInt32 CNavDTD::HandleStartToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleEndToken(CToken* aToken) { +nsresult CNavDTD::HandleEndToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); - PRInt32 result=kNoError; + nsresult result=NS_OK; CEndToken* et = (CEndToken*)(aToken); eHTMLTags tokenTagType=(eHTMLTags)et->GetTypeID(); @@ -733,11 +736,11 @@ PRInt32 CNavDTD::HandleEndToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleEntityToken(CToken* aToken) { +nsresult CNavDTD::HandleEntityToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); CEntityToken* et = (CEntityToken*)(aToken); - PRInt32 result=kNoError; + nsresult result=NS_OK; eHTMLTags tokenTagType=(eHTMLTags)et->GetTypeID(); if(PR_FALSE==CanOmit(GetTopNode(),tokenTagType)) { @@ -757,9 +760,9 @@ PRInt32 CNavDTD::HandleEntityToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleCommentToken(CToken* aToken) { +nsresult CNavDTD::HandleCommentToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); - return kNoError; + return NS_OK; } /** @@ -772,10 +775,10 @@ PRInt32 CNavDTD::HandleCommentToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleSkippedContentToken(CToken* aToken) { +nsresult CNavDTD::HandleSkippedContentToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); - PRInt32 result=kNoError; + nsresult result=NS_OK; if(HasOpenContainer(eHTMLTag_body)) { nsCParserNode aNode((CHTMLToken*)aToken); @@ -794,12 +797,12 @@ PRInt32 CNavDTD::HandleSkippedContentToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleAttributeToken(CToken* aToken) { +nsresult CNavDTD::HandleAttributeToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); NS_ERROR("attribute encountered -- this shouldn't happen!"); CAttributeToken* at = (CAttributeToken*)(aToken); - PRInt32 result=kNoError; + nsresult result=NS_OK; return result; } @@ -811,9 +814,9 @@ PRInt32 CNavDTD::HandleAttributeToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleScriptToken(CToken* aToken) { +nsresult CNavDTD::HandleScriptToken(CToken* aToken) { NS_PRECONDITION(0!=aToken,kNullToken); - PRInt32 result=kNoError; + nsresult result=NS_OK; PRInt32 pos=GetTopmostIndexOf(eHTMLTag_body); nsCParserNode theNode((CHTMLToken*)aToken); @@ -823,11 +826,11 @@ PRInt32 CNavDTD::HandleScriptToken(CToken* aToken) { if (kNotFound == pos) { // We're in the HEAD result=OpenHead(theNode); - if(kNoError==result) { + if(NS_OK==result) { mParser->CollectSkippedContent(attrNode,attrCount); - if(kNoError==result) { + if(NS_OK==result) { result=AddLeaf(attrNode); - if(kNoError==result) + if(NS_OK==result) result=CloseHead(theNode); } } @@ -835,7 +838,7 @@ PRInt32 CNavDTD::HandleScriptToken(CToken* aToken) { else { // We're in the BODY mParser->CollectSkippedContent(attrNode,attrCount); - if(kNoError==result) { + if(NS_OK==result) { result=AddLeaf(attrNode); } } @@ -851,11 +854,11 @@ PRInt32 CNavDTD::HandleScriptToken(CToken* aToken) { * @param aToken -- next (start) token to be handled * @return PR_TRUE if all went well; PR_FALSE if error occured */ -PRInt32 CNavDTD::HandleStyleToken(CToken* aToken){ +nsresult CNavDTD::HandleStyleToken(CToken* aToken){ NS_PRECONDITION(0!=aToken,kNullToken); CStyleToken* st = (CStyleToken*)(aToken); - PRInt32 result=kNoError; + nsresult result=NS_OK; return result; } @@ -872,7 +875,6 @@ void CNavDTD::DeleteTokenHandlers(void) { delete mTokenHandlers[i]; mTokenHandlers[i]=0; } - return; } @@ -1762,8 +1764,8 @@ PRInt32 CNavDTD::GetTopmostIndexOf(eHTMLTags aTag) const { * @param tag of the container just opened * @return 0 (for now) */ -PRInt32 CNavDTD::OpenTransientStyles(eHTMLTags aTag){ - PRInt32 result=0; +nsresult CNavDTD::OpenTransientStyles(eHTMLTags aTag){ + nsresult result=NS_OK; if(0==strchr(gWhitespaceTags,aTag)){ PRInt32 pos=0; @@ -1788,7 +1790,7 @@ PRInt32 CNavDTD::OpenTransientStyles(eHTMLTags aTag){ mContextStack.mBits[mContextStack.mCount-1]=PR_TRUE; } //switch } - if(kNoError!=result) + if(NS_OK!=result) break; }//for } @@ -1808,8 +1810,8 @@ PRInt32 CNavDTD::OpenTransientStyles(eHTMLTags aTag){ * @param tag of the container just opened * @return 0 (for now) */ -PRInt32 CNavDTD::CloseTransientStyles(eHTMLTags aTag){ - PRInt32 result=0; +nsresult CNavDTD::CloseTransientStyles(eHTMLTags aTag){ + nsresult result=NS_OK; if((mStyleStack.mCount>0) && (mContextStack.mBits[mContextStack.mCount-1])) { if(0==strchr(gWhitespaceTags,aTag)){ @@ -1829,12 +1831,11 @@ PRInt32 CNavDTD::CloseTransientStyles(eHTMLTags aTag){ * * @update gess4/22/98 * @param aNode -- next node to be added to model - * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenHTML(const nsIParserNode& aNode){ +nsresult CNavDTD::OpenHTML(const nsIParserNode& aNode){ NS_PRECONDITION(mContextStack.mCount >= 0, kInvalidTagStackPos); - PRInt32 result=mSink->OpenHTML(aNode); + nsresult result=mSink->OpenHTML(aNode); mContextStack.Push((eHTMLTags)aNode.GetNodeType()); return result; } @@ -1848,9 +1849,9 @@ PRInt32 CNavDTD::OpenHTML(const nsIParserNode& aNode){ * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseHTML(const nsIParserNode& aNode){ +nsresult CNavDTD::CloseHTML(const nsIParserNode& aNode){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); - PRInt32 result=mSink->CloseHTML(aNode); + nsresult result=mSink->CloseHTML(aNode); mContextStack.Pop(); return result; } @@ -1864,9 +1865,9 @@ PRInt32 CNavDTD::CloseHTML(const nsIParserNode& aNode){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenHead(const nsIParserNode& aNode){ +nsresult CNavDTD::OpenHead(const nsIParserNode& aNode){ mContextStack.Push(eHTMLTag_head); - PRInt32 result=mSink->OpenHead(aNode); + nsresult result=mSink->OpenHead(aNode); return result; } @@ -1878,8 +1879,8 @@ PRInt32 CNavDTD::OpenHead(const nsIParserNode& aNode){ * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseHead(const nsIParserNode& aNode){ - PRInt32 result=mSink->CloseHead(aNode); +nsresult CNavDTD::CloseHead(const nsIParserNode& aNode){ + nsresult result=mSink->CloseHead(aNode); mContextStack.Pop(); return result; } @@ -1892,10 +1893,10 @@ PRInt32 CNavDTD::CloseHead(const nsIParserNode& aNode){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenBody(const nsIParserNode& aNode){ +nsresult CNavDTD::OpenBody(const nsIParserNode& aNode){ NS_PRECONDITION(mContextStack.mCount >= 0, kInvalidTagStackPos); - PRInt32 result=kNoError; + nsresult result=NS_OK; eHTMLTags topTag=GetTopNode(); if(eHTMLTag_html!=topTag) { @@ -1923,7 +1924,7 @@ PRInt32 CNavDTD::OpenBody(const nsIParserNode& aNode){ } } - if(kNoError==result) { + if(NS_OK==result) { result=mSink->OpenBody(aNode); mContextStack.Push((eHTMLTags)aNode.GetNodeType()); } @@ -1938,9 +1939,9 @@ PRInt32 CNavDTD::OpenBody(const nsIParserNode& aNode){ * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseBody(const nsIParserNode& aNode){ +nsresult CNavDTD::CloseBody(const nsIParserNode& aNode){ NS_PRECONDITION(mContextStack.mCount >= 0, kInvalidTagStackPos); - PRInt32 result=mSink->CloseBody(aNode); + nsresult result=mSink->CloseBody(aNode); mContextStack.Pop(); return result; } @@ -1953,11 +1954,11 @@ PRInt32 CNavDTD::CloseBody(const nsIParserNode& aNode){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenForm(const nsIParserNode& aNode){ +nsresult CNavDTD::OpenForm(const nsIParserNode& aNode){ if(mHasOpenForm) CloseForm(aNode); - PRInt32 result=mSink->OpenForm(aNode); - if(kNoError==result) + nsresult result=mSink->OpenForm(aNode); + if(NS_OK==result) mHasOpenForm=PR_TRUE; return result; } @@ -1970,8 +1971,8 @@ PRInt32 CNavDTD::OpenForm(const nsIParserNode& aNode){ * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseForm(const nsIParserNode& aNode){ - PRInt32 result=kNoError; +nsresult CNavDTD::CloseForm(const nsIParserNode& aNode){ + nsresult result=NS_OK; if(mHasOpenForm) { mHasOpenForm=PR_FALSE; result=mSink->CloseForm(aNode); @@ -1987,15 +1988,11 @@ PRInt32 CNavDTD::CloseForm(const nsIParserNode& aNode){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenMap(const nsIParserNode& aNode){ +nsresult CNavDTD::OpenMap(const nsIParserNode& aNode){ if(mHasOpenMap) CloseMap(aNode); - - //NOTE: We need to change to method so that it opens a MAP, - // instead of a FORM. This was copy/paste coding at its best. - - PRInt32 result=mSink->OpenForm(aNode); - if(kNoError==result) + nsresult result=mSink->OpenMap(aNode); + if(NS_OK==result) mHasOpenMap=PR_TRUE; return result; } @@ -2008,15 +2005,11 @@ PRInt32 CNavDTD::OpenMap(const nsIParserNode& aNode){ * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseMap(const nsIParserNode& aNode){ - PRInt32 result=kNoError; +nsresult CNavDTD::CloseMap(const nsIParserNode& aNode){ + nsresult result=NS_OK; if(mHasOpenMap) { mHasOpenMap=PR_FALSE; - - //NOTE: We need to change to method so that it closes a MAP, - // instead of a FORM. This was copy/paste coding at its best. - - result=mSink->CloseForm(aNode); + result=mSink->CloseMap(aNode); } return result; } @@ -2029,9 +2022,9 @@ PRInt32 CNavDTD::CloseMap(const nsIParserNode& aNode){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenFrameset(const nsIParserNode& aNode){ +nsresult CNavDTD::OpenFrameset(const nsIParserNode& aNode){ NS_PRECONDITION(mContextStack.mCount >= 0, kInvalidTagStackPos); - PRInt32 result=mSink->OpenFrameset(aNode); + nsresult result=mSink->OpenFrameset(aNode); mContextStack.Push((eHTMLTags)aNode.GetNodeType()); return result; } @@ -2044,9 +2037,9 @@ PRInt32 CNavDTD::OpenFrameset(const nsIParserNode& aNode){ * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseFrameset(const nsIParserNode& aNode){ +nsresult CNavDTD::CloseFrameset(const nsIParserNode& aNode){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); - PRInt32 result=mSink->CloseFrameset(aNode); + nsresult result=mSink->CloseFrameset(aNode); mContextStack.Pop(); return result; } @@ -2060,10 +2053,11 @@ PRInt32 CNavDTD::CloseFrameset(const nsIParserNode& aNode){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::OpenContainer(const nsIParserNode& aNode,PRBool aUpdateStyleStack){ +nsresult +CNavDTD::OpenContainer(const nsIParserNode& aNode,PRBool aUpdateStyleStack){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); - PRInt32 result=kNoError; + nsresult result=NS_OK; eHTMLTags nodeType=(eHTMLTags)aNode.GetNodeType(); // CloseTransientStyles(nodeType); @@ -2082,6 +2076,10 @@ PRInt32 CNavDTD::OpenContainer(const nsIParserNode& aNode,PRBool aUpdateStyleSta case eHTMLTag_title: break; + case eHTMLTag_map: + result=OpenMap(aNode); + break; + case eHTMLTag_form: result=OpenForm(aNode); break; @@ -2091,7 +2089,7 @@ PRInt32 CNavDTD::OpenContainer(const nsIParserNode& aNode,PRBool aUpdateStyleSta break; } - if((kNoError==result) && (PR_TRUE==aUpdateStyleStack)){ + if((NS_OK==result) && (PR_TRUE==aUpdateStyleStack)){ UpdateStyleStackForOpenTag(nodeType,nodeType); } @@ -2106,9 +2104,11 @@ PRInt32 CNavDTD::OpenContainer(const nsIParserNode& aNode,PRBool aUpdateStyleSta * @param aNode -- next node to be removed from our model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseContainer(const nsIParserNode& aNode,eHTMLTags aTag,PRBool aUpdateStyles){ +nsresult +CNavDTD::CloseContainer(const nsIParserNode& aNode,eHTMLTags aTag, + PRBool aUpdateStyles){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); - PRInt32 result=kNoError; //was false + nsresult result=NS_OK; eHTMLTags nodeType=(eHTMLTags)aNode.GetNodeType(); //XXX Hack! We know this is wrong, but it works @@ -2129,6 +2129,10 @@ PRInt32 CNavDTD::CloseContainer(const nsIParserNode& aNode,eHTMLTags aTag,PRBool case eHTMLTag_body: result=CloseBody(aNode); break; + case eHTMLTag_map: + result=CloseMap(aNode); + break; + case eHTMLTag_form: result=CloseForm(aNode); break; @@ -2140,7 +2144,7 @@ PRInt32 CNavDTD::CloseContainer(const nsIParserNode& aNode,eHTMLTags aTag,PRBool } mContextStack.mBits[mContextStack.mCount]=PR_FALSE; - if((kNoError==result) && (PR_TRUE==aUpdateStyles)){ + if((NS_OK==result) && (PR_TRUE==aUpdateStyles)){ UpdateStyleStackForCloseTag(nodeType,aTag); } @@ -2155,9 +2159,11 @@ PRInt32 CNavDTD::CloseContainer(const nsIParserNode& aNode,eHTMLTags aTag,PRBool * @param * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseContainersTo(PRInt32 anIndex,eHTMLTags aTag,PRBool aUpdateStyles){ +nsresult +CNavDTD::CloseContainersTo(PRInt32 anIndex,eHTMLTags aTag, + PRBool aUpdateStyles){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); - PRInt32 result=kNoError; + nsresult result=NS_OK; CEndToken aToken(gEmpty); nsCParserNode theNode(&aToken); @@ -2180,7 +2186,7 @@ PRInt32 CNavDTD::CloseContainersTo(PRInt32 anIndex,eHTMLTags aTag,PRBool aUpdate * @param * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseContainersTo(eHTMLTags aTag,PRBool aUpdateStyles){ +nsresult CNavDTD::CloseContainersTo(eHTMLTags aTag,PRBool aUpdateStyles){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); PRInt32 pos=GetTopmostIndexOf(aTag); @@ -2203,7 +2209,7 @@ PRInt32 CNavDTD::CloseContainersTo(eHTMLTags aTag,PRBool aUpdateStyles){ } } - PRInt32 result=kNoError; + nsresult result=NS_OK; eHTMLTags theParentTag=GetDefaultParentTagFor(aTag); pos=GetTopmostIndexOf(theParentTag); if(kNotFound!=pos) { @@ -2221,7 +2227,7 @@ PRInt32 CNavDTD::CloseContainersTo(eHTMLTags aTag,PRBool aUpdateStyles){ * @param * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::CloseTopmostContainer(){ +nsresult CNavDTD::CloseTopmostContainer(){ NS_PRECONDITION(mContextStack.mCount > 0, kInvalidTagStackPos); CEndToken aToken(gEmpty); @@ -2239,8 +2245,8 @@ PRInt32 CNavDTD::CloseTopmostContainer(){ * @param aNode -- next node to be added to model * @return TRUE if ok, FALSE if error */ -PRInt32 CNavDTD::AddLeaf(const nsIParserNode& aNode){ - PRInt32 result=mSink->AddLeaf(aNode); +nsresult CNavDTD::AddLeaf(const nsIParserNode& aNode){ + nsresult result=mSink->AddLeaf(aNode); return result; } @@ -2255,10 +2261,10 @@ PRInt32 CNavDTD::AddLeaf(const nsIParserNode& aNode){ * create a new context vector * @return true if we succeeded, otherwise false */ -PRInt32 CNavDTD::CreateContextStackFor(eHTMLTags aChildTag){ +nsresult CNavDTD::CreateContextStackFor(eHTMLTags aChildTag){ nsAutoString theVector; - PRInt32 result=kNoError; + nsresult result=NS_OK; PRInt32 pos=0; PRInt32 cnt=0; eHTMLTags theTop=GetTopNode(); @@ -2268,7 +2274,7 @@ PRInt32 CNavDTD::CreateContextStackFor(eHTMLTags aChildTag){ pos=0; cnt=theVector.Length()-1; if(mContextStack.Last()==(eHTMLTags)theVector[cnt]) - result=kNoError; + result=NS_OK; else result=kContextMismatch; } else { @@ -2285,7 +2291,7 @@ PRInt32 CNavDTD::CreateContextStackFor(eHTMLTags aChildTag){ //propagation worked, so pop unwanted containers, push new ones, then exit... pos=0; cnt=theVector.Length(); - result=kNoError; + result=NS_OK; while(pos0) { switch (aTag) { @@ -2452,18 +2460,19 @@ PRInt32 CNavDTD::UpdateStyleStackForCloseTag(eHTMLTags aTag,eHTMLTags anActualTa * @param aToken is the out arg holding our new token * @return error code (may return kInterrupted). */ -PRInt32 CNavDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) { +nsresult +CNavDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) { nsAutoString empty(""); - PRInt32 result=aScanner.GetChar(aChar); + nsresult result=aScanner.GetChar(aChar); - if(kNoError==result) { + if(NS_OK==result) { switch(aChar) { case kForwardSlash: PRUnichar ch; result=aScanner.Peek(ch); - if(kNoError==result) { + if(NS_OK==result) { if(nsString::IsAlpha(ch)) aToken=new CEndToken(empty); else aToken=new CCommentToken(empty); //Special case: is treated as a comment @@ -2481,7 +2490,7 @@ PRInt32 CNavDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) } } //switch - if((0!=aToken) && (kNoError==result)) { + if((0!=aToken) && (NS_OK==result)) { result= aToken->Consume(aChar,aScanner); //tell new token to finish consuming text... if(result) { delete aToken; @@ -2501,13 +2510,14 @@ PRInt32 CNavDTD::ConsumeTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) * @param aScanner: see nsScanner.h * @return */ -PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken) { +nsresult +CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToken* aToken) { PRBool done=PR_FALSE; - PRInt32 result=kNoError; + nsresult result=NS_OK; nsAutoString as(""); PRInt16 theAttrCount=0; - while((!done) && (result==kNoError)) { + while((!done) && (result==NS_OK)) { CAttributeToken* theToken= new CAttributeToken(as); if(theToken){ result=theToken->Consume(aChar,aScanner); //tell new token to finish consuming text... @@ -2524,14 +2534,14 @@ PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToke aToken->SetEmpty(PR_TRUE); delete theToken; } - else if(kNoError==result){ + else if(NS_OK==result){ theAttrCount++; mTokenDeque.Push(theToken); }//if else delete theToken; //we can't keep it... }//if - if(kNoError==result){ + if(NS_OK==result){ result=aScanner.Peek(aChar); if(aChar==kGreaterThan) { //you just ate the '>' aScanner.GetChar(aChar); //skip the '>' @@ -2553,7 +2563,11 @@ PRInt32 CNavDTD::ConsumeAttributes(PRUnichar aChar,CScanner& aScanner,CStartToke * @param anErrorCode: arg that will hold error condition * @return new token or null */ -PRInt32 CNavDTD::ConsumeContentToEndTag(const nsString& aString,PRUnichar aChar,CScanner& aScanner,CToken*& aToken){ +nsresult +CNavDTD::ConsumeContentToEndTag(const nsString& aString, + PRUnichar aChar, + CScanner& aScanner, + CToken*& aToken){ //In the case that we just read the given tag, we should go and //consume all the input until we find a matching end tag. @@ -2575,22 +2589,23 @@ PRInt32 CNavDTD::ConsumeContentToEndTag(const nsString& aString,PRUnichar aChar, * @param anErrorCode: arg that will hold error condition * @return new token or null */ -PRInt32 CNavDTD::ConsumeStartTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) { +nsresult +CNavDTD::ConsumeStartTag(PRUnichar aChar,CScanner& aScanner,CToken*& aToken) { PRInt32 theDequeSize=mTokenDeque.GetSize(); - PRInt32 result=kNoError; + nsresult result=NS_OK; aToken=new CStartToken(nsAutoString("")); if(aToken) { result= aToken->Consume(aChar,aScanner); //tell new token to finish consuming text... - if(kNoError==result) { + if(NS_OK==result) { if(((CStartToken*)aToken)->IsAttributed()) { result=ConsumeAttributes(aChar,aScanner,(CStartToken*)aToken); } //now that that's over with, we have one more problem to solve. //In the case that we just read a