diff options
| author | Chiachang Wang <chiachangwang@google.com> | 2020-09-28 10:31:49 +0800 |
|---|---|---|
| committer | Chiachang Wang <chiachangwang@google.com> | 2020-10-07 06:48:15 +0000 |
| commit | 8c1372ed1deaec8e013055dfd1f5211679564840 (patch) | |
| tree | fed7347608739b69517f9e06ea1b82675791cc59 /tools/dump-coverage/dump_coverage.cc | |
| parent | c3207fbf8d436b92aa6b0f793aa01048d7faea97 (diff) | |
Update to positive logic to better match the method naming
The method checkInterfacePresent seems that should return if the
interface is present or not, but it actually return an opposite
result. It may confuse the caller. E.g. The override method in
VpnTest assume it always returns true in the test. It will fail
the vpn start flow due to interface gone.
Bug: 169215213
Test: atest FrameworksNetTess
Change-Id: I3e9a64f0900336b8205674b94db68f090b4e9ff9
Diffstat (limited to 'tools/dump-coverage/dump_coverage.cc')
0 files changed, 0 insertions, 0 deletions