[Chromedriver] Ensure baseURL is 'about:blank' for dummy page on Chrome 60+.

[Chromedriver] Ensure baseURL is 'about:blank' for dummy page on Chrome 60+.

From Chrome 60 onwards, root.baseURL will no longer be empty and will always return 'about:blank'.
See crbug/711562 for more details.

BUG=chromedriver:1775
Review-Url: https://codereview.chromium.org/2830903005
Cr-Commit-Position: refs/heads/master@{#467328}
Committed: https://chromium.googlesource.com/chromium/src/+/0889921a392dbd9305d9e747ffd8cdf01e14a22a

[gmanikpure, 2017-04-25 16:34:12]

Description was changed from

==========
[Chromedriver] Check baseURL is 'about:blank' for dummy page on Chrome 60+.

From Chrome 60 onwards, root.baseURL will no longer be empty and will always
return 'about:blank'.
See crbug/711562 for more details.

BUG=chromedriver:1775
==========

to

==========
[Chromedriver] Ensure baseURL is 'about:blank' for dummy page on Chrome 60+.

From Chrome 60 onwards, root.baseURL will no longer be empty and will always
return 'about:blank'.
See crbug/711562 for more details.

BUG=chromedriver:1775
==========

[gmanikpure, 2017-04-25 16:43:45]

gmanikpure@chromium.org changed reviewers:
+ johnchen@chromium.org, stgao@chromium.org

[gmanikpure, 2017-04-25 16:43:46]

Hi John,

This CL fixes https://bugs.chromium.org/p/chromedriver/issues/detail?id=1775

To make sure there are no regressions, I tested run_py_tests.py ,
run_java_tests.py, chromedriver_tests, chromedriver_unittests and also global
presubmit in google3.

https://test.corp.google.com/ui#cl=153781927
All the tests looks good. TGP has some flaky issues but not related to this CL.

Could you please take a look?

Thanks,
Gauri

[johnchen, 2017-04-25 23:56:08]

lgtm

[stgao, 2017-04-26 00:49:13]

On 2017/04/25 23:56:08, johnchen wrote:
> lgtm

lgtm

[gmanikpure, 2017-04-26 15:02:07]

The CQ bit was checked by gmanikpure@chromium.org

[commit-bot: I haz the power, 2017-04-26 15:02:27]

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-26 15:19:03]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493218927604970, "parent_rev":
"09b8b3a25d763b86f0f935edae3a0442188872e6", "commit_rev":
"0889921a392dbd9305d9e747ffd8cdf01e14a22a"}

[commit-bot: I haz the power, 2017-04-26 15:19:14]

Description was changed from

==========
[Chromedriver] Ensure baseURL is 'about:blank' for dummy page on Chrome 60+.

From Chrome 60 onwards, root.baseURL will no longer be empty and will always
return 'about:blank'.
See crbug/711562 for more details.

BUG=chromedriver:1775
==========

to

==========
[Chromedriver] Ensure baseURL is 'about:blank' for dummy page on Chrome 60+.

From Chrome 60 onwards, root.baseURL will no longer be empty and will always
return 'about:blank'.
See crbug/711562 for more details.

BUG=chromedriver:1775

Review-Url: https://codereview.chromium.org/2830903005
Cr-Commit-Position: refs/heads/master@{#467328}
Committed:
https://chromium.googlesource.com/chromium/src/+/0889921a392dbd9305d9e747ffd8...
==========

[commit-bot: I haz the power, 2017-04-26 15:19:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0889921a392dbd9305d9e747ffd8...