diff options
| author | vabr <vabr@chromium.org> | 2015-08-07 04:02:04 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-07 11:02:34 +0000 |
| commit | 5410d0bc855495b61ca084a8760d32613f684d01 (patch) | |
| tree | f850d4da7b5ac7a1af6ae2da78ac8699a4f7e4af /components/password_manager/DEPS | |
| parent | 97b25ca5565f00404e32f4b3215d207b3a79399b (diff) | |
| download | chromium_src-5410d0bc855495b61ca084a8760d32613f684d01.zip chromium_src-5410d0bc855495b61ca084a8760d32613f684d01.tar.gz chromium_src-5410d0bc855495b61ca084a8760d32613f684d01.tar.bz2 | |
Move ShouldFilterAutofillResult from ChromePasswordManagerClient to PasswordManager
The goal is to put as much of the platform-independent logic to the shared code, and leave the CPMC just a dumb redirector to the embedding system. This is one of many steps.
Also, this CL removes IsPasswordManagementEnabledForCurrentPage() from the client mock in the client unit test. It is dangerous to mix mocking methods with testing the real implementations, and none of the tests actually needs this method mocked.
BUG=474577, 515108
Review URL: https://codereview.chromium.org/1260263002
Cr-Commit-Position: refs/heads/master@{#342332}
Diffstat (limited to 'components/password_manager/DEPS')
| -rw-r--r-- | components/password_manager/DEPS | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/components/password_manager/DEPS b/components/password_manager/DEPS index d16b4f6..8c2766e 100644 --- a/components/password_manager/DEPS +++ b/components/password_manager/DEPS @@ -13,6 +13,7 @@ include_rules = [ "+third_party/sqlite/sqlite3.h", "+ui", # PasswordManager is a layered component; subdirectories must explicitly - # introduce the ability to use the content layer as appropriate. + # introduce the ability to use non-core layers as appropriate. "-components/password_manager/content", + "-components/password_manager/sync", ] |