[webkit-reviews] review denied: [Bug 4680] WebHTMLView (WebNSTextInputSupport) - attributedSubstringFromRange "not yet implemented" : [Attachment 4208] Updated patch

[bugzilla-request-daemon at opendarwin.org]

Darin Adler <darin at apple.com> has denied Evan Gross <evan at rainmakerinc.com>'s
request for review:
Bug 4680: WebHTMLView (WebNSTextInputSupport) - attributedSubstringFromRange
"not yet implemented"
http://bugzilla.opendarwin.org/show_bug.cgi?id=4680

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

------- Additional Comments from Darin Adler <darin at apple.com>
There's a fundamental problem here because _attributeStringFromDOMRange: uses
an entirely different code path than our usual conversion to a string, and
there's no way to guarantee they are in sync.

I don't like the strategy here; I think instead we need to create a new
conversion to attributed string that *is* based on the WebCore text machinery
(rather than relying no the entirely-outside-WebKit one from AppKit we are
currently using) for this call.

I'm sorry; I know that's a bigger undertaking than this patch.