Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always use JSON.parse for bot.json.parse #4603

Merged
merged 1 commit into from
Sep 1, 2017
Merged

Conversation

vrana
Copy link
Contributor

@vrana vrana commented Aug 30, 2017

goog.json.parse is deprecated and about to be removed in late September 2017. This change prepares for that.

goog.json.parse is deprecated and about to be removed in late September 2017. This change prepares for that.
@juangj juangj merged commit b5999f6 into SeleniumHQ:master Sep 1, 2017
@juangj
Copy link
Contributor

juangj commented Sep 1, 2017

I wonder if anybody is setting NATIVE_JSON=false anymore when compiling. I wouldn't think so... IE7 was probably the last one that needed it.

@juangj
Copy link
Contributor

juangj commented Sep 1, 2017

Oh, hm.

flags.push("--define=bot.json.NATIVE_JSON=false")

Regardless, I would think that's unnecessary at this point.

Tom-Trumper pushed a commit to Tom-Trumper/selenium that referenced this pull request Sep 11, 2017
goog.json.parse is deprecated and about to be removed in late September 2017. This change prepares for that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants