204b133883a57daad41500d4502e45ab88b7d35c - platform_frameworks_base

commit	204b133883a57daad41500d4502e45ab88b7d35c	[log] [tgz]
author	Soonil Nagarkar <sooniln@google.com>	Tue Feb 02 15:29:23 2021 -0800
committer	Soonil Nagarkar <sooniln@google.com>	Wed Feb 03 17:08:16 2021 +0000
tree	44307a93ccf1a7d2af6388019f7243f75b622c18
parent	9ee2e0d368836a53154f45c7b6d4a34614175481 [diff]

Reduce LocationManager memory usage

Move GNSS listener overhead into statics so it's shared per process, and
do not instantiate by default.

Previously GNSS requests were multiplexed by LocationManager instance,
this is now removed. This could mean additional binder overhead, but
it's pretty unlikely that this feature was in active use anywhere, as in
any large application that has multiple gnss requests, they are usually
coming from different contexts (and thus LocationManager instances) and
would not have seen the benefit of multiplexing anyways.

Bug: 179261871
Test: atest CtsLocationFineTestCases
Change-Id: I8b7a41c13dfe219c326b1e519483d89d2b721ce5

core/java/com/android/internal/listeners/ListenerTransport.java[diff]
core/java/com/android/internal/listeners/ListenerTransportManager.java[Added - diff]
core/java/com/android/internal/listeners/ListenerTransportMultiplexer.java[Deleted - diff]
core/java/com/android/internal/listeners/RequestListenerTransport.java[Deleted - diff]
core/tests/coretests/src/com/android/internal/listeners/ListenerTransportMultiplexerTest.java[Deleted - diff]
location/java/android/location/LocationManager.java[diff]

6 files changed