Fix some gn_visibility TODOs.

BUILD.gn

Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index 6bde7c0e2bdeaf735027cf86f4823d0ceab66c1c..8cbfd549b7060a84efed26603b2648c7a3daae8b 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -471,8 +471,8 @@ group("both_gn_and_gyp") {
       "//ui/snapshot:snapshot_unittests",
       "//ui/views/examples:views_examples_with_content_exe",
 
-      # "//v8:v8_snapshot",  # TODO(GYP): visibility?
-      # "//v8:postmortem-metadata",  # TODO(GYP): visibility?
+      "//v8:v8_snapshot",
+      "//v8:postmortem-metadata",

[Dirk Pranke, 2015/08/12 19:03:18]

You shouldn't list these targets in two places (and GN won't let you anyway,
because of the visibility rules).

[tfarina, 2015/08/12 19:17:44]

yep, bot caught this as well. Removed.

 
       "//third_party/codesighs:nm2tsv",
       "//third_party/sqlite:sqlite_shell",
@@ -742,8 +742,8 @@ group("gn_visibility") {
 
     "//ui/resources:repack_ui_test_mac_locale_pack",
 
-    # "//v8:v8_snapshot",  # TODO(GYP): visibility?
-    # "//v8:postmortem-metadata",  # TODO(GYP): visibility?
+    "//v8:v8_snapshot",
+    "//v8:postmortem-metadata",
   ]
 }