[webkit-reviews] review requested: [Bug 7578] TinyMCE: Implement execCommand CreateLink : [Attachment 7066] patch

[bugzilla-request-daemon at opendarwin.org]

Justin Garcia <justin.garcia at apple.com> has asked  for review:
Bug 7578: TinyMCE: Implement execCommand CreateLink
http://bugzilla.opendarwin.org/show_bug.cgi?id=7578

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

------- Additional Comments from Justin Garcia <justin.garcia at apple.com>
Another way to do this would be to pull out the parts of ApplyStyleCommand that
apply style to text runs from the parts that do the setup, traversal and
cleanup, and make two new classes, PushInlineChangeCommand and
ApplyInlineStyleCommand, with CreateLink and ApplyinlineStyleCommand as
subclasses of PushInlineChangeCommand.	That's more risky and probably not
worth the effort unless we think we'll end up needing PushInlineChange in order
to implement something else.