[string] Fix null handling in MaybeCallFunctionAtSymbol

[string] Fix null handling in MaybeCallFunctionAtSymbol

The spec requires that a null @@split / @@replace symbol is treated exactly the
same as if it were undefined, i.e. execution should move on to the default
implementation instead of throwing a TypeError.

BUG=v8:6313
Review-Url: https://codereview.chromium.org/2845153002
Cr-Commit-Position: refs/heads/master@{#44959}
Committed: https://chromium.googlesource.com/v8/v8/+/281488547744d3341cd9c1d92489696420ee63c1

[jgruber, 2017-04-27 16:27:12]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-27 16:27:19]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2017-04-27 16:35:02]

jgruber@chromium.org changed reviewers:
+ bmeurer@chromium.org, yangguo@chromium.org

[jgruber, 2017-04-27 16:35:03]

ptal

[jgruber, 2017-04-27 16:36:33]

Description was changed from

==========
[string] Fix null handling in MaybeCallFunctionAtSymbol

The spec requires that a null @@split / @@replace symbol is treated exactly the
same as if it were undefined, i.e. execution should move on to the default
implementation.

BUG=v8:6313
==========

to

==========
[string] Fix null handling in MaybeCallFunctionAtSymbol

The spec requires that a null @@split / @@replace symbol is treated exactly the
same as if it were undefined, i.e. execution should move on to the default
implementation instead of throwing a TypeError.

BUG=v8:6313
==========

[jgruber, 2017-04-27 16:36:53]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-27 16:36:58]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Benedikt Meurer, 2017-04-27 16:47:34]

LGTM, thanks!

[commit-bot: I haz the power, 2017-04-27 17:04:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-27 17:04:19]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-04-28 05:56:56]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-04-28 05:57:02]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-28 05:58:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493359016304440,
"parent_rev": "53d31d0623bd88ce9a0041a767f36eafd3f1de8b", "commit_rev":
"281488547744d3341cd9c1d92489696420ee63c1"}

[commit-bot: I haz the power, 2017-04-28 05:58:23]

Description was changed from

==========
[string] Fix null handling in MaybeCallFunctionAtSymbol

The spec requires that a null @@split / @@replace symbol is treated exactly the
same as if it were undefined, i.e. execution should move on to the default
implementation instead of throwing a TypeError.

BUG=v8:6313
==========

to

==========
[string] Fix null handling in MaybeCallFunctionAtSymbol

The spec requires that a null @@split / @@replace symbol is treated exactly the
same as if it were undefined, i.e. execution should move on to the default
implementation instead of throwing a TypeError.

BUG=v8:6313

Review-Url: https://codereview.chromium.org/2845153002
Cr-Commit-Position: refs/heads/master@{#44959}
Committed:
https://chromium.googlesource.com/v8/v8/+/281488547744d3341cd9c1d92489696420e...
==========

[commit-bot: I haz the power, 2017-04-28 05:58:24]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/281488547744d3341cd9c1d92489696420e...