Bug 851841 - Make nsListBoxBodyFrame::RemoveChildFrame assertions fatal. r=bzbarsky

layout/xul/base/src/nsListBoxBodyFrame.cpp

@@ -1466,15 +1466,8 @@ void
 nsListBoxBodyFrame::RemoveChildFrame(nsBoxLayoutState &aState,
                                      nsIFrame         *aFrame)
 {
-  if (!mFrames.ContainsFrame(aFrame)) {
+  MOZ_ASSERT(mFrames.ContainsFrame(aFrame));
-    NS_ERROR("tried to remove a child frame which isn't our child");
+  MOZ_ASSERT(aFrame != GetContentInsertionFrame());
-    return;
-  }
-
-  if (aFrame == GetContentInsertionFrame()) {
-    // Don't touch that one
-    return;
-  }
 
 #ifdef ACCESSIBILITY
   nsAccessibilityService* accService = nsIPresShell::AccService();