Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(279)

Unified Diff: components/safe_browsing/password_protection/DEPS

Issue 2912383004: Fill is_chrome_signin_password field in the password entry pings. (Closed)
Patch Set: clean up DEPS file Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/safe_browsing/password_protection/DEPS
diff --git a/components/safe_browsing/password_protection/DEPS b/components/safe_browsing/password_protection/DEPS
index f04e7f6794a557d26bdaa8b0415bdb477ca7a4fa..04ee58694da1ca40004cfb63dc94863b25dbf8f8 100644
--- a/components/safe_browsing/password_protection/DEPS
+++ b/components/safe_browsing/password_protection/DEPS
@@ -1,12 +1,7 @@
include_rules = [
- "-content",
"+components/content_settings/core/browser",
"+components/history/core/browser",
- "+components/safe_browsing/csd.pb.h",
- "+components/safe_browsing_db",
Nathan Parker 2017/05/31 22:40:55 how is this not needed? Or is it included from the
vakh (use Gerrit instead) 2017/06/01 00:02:25 Yes, but I appreciate the RED color :)
Jialiu Lin 2017/06/01 01:22:01 Acknowledged.
Jialiu Lin 2017/06/01 01:22:01 Yes, these are already included in the components/
"+components/sync_preferences/testing_pref_service_syncable.h",
- "+content/public/browser",
"+content/public/test",
- "+net",
]

Powered by Google App Engine
This is Rietveld 408576698