DescriptionConvert tests to use GetURL(host, path) instead of redirector.
As discovered by japhet@, if we navigate cross-site in a browser test using the '/cross-site/' redirector syntax, the initial URL is relative to the embedded_test_server, which happens to also be the SiteInstance of the top-level page. This causes an extra process swap, which is unnecessary and can create unexpected effects in the test. I've added GetURL(hostname, path) method to EmbeddedTestServer to give us the proper URL directly without following a redirect path and this CL updates the tests to use this new method.
BUG=418236
Committed: https://crrev.com/30374c75c1736fe2a476382edcadd68fc5ddc0d0
Cr-Commit-Position: refs/heads/master@{#302126}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|