tree 8575c20d94df4abe06a48d8e87cc75a9d4798ca6
parent c861b10841cbc6610e541a581abb4f80567a467b
author junyulai <junyulai@google.com> 1588155932 +0800
committer Junyu Lai <junyulai@google.com> 1588226264 +0000

Use CopyOnWriteArrayList to store list of NetworkStatsProviderCb

In NetworkStatsService, mStatsLock will be held when
iterating through the provider list. This is to protect the list
from concurrent broadcast that triggered by
NetworkPolicyManagerService. This is not good since the binder
call is oneway, it does not make sense to block every
access to the providers.

This change also remove unuse variable and reduce verbose log.

Test: atest FrameworksNetTests TetheringTests
Bug: 150418178
Change-Id: If74e9f2ea597a0d5ae4668c3358bc687f342bbb5