tree f5fd80eae32908c02c648ede197553ec1b987990
parent e1dfbe0cfa646b5c7a1f2355ac0da8187abf9a88
author Xiao Ma <xiaom@google.com> 1651024565 +0900
committer Cherrypicker Worker <android-build-cherrypicker-worker@google.com> 1651329765 +0000

Unicast state of existing ethernet interfaces when registering new Ethernet Listener.

When adding a listener via the addInterfaceStateListener API, the
expectation is that ethernet service calls that listener immediately
with a list of all interfaces and their respective state. However,
this doesn't work as expectation due to the EthernetManager stores
all registered listeners within one global service listener class,
the new registered listener will be invoked until the state of any
interface has changed.

Instead of storing all registered listeners within one global service
listener class, associate each new registered listener to an instance
of IEthernetServiceListener, and pass it to the EthernetService, that
will be invoked when iterating the state of all existing interfaces.

In order to avoid the scanning of all registered listeners, that would
be slow if there are lots of registered callbacks, moving the service
listener to the ListenerInfo class, where the service listener can
access the executor and listener passed from the caller naturely,
unnecessary to iterate the whole list.

Bug: 229125889
Test: atest android.net.cts.EthernetManagerTest --iterations
Change-Id: I111eba78d1066794c414ab5fb4a31258ea311413
(cherry picked from commit bb9a49cf72ff83cf969a86d3c51087fa77f08d69)
Merged-In: I111eba78d1066794c414ab5fb4a31258ea311413