update selenium-webdriver

[benitogf]

No description provided.

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[NickTomlin]

We should be very careful when updating dependencies like Webdriver. In addition, you've commented out some linting related code without an explanation.

Could you provide some reasoning behind these changes? If not, i'll go ahead and close this PR in 3 days. Thanks!

[benitogf]

Hello Nick, about the commented lines, both commands (npm install, tsc) produce no output so checking that to determine if there was an error is breaking this test on my side, please let me know if I'm missing something about it

[googlebot]

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

[googlebot]

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

[benitogf]

cla signed

[googlebot]

CLAs look good, thanks!

[googlebot]

CLAs look good, thanks!

[csvan]

Any news on this? What is needed to get it included? The older Selenium version in Protractor is a problem for running tests in Chrome Headless.

[timbru31]

Is this still relevant or can this (stale) PR closed? Because 0fbc2c0 landed and bumped the version of selenium-webdriver.

[qiyigg]

Already updated to 3.6.0. Close the PR