Convert GN visibility to be a list.

tools/gn/variables.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "tools/gn/variables.h"
 
 namespace variables {
 
 // Built-in variables ----------------------------------------------------------
 
@@ skipping 805 matching lines @@
     "    testonly = true\n"
     "    ...\n"
     "  }\n";
 
 const char kVisibility[] = "visibility";
 const char kVisibility_HelpShort[] =
     "visibility: [label list] A list of labels that can depend on a target.";
 const char kVisibility_Help[] =
     "visibility: A list of labels that can depend on a target.\n"
     "\n"
-    "  A label or a list of labels and label patterns that define which\n"
-    "  targets can depend on the current one. These permissions are checked\n"
-    "  via then \"check\" command (see \"gn help check\").\n"
+    "  A list of labels and label patterns that define which targets can\n"
+    "  depend on the current one. These permissions are checked via the\n"
+    "  \"check\" command (see \"gn help check\").\n"
     "\n"
     "  If visibility is not defined, it defaults to public (\"*\").\n"
     "\n"
     "  If visibility is defined, only the targets with labels that match it\n"
     "  can depend on the current target. The empty list means no targets\n"
     "  can depend on the current target.\n"
     "\n"
     "  Tip: Often you will want the same visibility for all targets in a\n"
     "  BUILD file. In this case you can just put the definition at the top,\n"
     "  outside of any target, and the targets will inherit that scope and see\n"
     "  the definition.\n"
     "\n"
     "Patterns\n"
     "\n"
     "  See \"gn help label_pattern\" for more details on what types of\n"
     "  patterns are supported. If a toolchain is specified, only targets\n"
     "  in that toolchain will be matched. If a toolchain is not specified on\n"
     "  a pattern, targets in all toolchains will be matched.\n"
     "\n"
     "Examples\n"
     "\n"
     "  Only targets in the current buildfile (\"private\", the default):\n"
-    "    visibility = \":*\"\n"
+    "    visibility = [ \":*\" ]\n"
     "\n"
     "  No targets (used for targets that should be leaf nodes):\n"
     "    visibility = []\n"
     "\n"
     "  Any target (\"public\"):\n"
-    "    visibility = \"*\"\n"
+    "    visibility = [ \"*\" ]\n"
     "\n"
     "  All targets in the current directory and any subdirectory:\n"
-    "    visibility = \"./*\"\n"
+    "    visibility = [ \"./*\" ]\n"
     "\n"
     "  Any target in \"//bar/BUILD.gn\":\n"
-    "    visibility = \"//bar:*\"\n"
+    "    visibility = [ \"//bar:*\" ]\n"
     "\n"
     "  Any target in \"//bar/\" or any subdirectory thereof:\n"
-    "    visibility = \"//bar/*\"\n"
+    "    visibility = [ \"//bar/*\"\n ]"
     "\n"
     "  Just these specific targets:\n"
     "    visibility = [ \":mything\", \"//foo:something_else\" ]\n"
     "\n"
     "  Any target in the current directory and any subdirectory thereof, plus\n"
     "  any targets in \"//bar/\" and any subdirectory thereof.\n"
     "    visibility = [ \"./*\", \"//bar/*\" ]\n";
 
 // -----------------------------------------------------------------------------
 
@@ skipping 62 matching lines @@
     INSERT_VARIABLE(Sources)
     INSERT_VARIABLE(Testonly)
     INSERT_VARIABLE(Visibility)
   }
   return info_map;
 }
 
 #undef INSERT_VARIABLE
 
 }  // namespace variables