String.prototype.{match,search} should do only one RegExp brand check

String.prototype.{match,search} should do only one RegExp brand check

Previously, they would check for Symbol.match/Symbol.search, and then
do another check for Symbol.match in the RegExp constructor. This patch
avoids the second one by skipping the RegExp constructor, as the spec does.

Committed: https://crrev.com/0fdf32556c97068a03273606fecdac287325b4fb
Cr-Commit-Position: refs/heads/master@{#35085}

[adamk, 2016-03-28 17:55:22]

adamk@chromium.org changed reviewers:
+ littledan@chromium.org

[Dan Ehrenberg, 2016-03-28 18:46:19]

lgtm

[adamk, 2016-03-28 18:50:26]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-03-28 18:50:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1840723002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1840723002/1

[commit-bot: I haz the power, 2016-03-28 18:52:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-28 18:53:49]

Description was changed from

==========
String.prototype.{match,search} should do only one RegExp brand check

Previously, they would check for Symbol.match/Symbol.search, and then
do another check for Symbol.match in the RegExp constructor. This patch
avoids the second one by skipping the RegExp constructor, as the spec does.
==========

to

==========
String.prototype.{match,search} should do only one RegExp brand check

Previously, they would check for Symbol.match/Symbol.search, and then
do another check for Symbol.match in the RegExp constructor. This patch
avoids the second one by skipping the RegExp constructor, as the spec does.

Committed: https://crrev.com/0fdf32556c97068a03273606fecdac287325b4fb
Cr-Commit-Position: refs/heads/master@{#35085}
==========

[commit-bot: I haz the power, 2016-03-28 18:53:51]

Patchset 1 (id:??) landed as
https://crrev.com/0fdf32556c97068a03273606fecdac287325b4fb
Cr-Commit-Position: refs/heads/master@{#35085}