tree 2d0c7aa948317f426363c75d443a2fda05f2b900
parent 8e189bf34dc75cc237ffc24dd90b9e057d00ba03
author Vaclav Brozek <vabr@chromium.org> 1500309386 +0000
committer Commit Bot <commit-bot@chromium.org> 1500309386 +0000

Don't try to enable saving password in EG tests

The password settings EG tests currently try to enable saving passwords in
PrefService. This is not necesary, because saving passwords is enabled by
default (see PasswordManager::RegisterProfilePrefs).

It is also harmful, because it:
(1) Won't catch if the default setting being "enabled" regresses, or some other
    tests start failing to clean-up.
(2) Increases the complexity of the code.

Therefore, this CL removes the code which enables the password setting in the
passwords settings EG tests.

Bug: 744058
Change-Id: I315935319cbaacb871ba6214b832c04902afffa9
Reviewed-on: https://chromium-review.googlesource.com/572903
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487109}