Normative: Don't call well-known Symbol methods for RegExp on primitive values

Author: petamoriken

spec.html

@@ -35649,7 +35649,7 @@ <h1>String.prototype.match ( _regexp_ )</h1>
         <emu-alg>
           1. Let _O_ be the *this* value.
           1. Perform ? RequireObjectCoercible(_O_).
-          1. If _regexp_ is neither *undefined* nor *null*, then
+          1. If _regexp_ is an Object, then
             1. Let _matcher_ be ? GetMethod(_regexp_, %Symbol.match%).
             1. If _matcher_ is not *undefined*, then
               1. Return ? Call(_matcher_, _regexp_, « _O_ »).
@@ -35670,7 +35670,7 @@ <h1>String.prototype.matchAll ( _regexp_ )</h1>
         <emu-alg>
           1. Let _O_ be the *this* value.
           1. Perform ? RequireObjectCoercible(_O_).
-          1. If _regexp_ is neither *undefined* nor *null*, then
+          1. If _regexp_ is an Object, then
             1. Let _isRegExp_ be ? IsRegExp(_regexp_).
             1. If _isRegExp_ is *true*, then
               1. Let _flags_ be ? Get(_regexp_, *"flags"*).
@@ -35816,7 +35816,7 @@ <h1>String.prototype.replace ( _searchValue_, _replaceValue_ )</h1>
         <emu-alg>
           1. Let _O_ be the *this* value.
           1. Perform ? RequireObjectCoercible(_O_).
-          1. If _searchValue_ is neither *undefined* nor *null*, then
+          1. If _searchValue_ is an Object, then
             1. Let _replacer_ be ? GetMethod(_searchValue_, %Symbol.replace%).
             1. If _replacer_ is not *undefined*, then
               1. Return ? Call(_replacer_, _searchValue_, « _O_, _replaceValue_ »).
@@ -35930,7 +35930,7 @@ <h1>String.prototype.replaceAll ( _searchValue_, _replaceValue_ )</h1>
         <emu-alg>
           1. Let _O_ be the *this* value.
           1. Perform ? RequireObjectCoercible(_O_).
-          1. If _searchValue_ is neither *undefined* nor *null*, then
+          1. If _searchValue_ is an Object, then
             1. Let _isRegExp_ be ? IsRegExp(_searchValue_).
             1. If _isRegExp_ is *true*, then
               1. Let _flags_ be ? Get(_searchValue_, *"flags"*).
@@ -35975,7 +35975,7 @@ <h1>String.prototype.search ( _regexp_ )</h1>
         <emu-alg>
           1. Let _O_ be the *this* value.
           1. Perform ? RequireObjectCoercible(_O_).
-          1. If _regexp_ is neither *undefined* nor *null*, then
+          1. If _regexp_ is an Object, then
             1. Let _searcher_ be ? GetMethod(_regexp_, %Symbol.search%).
             1. If _searcher_ is not *undefined*, then
               1. Return ? Call(_searcher_, _regexp_, « _O_ »).
@@ -36020,7 +36020,7 @@ <h1>String.prototype.split ( _separator_, _limit_ )</h1>
         <emu-alg>
           1. Let _O_ be the *this* value.
           1. Perform ? RequireObjectCoercible(_O_).
-          1. If _separator_ is neither *undefined* nor *null*, then
+          1. If _separator_ is an Object, then
             1. Let _splitter_ be ? GetMethod(_separator_, %Symbol.split%).
             1. If _splitter_ is not *undefined*, then
               1. Return ? Call(_splitter_, _separator_, « _O_, _limit_ »).