[webkit-reviews] review granted: [Bug 200589] Fix Crash in Mail Search : [Attachment 375968] Patch

[bugzilla-daemon at webkit.org]

Tim Horton <thorton at apple.com> has granted Megan Gardner
<megan_gardner at apple.com>'s request for review:
Bug 200589: Fix Crash in Mail Search
https://bugs.webkit.org/show_bug.cgi?id=200589

Attachment 375968: Patch

https://bugs.webkit.org/attachment.cgi?id=375968&action=review




--- Comment #9 from Tim Horton <thorton at apple.com> ---
Comment on attachment 375968
  --> https://bugs.webkit.org/attachment.cgi?id=375968
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=375968&action=review

> Source/WebKit/UIProcess/mac/WKTextFinderClient.mm:84
> -	       allMatches[matchIndex].appendVector(matchRects);
> +	       if (matchIndex >= 0 && (uint32_t)matchIndex < matchCount)

This is fine for now given current clients, but maybe leave a FIXME that we
should have match rects when doing an incremental backwards search (did you
test Notes??).