Page::findStringMatchingRanges() should take Vector<RefPtr<Range>> by reference instead of pointer
Created attachment 232772 [details] Patch
Comment on attachment 232772 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232772&action=review > Source/WebCore/page/Page.cpp:542 > +void Page::findStringMatchingRanges(const String& target, FindOptions options, int limit, Vector<RefPtr<Range>>& matchRanges, int& indexForSelection) Can this be a const reference?
Comment on attachment 232772 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232772&action=review >> Source/WebCore/page/Page.cpp:542 >> +void Page::findStringMatchingRanges(const String& target, FindOptions options, int limit, Vector<RefPtr<Range>>& matchRanges, int& indexForSelection) > > Can this be a const reference? No, that would clash with Editor::countMatchesForText() which expects a pointer to a non-const Vector.
Created attachment 233112 [details] Patch
Comment on attachment 233112 [details] Patch Clearing flags on attachment: 233112 Committed r170012: <http://trac.webkit.org/changeset/170012>
All reviewed patches have been landed. Closing bug.