commit 138d4ac54d9eebbdbe0469f301ec64cbf6731af3
author Karl Wiberg <kwiberg@webrtc.org> Mon Oct 16 11:16:19 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Oct 16 11:04:07 2017 +0000
tree dc6a28feaae62ef398cad64cd8bd75bf9e62c005
parent 6fa6dd4f83750b3dcacbf236a83033b1ac017912

GN rtc_* templates: Forward global "visibility"

`forward_variables_from(invoker, "*")` forwards all locally declared
variables, but not global variables. This means that while setting
`visibility` locally in an rtc_* build target works fine, it does not
currently work to set `visibility` globally for all build targets in a
file.

Fix this by manually forwarding `visibility`.

BUG=webrtc:8254, webrtc:8255

Change-Id: I9e1a5f8ac9cb5991fff2af7c094fe677e1483964
Reviewed-on: https://webrtc-review.googlesource.com/10806
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20305}

webrtc.gni

diff --git a/webrtc.gni b/webrtc.gni
index b36352d..de1c5e9 100644
--- a/webrtc.gni
+++ b/webrtc.gni
@@ -267,7 +267,9 @@
                              "configs",
                              "public_configs",
                              "suppressed_configs",
+                             "visibility",
                            ])
+    forward_variables_from(invoker, [ "visibility" ])
     configs += invoker.configs
     configs -= rtc_remove_configs
     configs -= invoker.suppressed_configs
@@ -290,7 +292,9 @@
                              "configs",
                              "public_configs",
                              "suppressed_configs",
+                             "visibility",
                            ])
+    forward_variables_from(invoker, [ "visibility" ])
     configs += invoker.configs
     configs -= rtc_remove_configs
     configs -= invoker.suppressed_configs
@@ -310,7 +314,9 @@
                              "configs",
                              "public_configs",
                              "suppressed_configs",
+                             "visibility",
                            ])
+    forward_variables_from(invoker, [ "visibility" ])
     configs += invoker.configs
     configs -= rtc_remove_configs
     configs -= invoker.suppressed_configs
@@ -334,7 +340,9 @@
                              "configs",
                              "public_configs",
                              "suppressed_configs",
+                             "visibility",
                            ])
+    forward_variables_from(invoker, [ "visibility" ])
     configs += invoker.configs
     configs -= rtc_remove_configs
     configs -= invoker.suppressed_configs
@@ -353,7 +361,9 @@
                              "configs",
                              "public_configs",
                              "suppressed_configs",
+                             "visibility",
                            ])
+    forward_variables_from(invoker, [ "visibility" ])
     configs += invoker.configs
     configs -= rtc_remove_configs
     configs -= invoker.suppressed_configs
@@ -378,7 +388,9 @@
                                "configs",
                                "public_configs",
                                "suppressed_configs",
+                               "visibility",
                              ])
+      forward_variables_from(invoker, [ "visibility" ])
       configs += invoker.configs
       configs -= rtc_remove_configs
       configs -= invoker.suppressed_configs