--- /home/tglek/work/mozilla/layout/base/nsCSSFrameConstructor.cpp +++ /home/tglek/work/mozilla/layout/base/nsCSSFrameConstructor.cpp @@ -2359,5 +2359,4 @@ -nsresult -nsCSSFrameConstructor::CreateHTMLImageFrame(nsIContent* aContent, - nsStyleContext* aStyleContext, - ImageFrameCreatorFunc aFunc, - nsIFrame** aFrame) +nsIFrame* +nsCSSFrameConstructor::CreateHTMLImageFrame(nsIContent* aContent, + nsStyleContext* aStyleContext, + ImageFrameCreatorFunc aFunc) @@ -2364,1 +2364,2 @@ -{ +{ + nsIFrame* aFrame = nsnull; @@ -2365,1 +2365,1 @@ - *aFrame = nsnull; + aFrame = nsnull; @@ -2369,1 +2369,1 @@ - *aFrame = (*aFunc)(mPresShell, aStyleContext); + aFrame = (*aFunc)(mPresShell, aStyleContext); @@ -2371,1 +2371,1 @@ - if (NS_UNLIKELY(!*aFrame)) + if (NS_UNLIKELY(!aFrame)) @@ -2372,1 +2372,1 @@ - return NS_ERROR_OUT_OF_MEMORY; + return aFrame; @@ -2375,1 +2375,1 @@ - return NS_OK; + return aFrame; @@ -4061,5 +4061,4 @@ -nsresult -nsCSSFrameConstructor::ConstructDocElementTableFrame(nsIContent* aDocElement, - nsIFrame* aParentFrame, - nsIFrame** aNewTableFrame, - nsFrameConstructorState& aState) +nsIFrame* +nsCSSFrameConstructor::ConstructDocElementTableFrame(nsIContent* aDocElement, + nsIFrame* aParentFrame, + nsFrameConstructorState& aState) @@ -4066,1 +4066,2 @@ -{ +{ + nsIFrame* aNewTableFrame = nsnull; @@ -4078,1 +4078,1 @@ - *aNewTableFrame = frameItems.childList; + aNewTableFrame = frameItems.childList; @@ -4079,1 +4079,1 @@ - if (!*aNewTableFrame) { + if (!aNewTableFrame) { @@ -4082,1 +4082,1 @@ - return NS_ERROR_FAILURE; + return aNewTableFrame; @@ -4084,1 +4084,1 @@ - return NS_OK; + return aNewTableFrame; @@ -4695,4 +4695,4 @@ -nsresult -nsCSSFrameConstructor::ConstructRadioControlFrame(nsIFrame** aNewFrame, - nsIContent* aContent, - nsStyleContext* aStyleContext) +nsIFrame* +nsCSSFrameConstructor::ConstructRadioControlFrame( + nsIContent* aContent, + nsStyleContext* aStyleContext) @@ -4699,1 +4699,2 @@ -{ +{ + nsIFrame* aNewFrame = nsnull; @@ -4700,1 +4700,1 @@ - *aNewFrame = NS_NewGfxRadioControlFrame(mPresShell, aStyleContext); + aNewFrame = NS_NewGfxRadioControlFrame(mPresShell, aStyleContext); @@ -4701,1 +4701,1 @@ - if (NS_UNLIKELY(!*aNewFrame)) { + if (NS_UNLIKELY(!aNewFrame)) { @@ -4702,1 +4702,1 @@ - return NS_ERROR_OUT_OF_MEMORY; + return aNewFrame; @@ -4710,1 +4710,1 @@ - if (*aNewFrame && NS_SUCCEEDED(CallQueryInterface(*aNewFrame, &radio))) { + if (aNewFrame && NS_SUCCEEDED(CallQueryInterface(aNewFrame, &radio))) { @@ -4713,1 +4713,1 @@ - return NS_OK; + return aNewFrame; @@ -4716,4 +4716,4 @@ -nsresult -nsCSSFrameConstructor::ConstructCheckboxControlFrame(nsIFrame** aNewFrame, - nsIContent* aContent, - nsStyleContext* aStyleContext) +nsIFrame* +nsCSSFrameConstructor::ConstructCheckboxControlFrame( + nsIContent* aContent, + nsStyleContext* aStyleContext) @@ -4720,1 +4720,2 @@ -{ +{ + nsIFrame* aNewFrame = nsnull; @@ -4721,1 +4721,1 @@ - *aNewFrame = NS_NewGfxCheckboxControlFrame(mPresShell, aStyleContext); + aNewFrame = NS_NewGfxCheckboxControlFrame(mPresShell, aStyleContext); @@ -4722,1 +4722,1 @@ - if (NS_UNLIKELY(!*aNewFrame)) { + if (NS_UNLIKELY(!aNewFrame)) { @@ -4723,1 +4723,1 @@ - return NS_ERROR_OUT_OF_MEMORY; + return aNewFrame; @@ -4731,1 +4731,1 @@ - if (*aNewFrame && NS_SUCCEEDED(CallQueryInterface(*aNewFrame, &checkbox))) { + if (aNewFrame && NS_SUCCEEDED(CallQueryInterface(aNewFrame, &checkbox))) { @@ -4734,1 +4734,1 @@ - return NS_OK; + return aNewFrame; @@ -10291,8 +10291,7 @@ -nsresult -nsCSSFrameConstructor::CreateContinuingTableFrame(nsIPresShell* aPresShell, - nsPresContext* aPresContext, - nsIFrame* aFrame, - nsIFrame* aParentFrame, - nsIContent* aContent, - nsStyleContext* aStyleContext, - nsIFrame** aContinuingFrame) +nsIFrame* +nsCSSFrameConstructor::CreateContinuingTableFrame(nsIPresShell* aPresShell, + nsPresContext* aPresContext, + nsIFrame* aFrame, + nsIFrame* aParentFrame, + nsIContent* aContent, + nsStyleContext* aStyleContext) @@ -10299,1 +10299,2 @@ -{ +{ + nsIFrame* aContinuingFrame = nsnull; @@ -10350,1 +10350,1 @@ - *aContinuingFrame = newFrame; + aContinuingFrame = newFrame; @@ -10351,1 +10351,1 @@ - return NS_OK; + return aContinuingFrame; @@ -10354,1 +10354,1 @@ - *aContinuingFrame = nsnull; + aContinuingFrame = nsnull; @@ -10355,1 +10355,1 @@ - return NS_ERROR_OUT_OF_MEMORY; + return aContinuingFrame; @@ -10927,5 +10927,4 @@ -nsresult -nsCSSFrameConstructor::GetInsertionPoint(nsIFrame* aParentFrame, - nsIContent* aChildContent, - nsIFrame** aInsertionPoint, - PRBool* aMultiple) +nsIFrame* +nsCSSFrameConstructor::GetInsertionPoint(nsIFrame* aParentFrame, + nsIContent* aChildContent, + PRBool* aMultiple) @@ -10932,1 +10932,2 @@ -{ +{ + nsIFrame* aInsertionPoint = nsnull; @@ -10935,1 +10935,1 @@ - *aInsertionPoint = aParentFrame; + aInsertionPoint = aParentFrame; @@ -10939,1 +10939,1 @@ - return NS_OK; + return aInsertionPoint; @@ -10950,1 +10950,1 @@ - return NS_OK; + return aInsertionPoint; @@ -10978,1 +10978,1 @@ - *aInsertionPoint = nsnull; + aInsertionPoint = nsnull; @@ -10992,1 +10992,1 @@ - return NS_OK; + return aInsertionPoint; --- /home/tglek/work/mozilla/layout/base/nsCSSFrameConstructor.h +++ /home/tglek/work/mozilla/layout/base/nsCSSFrameConstructor.h @@ -182,4 +182,3 @@ - nsresult GetInsertionPoint(nsIFrame* aParentFrame, - nsIContent* aChildContent, - nsIFrame** aInsertionPoint, - PRBool* aMultiple = nsnull); + nsIFrame* GetInsertionPoint(nsIFrame* aParentFrame, + nsIContent* aChildContent, + PRBool* aMultiple = nsnull); @@ -246,4 +246,3 @@ - nsresult ConstructDocElementTableFrame(nsIContent* aDocElement, - nsIFrame* aParentFrame, - nsIFrame** aNewTableFrame, - nsFrameConstructorState& aState); + nsIFrame* ConstructDocElementTableFrame(nsIContent* aDocElement, + nsIFrame* aParentFrame, + nsFrameConstructorState& aState); @@ -471,3 +471,3 @@ - nsresult ConstructRadioControlFrame(nsIFrame** aNewFrame, - nsIContent* aContent, - nsStyleContext* aStyleContext); + nsIFrame* ConstructRadioControlFrame( + nsIContent* aContent, + nsStyleContext* aStyleContext); @@ -476,3 +476,3 @@ - nsresult ConstructCheckboxControlFrame(nsIFrame** aNewFrame, - nsIContent* aContent, - nsStyleContext* aStyleContext); + nsIFrame* ConstructCheckboxControlFrame( + nsIContent* aContent, + nsStyleContext* aStyleContext); @@ -666,4 +666,3 @@ - nsresult CreateHTMLImageFrame(nsIContent* aContent, - nsStyleContext* aStyleContext, - ImageFrameCreatorFunc aFunc, - nsIFrame** aFrame); + nsIFrame* CreateHTMLImageFrame(nsIContent* aContent, + nsStyleContext* aStyleContext, + ImageFrameCreatorFunc aFunc); @@ -742,7 +742,6 @@ - nsresult CreateContinuingTableFrame(nsIPresShell* aPresShell, - nsPresContext* aPresContext, - nsIFrame* aFrame, - nsIFrame* aParentFrame, - nsIContent* aContent, - nsStyleContext* aStyleContext, - nsIFrame** aContinuingFrame); + nsIFrame* CreateContinuingTableFrame(nsIPresShell* aPresShell, + nsPresContext* aPresContext, + nsIFrame* aFrame, + nsIFrame* aParentFrame, + nsIContent* aContent, + nsStyleContext* aStyleContext);