--- /home/tglek/work/mozilla/layout/base/nsCSSFrameConstructor.cpp +++ /home/tglek/work/mozilla/layout/base/nsCSSFrameConstructor.cpp @@ -2316,1 +2316,1 @@ - return ConstructCheckboxControlFrame(aFrame, aContent, aStyleContext); + return *aFrame = ConstructCheckboxControlFrame(, aContent, aStyleContext); @@ -2321,1 +2321,1 @@ - return ConstructRadioControlFrame(aFrame, aContent, aStyleContext); + return *aFrame = ConstructRadioControlFrame(, aContent, aStyleContext); @@ -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; @@ -4291,1 +4291,1 @@ - rv = ConstructDocElementTableFrame(aDocElement, aParentFrame, &contentFrame, + rv = contentFrame = ConstructDocElementTableFrame(aDocElement, aParentFrame, @@ -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; @@ -8297,1 +8297,1 @@ - GetInsertionPoint(parentFrame, nsnull, &insertionPoint, &multiple); + insertionPoint = GetInsertionPoint(parentFrame, &insertionPoint, &multiple); @@ -8346,1 +8346,1 @@ - GetInsertionPoint(parentFrame, child, &insertionPoint); + insertionPoint = GetInsertionPoint(parentFrame, child); @@ -8883,1 +8883,1 @@ - GetInsertionPoint(parentFrame, aChild, &insertionPoint); + insertionPoint = GetInsertionPoint(parentFrame, aChild); @@ -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; @@ -10443,1 +10443,1 @@ - rv = CreateContinuingTableFrame(shell, aPresContext, aFrame, aParentFrame, + rv = newFrame = CreateContinuingTableFrame(shell, aPresContext, aFrame, aParentFrame, @@ -10444,1 +10444,1 @@ - content, styleContext, &newFrame); + content, styleContext); @@ -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; @@ -10974,1 +10974,1 @@ - GetInsertionPoint(insertionPoint, aChildContent, aInsertionPoint, aMultiple); + *aInsertionPoint = GetInsertionPoint(insertionPoint, aChildContent, aMultiple); @@ -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); @@ -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);