Commit eefe0105 authored by Vaclav Brozek's avatar Vaclav Brozek Committed by Commit Bot

Fix NewPasswordFormManager test

The test has a helper method CreateFormManager, which takes
a FormData argument and should build a NewPasswordFormManager
for that FormData.

However, due to a typo, that method always builds the NPFM
for |observed_form_| instead.

This CL fixes that typo.

Bug: 831123

Change-Id: Ibb313866f8f04d444f68f4af046716ac5ad264be
Reviewed-on: https://chromium-review.googlesource.com/c/1262515Reviewed-by: default avatarVadym Doroshenko <dvadym@chromium.org>
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597051}
parent f6ef1878
......@@ -208,7 +208,7 @@ class NewPasswordFormManagerTest : public testing::Test {
fetcher_.reset(new FakeFormFetcher());
fetcher_->Fetch();
form_manager_.reset(new NewPasswordFormManager(
&client_, driver_.AsWeakPtr(), observed_form_, fetcher_.get(),
&client_, driver_.AsWeakPtr(), observed_form, fetcher_.get(),
std::make_unique<NiceMock<MockFormSaver>>()));
}
};
......
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