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

Unified Diff: build/config/features.gni

Issue 2360853002: Better handle safe_browsing logic in build dependencies. (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « no previous file | chrome/browser/BUILD.gn » ('j') | chrome/common/features.gni » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/features.gni
diff --git a/build/config/features.gni b/build/config/features.gni
index 561c4d352ab4bcf8065035beebc2ce2ce3ae6ac1..1f7ae344b1969fc02aa7e6abf01db8ee830dc71a 100644
--- a/build/config/features.gni
+++ b/build/config/features.gni
@@ -79,10 +79,10 @@ declare_args() {
# safe browsing feature. Safe browsing can be compiled in 3 different levels:
# 0 disables it, 1 enables it fully, and 2 enables mobile protection via an
# external API.
- if (is_android) {
- safe_browsing_mode = 2
- } else if (is_ios || is_chromecast) {
+ if (is_ios || is_chromecast) {
safe_browsing_mode = 0
+ } else if (is_android) {
+ safe_browsing_mode = 2
} else {
safe_browsing_mode = 1
}
« no previous file with comments | « no previous file | chrome/browser/BUILD.gn » ('j') | chrome/common/features.gni » ('J')

Powered by Google App Engine
This is Rietveld 408576698