The documentation for ui_test_utils::NavigateToURLWithDisposition says that it waits for navigations to complete. However, the code doesn't wait if the last param is BROWSER_TEST_NONE. This is confusing, would be better to just rename BROWSER_TEST_NONE.