Properly initialize out parameters to prevent the frame constructor from thinking it has to handle insertion points when it really doesn't. r=bryner sr=hyatt b=106802

content/xbl/src/nsBindingManager.cpp

@@ -1134,13 +1134,16 @@ nsBindingManager::GetInsertionPoint(nsIContent* aParent, nsIContent* aChild, nsI
   nsCOMPtr<nsIXBLBinding> binding;
   GetBinding(aParent, getter_AddRefs(binding));
 
-  nsCOMPtr<nsIContent> defContent;
-  if (binding)
-    return binding->GetInsertionPoint(aChild, aResult, aIndex, getter_AddRefs(defContent));
-  
+  if (!binding) {
+    *aResult = nsnull;
     return NS_OK;
   }
   
+  nsCOMPtr<nsIContent> defContent;
+  return binding->GetInsertionPoint(aChild, aResult, aIndex,
+                                    getter_AddRefs(defContent));
+}
+
 NS_IMETHODIMP
 nsBindingManager::GetSingleInsertionPoint(nsIContent* aParent, nsIContent** aResult, PRUint32* aIndex,
                                           PRBool* aMultipleInsertionPoints)
@@ -1148,14 +1151,18 @@ nsBindingManager::GetSingleInsertionPoint(nsIContent* aParent, nsIContent** aRes
   nsCOMPtr<nsIXBLBinding> binding;
   GetBinding(aParent, getter_AddRefs(binding));
   
-  nsCOMPtr<nsIContent> defContent;
-
-  if (binding)
-    return binding->GetSingleInsertionPoint(aResult, aIndex, aMultipleInsertionPoints, getter_AddRefs(defContent));
-  
+  if (!binding) {
+    *aMultipleInsertionPoints = PR_FALSE;
+    *aResult = nsnull;
     return NS_OK;
   }
 
+  nsCOMPtr<nsIContent> defContent;
+  return binding->GetSingleInsertionPoint(aResult, aIndex,
+                                          aMultipleInsertionPoints,
+                                          getter_AddRefs(defContent));
+}
+
 NS_IMETHODIMP
 nsBindingManager::AddLayeredBinding(nsIContent* aContent, const nsAReadableString& aURL)
 {