commit 6dada6be7f5340a06a1ae175226ccabbfe56ce0c
author Pirama Arumuga Nainar <pirama@google.com> Thu Aug 26 18:04:42 2021 -0700
committer Pirama Arumuga Nainar <pirama@google.com> Mon Aug 30 12:16:15 2021 -0700
tree f8c4383cec29d7be4f030fa348ef48e5b40571ba
parent 386643fc4dc34ad93d4b22d4b86c47e1952ab17e

[TimedTextUnitTest] use shared libstagefright_foundation instead of static

Bug: http://b/197965342

libmedia.so, a dependency of TimedTextUnitTest, also depends on
libstagefright_foundation.so.  If we include libstagefright_foundation.a
when linking TimedTextUnitTest, upstream's lld linker will unnecessarily
fetch symbols from libstagefright_foundation.a to redefine undefined
symbols from libmedia.so.  Switching to libstagefright_foundation.so
instead does not increase the closure of shared libraries needed for
this test.

Test: build with aosp/1809741 which has lld with the above behavior.
Change-Id: I4077a385f8c6233108f09a69260d0bda7a022099

media/libstagefright/timedtext/test/Android.bp

diff --git a/media/libstagefright/timedtext/test/Android.bp b/media/libstagefright/timedtext/test/Android.bp
index e8ee2ee..120819c 100644
--- a/media/libstagefright/timedtext/test/Android.bp
+++ b/media/libstagefright/timedtext/test/Android.bp
@@ -35,7 +35,6 @@
 
     static_libs: [
         "libstagefright_timedtext",
-        "libstagefright_foundation",
     ],
 
     include_dirs: [
@@ -46,6 +45,7 @@
         "liblog",
         "libmedia",
         "libbinder",
+        "libstagefright_foundation",
     ],
 
     cflags: [