Fix some gn_visibility TODOs.

Fix some gn_visibility TODOs.

Tested on Mac with the following command lines:
$ gn gen out-gn/Release --args='is_debug=false is_component_build=false symbol_level=1'
$ ninja -C out-gn/Release gn_visibility

BUG=504076
TEST=see above
R=dpranke@chromium.org

Committed: https://crrev.com/5132ba00872ecc7a9f1a71238bf390e09359ff5b
Cr-Commit-Position: refs/heads/master@{#343120}

[tfarina, 2015-08-12 02:55:01]

Is this correct?

[Dirk Pranke, 2015-08-12 19:03:19]

lgtm

https://codereview.chromium.org/1283133002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1283133002/diff/1/BUILD.gn#newcode475
BUILD.gn:475: "//v8:postmortem-metadata",
You shouldn't list these targets in two places (and GN won't let you anyway,
because of the visibility rules).

[Dirk Pranke, 2015-08-12 19:03:30]

argh, not-lgtm.

[tfarina, 2015-08-12 19:17:45]

ptal.

https://codereview.chromium.org/1283133002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1283133002/diff/1/BUILD.gn#newcode475
BUILD.gn:475: "//v8:postmortem-metadata",
On 2015/08/12 19:03:18, Dirk Pranke wrote:
> You shouldn't list these targets in two places (and GN won't let you anyway,
> because of the visibility rules).
yep, bot caught this as well. Removed.

[Dirk Pranke, 2015-08-12 19:18:20]

lgtm

[tfarina, 2015-08-12 23:14:48]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2015-08-12 23:15:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1283133002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1283133002/20001

[commit-bot: I haz the power, 2015-08-12 23:20:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-12 23:41:22]

Patchset 2 (id:??) landed as
https://crrev.com/5132ba00872ecc7a9f1a71238bf390e09359ff5b
Cr-Commit-Position: refs/heads/master@{#343120}