RESOLVED FIXED50530
Add text search API for counting/marking/highlighting matches in a range
https://bugs.webkit.org/show_bug.cgi?id=50530
Summary Add text search API for counting/marking/highlighting matches in a range
mitz
Reported 2010-12-04 16:41:38 PST
Add text search API for counting/marking/highlighting matches in a range
Attachments
WebView API and WebCore implementation (11.78 KB, patch)
2010-12-04 16:55 PST, mitz
sam: review+
mitz
Comment 1 2010-12-04 16:42:55 PST
mitz
Comment 2 2010-12-04 16:55:29 PST
Created attachment 75621 [details] WebView API and WebCore implementation
mitz
Comment 3 2010-12-04 19:22:20 PST
Darin Adler
Comment 4 2010-12-06 09:35:09 PST
Comment on attachment 75621 [details] WebView API and WebCore implementation View in context: https://bugs.webkit.org/attachment.cgi?id=75621&action=review > WebCore/editing/Editor.cpp:3353 > +static bool isFrameInRange(Frame* frame, Range* range) We could factor this into two functions, one that takes a document and a frame and finds the HTMLFrameOwnerElement, and a second that calls range->intersectsNode. Not sure if the function that takes a document and a frame and finds the owner element is useful in other contexts. > WebKit/mac/WebView/WebPDFView.mm:632 > +static BOOL isFrameInRange(WebFrame *frame, DOMRange *range) If you refactored the function above the way I suggested, then you could share code here.
Note You need to log in before you can comment on or make changes to this bug.