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

Unified Diff: components/safe_browsing/password_protection/BUILD.gn

Issue 2912383004: Fill is_chrome_signin_password field in the password entry pings. (Closed)
Patch Set: address nparker's comments Created 3 years, 6 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/BUILD.gn
diff --git a/components/safe_browsing/password_protection/BUILD.gn b/components/safe_browsing/password_protection/BUILD.gn
index a7be564812e0e0b3511343b118915ce1a8a72ab4..dcd862fa27f18c9d518455342cd332034b4235d7 100644
--- a/components/safe_browsing/password_protection/BUILD.gn
+++ b/components/safe_browsing/password_protection/BUILD.gn
@@ -3,48 +3,54 @@
# found in the LICENSE file.
source_set("password_protection") {
- sources = [
- "password_protection_request.cc",
- "password_protection_request.h",
- "password_protection_service.cc",
- "password_protection_service.h",
- ]
+ if (!is_android && !is_ios) {
+ sources = [
+ "password_protection_request.cc",
+ "password_protection_request.h",
+ "password_protection_service.cc",
+ "password_protection_service.h",
+ ]
- public_deps = [
- "//google_apis:google_apis",
- ]
+ public_deps = [
+ "//google_apis:google_apis",
+ ]
- deps = [
- "//base:base",
- "//components/content_settings/core/browser:browser",
- "//components/data_use_measurement/core:core",
- "//components/history/core/browser:browser",
- "//components/safe_browsing:csd_proto",
- "//components/safe_browsing_db:database_manager",
- "//components/safe_browsing_db:v4_protocol_manager_util",
- "//content/public/browser:browser",
- "//net:net",
- "//third_party/protobuf:protobuf_lite",
- ]
+ deps = [
+ "//base:base",
+ "//components/content_settings/core/browser:browser",
+ "//components/data_use_measurement/core:core",
+ "//components/history/core/browser:browser",
+ "//components/password_manager/core/browser:browser",
+ "//components/safe_browsing:csd_proto",
+ "//components/safe_browsing_db:database_manager",
+ "//components/safe_browsing_db:v4_protocol_manager_util",
+ "//content/public/browser:browser",
+ "//net:net",
+ "//third_party/protobuf:protobuf_lite",
+ ]
+ }
}
source_set("password_protection_unittest") {
testonly = true
- sources = [
- "password_protection_service_unittest.cc",
- ]
- deps = [
- ":password_protection",
- "//base",
- "//base/test:test_support",
- "//components/content_settings/core/browser:browser",
- "//components/history/core/browser:browser",
- "//components/safe_browsing_db:test_database_manager",
- "//components/sync_preferences:test_support",
- "//content/test:test_support",
- "//net:test_support",
- "//testing/gmock",
- "//testing/gtest",
- "//third_party/protobuf:protobuf_lite",
- ]
+ if (!is_android && !is_ios) {
+ sources = [
+ "password_protection_service_unittest.cc",
+ ]
+ deps = [
+ ":password_protection",
+ "//base",
+ "//base/test:test_support",
+ "//components/content_settings/core/browser:browser",
+ "//components/history/core/browser:browser",
+ "//components/password_manager/core/browser:browser",
+ "//components/safe_browsing_db:test_database_manager",
+ "//components/sync_preferences:test_support",
+ "//content/test:test_support",
+ "//net:test_support",
+ "//testing/gmock",
+ "//testing/gtest",
+ "//third_party/protobuf:protobuf_lite",
+ ]
+ }
}
« no previous file with comments | « components/password_manager/core/browser/password_store_unittest.cc ('k') | components/safe_browsing/password_protection/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698