[webkit-reviews] review cancelled: [Bug 7683] Implement execCommand(UnLink) : [Attachment 7103] patch

[bugzilla-request-daemon at opendarwin.org]

Justin Garcia <justin.garcia at apple.com> has cancelled Justin Garcia
<justin.garcia at apple.com>'s request for review:
Bug 7683: Implement execCommand(UnLink)
http://bugzilla.opendarwin.org/show_bug.cgi?id=7683

Attachment 7103: patch
http://bugzilla.opendarwin.org/attachment.cgi?id=7103&action=edit

------- Additional Comments from Justin Garcia <justin.garcia at apple.com>
Eek, I forgot to svn add Unlink.{h,cpp}!  Removing the review flag until I
check in a new patch.  Here are some other problems I noticed:

This should be a FIXME
+    // This pushes down anchors even if they are fully selected.  This is 
+    // less efficient, but makes this code much less complicated.

I now use the new form of the ApplyStyleCommand constructor in 3 places:
+    EditCommandPtr cmd(new ApplyStyleCommand(document(), new
CSSMutableStyleDeclarationImpl(),
static_cast<ElementImpl*>(newAnchorElement.get())));
+    applyCommandToComposite(cmd);
I should put a convenience method in CompositeEditCommand.

I should follow the style guidelines even in JS code:
+    }
+    else {
+	 execUnlinkCommand();
+    }
+}

All I really care about in the test case is the result of innerHTML, so I
should probably make the test use dumpAsText.