Bug 173227

Summary:

[JSC] Update RegExp.prototype.[@@search]] implementation according to the latest spec

Product:

WebKit

Reporter:

Yusuke Suzuki <ysuzuki>

Component:

New Bugs

Assignee:

Yusuke Suzuki <ysuzuki>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

keith_miller, mark.lam, msaboff, saam

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	mark.lam: review+

Yusuke Suzuki

Reported 2017-06-10 10:46:45 PDT

[JSC] Update RegExp.prototype.[@@search]] implementation according to the latest spec

Attachments
Patch (3.94 KB, patch) 2017-06-10 10:48 PDT, Yusuke Suzuki	mark.lam: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Yusuke Suzuki

Comment 1 2017-06-10 10:48:34 PDT

Created attachment 312574 [details] Patch

Mark Lam

Comment 2 2017-06-10 10:56:42 PDT

Comment on attachment 312574 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=312574&action=review r=me with suggested ChangeLog change. > Source/JavaScriptCore/ChangeLog:10 > + The latest spec introduces slight change to RegExp.prototype.[@@search]. > + This patch applies this change. Basically, this change is done in the slow path of > + the RegExp.prototype[@@search]. Please include the spec url here for easy reference.

Yusuke Suzuki

Comment 3 2017-06-10 11:01:58 PDT

Committed r218051: <http://trac.webkit.org/changeset/218051>

Yusuke Suzuki

Comment 4 2017-06-10 11:35:27 PDT

Committed r218053: <http://trac.webkit.org/changeset/218053>

Yusuke Suzuki

Comment 5 2017-06-10 12:20:56 PDT

Committed r218054: <http://trac.webkit.org/changeset/218054>

Yusuke Suzuki

Comment 6 2017-06-11 00:53:31 PDT

Committed r218065: <http://trac.webkit.org/changeset/218065>

Note You need to log in before you can comment on or make changes to this bug.