Bug 756579 - nsXBLBinding::GetInsertionPointsFor is now infallible.

content/xbl/src/nsXBLBinding.cpp

@@ -1395,7 +1395,7 @@ nsXBLBinding::HasInsertionParent(nsIContent* aParent)
   return mNextBinding ? mNextBinding->HasInsertionParent(aParent) : false;
 }
 
-nsresult
+void
 nsXBLBinding::GetInsertionPointsFor(nsIContent* aParent,
                                     nsInsertionPointList** aResult)
 {
@@ -1414,8 +1414,6 @@ nsXBLBinding::GetInsertionPointsFor(nsIContent* aParent,
       aParent->SetFlags(NODE_IS_INSERTION_PARENT);
     }
   }
-
-  return NS_OK;
 }
 
 nsInsertionPointList*

content/xbl/src/nsXBLBinding.h

@@ -89,8 +89,8 @@ public:
 
   // Get the list of insertion points for aParent. The nsInsertionPointList
   // is owned by the binding, you should not delete it.
-  nsresult GetInsertionPointsFor(nsIContent* aParent,
-                                 nsInsertionPointList** aResult);
+  void GetInsertionPointsFor(nsIContent* aParent,
+                             nsInsertionPointList** aResult);
 
   nsInsertionPointList* GetExistingInsertionPointsFor(nsIContent* aParent);