Better handle safe_browsing logic in build dependencies.

chrome/common/features.gni

Index: chrome/common/features.gni
diff --git a/chrome/common/features.gni b/chrome/common/features.gni
index 045bb9f21b20b52b03b908f4bdbd95da7e65f530..2ae944573e5f8f9f67ca82e3d1f51a4772fd86d7 100644
--- a/chrome/common/features.gni
+++ b/chrome/common/features.gni
@@ -8,6 +8,7 @@
 import("//build/config/chrome_build.gni")
 import("//build/config/chromecast_build.gni")
 import("//build/config/compiler/compiler.gni")
+import("//build/config/features.gni")
 
 # Please keep features in alphabetical order.
 declare_args() {
@@ -58,5 +59,6 @@ chrome_grit_defines = [
   "enable_google_now=$enable_google_now",
   "enable_hotwording=$enable_hotwording",
   "enable_vr_shell=$enable_vr_shell",
+  "safe_browsing_mode=$safe_browsing_mode",

[slan, 2016/09/21 23:41:59]

hmm... AFAIK, this string will resolve to something containing an int (ex. 
"safe_browsing_mode=2")

I'm not 100% positive of that, but I don't know how else it would know to
resolve this as a bool. You probably need to do something like
(safe_browsing_mode != 0)

You can check by calling print(chrome_grit_defines) below.

P.S. I'm also not sure how grit interprets an int... Clearly it seems to be
working ok

[mbjorge, 2016/09/22 15:02:50]

https://www.chromium.org/developers/tools-we-use-in-chromium/grit/grit-users-...

"""
 The 'if' element has a single attribute 'expr', the contents of which are
evaluated as a Python expression, and the resources within the 'if' element are
only output if the result of that expression is true.
"""

   "use_vulcanize=$use_vulcanize",
 ]