Should always return true, the caching of the "true" value is most important. Failing on getting the

preshell or frames should not be passed back.
r=self, bug 18784

content/html/content/src/nsHTMLSelectElement.cpp

@@ -726,7 +726,7 @@ nsHTMLSelectElement::DoneAddingContent()
       result = selectFrame->DoneAddingContent();
     }
   }
-  return result;
+  return NS_OK;
 }
 
 NS_IMETHODIMP

layout/html/content/src/nsHTMLSelectElement.cpp

@@ -726,7 +726,7 @@ nsHTMLSelectElement::DoneAddingContent()
       result = selectFrame->DoneAddingContent();
     }
   }
-  return result;
+  return NS_OK;
 }
 
 NS_IMETHODIMP