[webkit-reviews] review granted: [Bug 23389] Clean up ApplyStyleCommand and deploy a bit more PassRefPtr : [Attachment 26807] Small cleanups

[bugzilla-daemon at webkit.org]

Justin Garcia <justin.garcia at apple.com> has granted Eric Seidel
<eric at webkit.org>'s request for review:
Bug 23389: Clean up ApplyStyleCommand and deploy a bit more PassRefPtr
https://bugs.webkit.org/show_bug.cgi?id=23389

Attachment 26807: Small cleanups
https://bugs.webkit.org/attachment.cgi?id=26807&action=review

------- Additional Comments from Justin Garcia <justin.garcia at apple.com>
-    if (styleChange.applyBold()) {
-	 RefPtr<Element> boldElement =
document()->createElementNS(xhtmlNamespaceURI, "b", ec);
-	 ASSERT(ec == 0);
-	 insertNodeBefore(boldElement.get(), startNode);
-	 surroundNodeRangeWithElement(startNode, endNode, boldElement.get());
-    }
+    if (styleChange.applyBold())
+	 surroundNodeRangeWithElement(startNode, endNode,
document()->createElementNS(xhtmlNamespaceURI, "b", ec));

You no longer ASSERT that createElementNS completed without error, I suppose if
it encounters an error it will return 0 and that will be caught by
surroundNodeRangeWithElement, ya?

r=me