Script execution: Throw error for invalid reference argument

[yezhizhen]

According to spec, the window/frame/element/shadow root reference must be String.

1. Let node id be a new globally unique string. (https://w3c.github.io/webdriver/#dfn-get-or-create-a-node-reference)
2. Each browsing context has an associated window handle which uniquely identifies it. This must be a String and must not be "current".

However, current spec does not report error for this, even though there are tests created 2 years ago
https://github.com/web-platform-tests/wpt/blob/427276cdcaf28068aa1a2dae9206f03df716d367/webdriver/tests/classic/execute_async_script/arguments.py#L165-L167 for this.

This PR add the step to throw error if reference is not string.

---

Preview | Diff

[yezhizhen]

cc @xiaochengh

[whimboo]

The changes look fine to me. But there is a problem with the IPR check.

@yezhizhen you should probably join the group so that this wont be an issue in the future anymore.

[w3cbot]

whimboo marked as non substantive for IPR from ash-nazg.

[yezhizhen]

The changes look fine to me. But there is a problem with the IPR check.

@yezhizhen you should probably join the group so that this wont be an issue in the future anymore.

Thanks for inviting. I will try to clear some internal procedures required for it..