Commit 19814592 authored by Vadym Doroshenko's avatar Vadym Doroshenko Committed by Commit Bot

[Password Manager] Fix browser test.

No need to wait a navigation since NavigateToURL waits for navigation
completed (it eventually calls
NavigateToURLWithDispositionBlockUntilNavigationsComplete).

Bug: 875052
Change-Id: I96ebd7d5ec4ce0f676a82f6714816a7ddefb2b08
Reviewed-on: https://chromium-review.googlesource.com/1179160Reviewed-by: default avatarVaclav Brozek <vabr@chromium.org>
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584011}
parent 2db0839d
......@@ -3862,9 +3862,8 @@ IN_PROC_BROWSER_TEST_P(PasswordManagerBrowserTestWithViewsFeature,
const GURL url = https_test_server().GetURL("accounts.google.com",
"/password/gaia_reath_form.html");
NavigationObserver observer(WebContents());
ui_test_utils::NavigateToURL(browser(), url);
observer.Wait();
// Expects no requests to the password store. So no filling.
EXPECT_EQ(0, password_store->fill_matching_logins_calls());
}
......@@ -3881,9 +3880,8 @@ IN_PROC_BROWSER_TEST_P(PasswordManagerBrowserTestWithViewsFeature,
const GURL url = https_test_server().GetURL(
"accounts.google.com", "/password/password_form.html?ssp=1");
NavigationObserver observer(WebContents());
ui_test_utils::NavigateToURL(browser(), url);
observer.Wait();
// Expects no requests to the password store. So no filling.
EXPECT_EQ(0, password_store->fill_matching_logins_calls());
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment