WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
50530
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+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
mitz
Comment 1
2010-12-04 16:42:55 PST
<
rdar://problem/8145955
>
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
Committed
r73337
: <
http://trac.webkit.org/changeset/73337
>
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.
Top of Page
Format For Printing
XML
Clone This Bug