-
Brett Wilson authored
This also reworks visibility so both it and the testonly flag are checked when a target is marked resolved, rather than when it is written out. The previous code would not check visibility when doing non-"gen" commands like "check", which is counterintuitive. This change required OnResolved to be able to report a failure, which required updating many callers. But it makes visibility mush easier to test, so I added some additional visibility tests. BUG=357779 R=hclam@chromium.org Review URL: https://codereview.chromium.org/524623005 Cr-Commit-Position: refs/heads/master@{#292976}
85423a03
