WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
69230
Strength reduce Mac only Frame::searchForLabelsBeforeElement and Frame::matchLabelsAgainstElement into callers
https://bugs.webkit.org/show_bug.cgi?id=69230
Summary
Strength reduce Mac only Frame::searchForLabelsBeforeElement and Frame::match...
Sam Weinig
Reported
2011-10-02 16:41:56 PDT
Strength reduce Mac only Frame::searchForLabelsBeforeElement and Frame::matchLabelsAgainstElement into callers
Attachments
Patch
(30.83 KB, patch)
2011-10-02 16:44 PDT
,
Sam Weinig
andersca
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Sam Weinig
Comment 1
2011-10-02 16:44:38 PDT
Created
attachment 109431
[details]
Patch
Anders Carlsson
Comment 2
2011-10-02 17:33:25 PDT
Comment on
attachment 109431
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=109431&action=review
> Source/WebKit/mac/WebView/WebHTMLRepresentation.mm:537 > + // FIXME: This code is mirrored in Frame.cpp. It would be nice to make the Mac code call the platform-agnostic > + // code, which would require converting the NSArray of NSStrings to a Vector of Strings somewhere along the way.
Is this comment still relevant?
Sam Weinig
Comment 3
2011-10-02 17:39:40 PDT
Committed
r96480
: <
http://trac.webkit.org/changeset/96480
>
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