History log of /frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
6969da62b068aff2aee650d57590532f8fc8de69 13-Feb-2018 Dongwon Kang <dwkang@google.com> Revert "Remove IDataSource dependency from DataSource."

This reverts commit 514674257ef6954a7479ac93a26a07061f54462d.

The original change Iaba6d9be is no longer needed since we
separated out DataSourceBase from DataSource. Now, DataSource
may have a method relying on a class from libmedia, IDataSource.

Test: checked the correct wrapping path with 'dumpsys media.extractor'
Bug: 72869975

Change-Id: Ib0d5ca863fa70a4a96f97b3377c209e1f288f443
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
17e172b4c3c87ecaa7c87eecc42b4dc47e3e9734 30-Jan-2018 Dongwon Kang <dwkang@google.com> Remove String8 and AMessage from extractor plugin interface.

- mime(String8): is dropped from the interface since it was only used
for debug message. debug message is replaced with MediaExtractor::name.
- meta(AMessage): is replaced with an opaque pointer, void *, in addition
to an additional function pointer to free the object.
(In MPEG4Extractor case, AMessage with "meta-data-size", which was
introduced in Id2acdde89, is removed since the use case had been gone
with AwesomePlayer.)

Test: cts-tradefed run cts-dev --module CtsMediaTestCases --compatibility:module-arg
CtsMediaTestCases:include-annotation:android.platform.test.annotations.RequiresDevice
Bug: 67908556
Change-Id: I242944332bf0ff7dd4ad2106fe8f68e97264010b
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
2a243f08193fe9ff1afe018e9953f01c44ced9de 30-Jan-2018 Marco Nelissen <marcone@google.com> Remove RefBase from plugin-side DataSource

Extractor plugins now use DataSourceBase, which is not refcounted.
Framework-side DataSource derives from DataSourceBase and RefBase.
Also remove MediaExtractor::release, because it is not needed
anymore due to the way the extractor/track/datasource lifecycle
is managed now.
Also add source and target pid to CallbackDataSource name, to
make the extractor dumpsys more useful.

Bug: 67908556
Test: build, boot, CTS, skia test app for heif
Change-Id: I6b9cbe903f76b0cf27ba87f4a456f424803efd08
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
f3b8a779fae3ce135ef066a9ddddba77c07a8cae 26-Jan-2018 Marco Nelissen <marcone@google.com> Fix media.extractor dumpsys crash

Bug: 72551474
Test: build, boot, dumpsys
Change-Id: If4079a3d2e4e997aeb63c2529aacdbfff1dccc8e
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
ab7429b4cec5e619f0ed77d13363da4167212874 23-Jan-2018 Marco Nelissen <marcone@google.com> Merge "Remove unnecessary includes and forward declarations"
74065036f03cb9f96acb43c6e4f105ec91132395 23-Jan-2018 Marco Nelissen <marcone@google.com> Remove unnecessary includes and forward declarations

Bug: 67908544
Test: build
Change-Id: I824625940ba562571b16ed21e351ee17807d7d30
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
4e3815d14dceaf31d60ea325e650be4db17ce6c3 23-Jan-2018 TreeHugger Robot <treehugger-gerrit@google.com> Merge "MediaExtractor: Fix use-after-free"
2b22e7083877778100b7e25b375842b23d14e701 22-Jan-2018 Andreas Gampe <agampe@google.com> MediaExtractor: Fix use-after-free

ZipString only references the given string and does not copy it.
It is thus important that the caller keeps the source, in this
case the formatted String8, alive.

Bug: 72329184
Test: m
Test: m && m SANITIZE_TARGET=address ; no aborts
Change-Id: I20f9acd544eeeb26cc115b753ac64b3bce6235b6
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
767456dfce275fbe2d0bea227c271fcbd4c5be19 20-Jan-2018 Dongwon Kang <dwkang@google.com> Load system extractor plugins from /system|vendor/lib[64]/extractors.

Plugins will be placed under /system|vendor/lib[64]/extractors and the
device will include a stub MediaComponents apk for later update.

Test: boots and play mp4 files
Bug: 67908547
Change-Id: I5ab30093d9dd50324ba327baa8270a204e10f8a5
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
c96ca4397cca129afd6b111f61f98dcfc2c1e3a7 19-Jan-2018 Marco Nelissen <marcone@google.com> Show list of loaded extractors in dumpsys

Bug: 67908547
Test: build, boot, dumpsys
Change-Id: Ib398dd000979b6b5d85db5e446946c0c8ded1793
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
73a2b25fd96f16fa4590fe3b8e6a4a4d334cd422 17-Jan-2018 Marco Nelissen <marcone@google.com> Rename MediaUpdate to MediaComponents

Bug: 67908547
Test: build, boot
Change-Id: Idb873b1e0eddeb8d56f818ba3967d05727060cd4
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
d73a1d99c5a2594a81bc8d560d208696b44656b3 17-Jan-2018 Dongwon Kang <dwkang@google.com> Fix for sdk builds.

Test: sdk build passes.
Bug: 67908547
Change-Id: I43a81e65bd2486b1e8d8431c0d4f451f170e58bc
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
3e6d245e154f8a382fd88529c2572d1117685511 30-Jun-2017 Dongwon Kang <dwkang@google.com> Load extractor libs directly from update APK

Test: play MP4 file. install and uninstall media update apk.
Bug: 67908547
Change-Id: I80c7772e25ba010be468834ce238f0fb86bc8d14
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
5e1897bbd25d8353d9567b746ce50d4b80d30564 30-Jun-2017 Dongwon Kang <dwkang@google.com> Keep the in-use extractor plugins when updating.

To track usage of extractors and close the so handle when they are
destroied, MediaExtractorPlugin/RefBase class is introduced.

Test: play MP4 file. install and uninstall media update apk.
Bug: 67908547
Change-Id: I24926f943bc7247627e62d38edafd13d9c305a51
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
661e6990bf0fc4174f43cea928d80fcdcc623ff6 05-Jan-2018 Dongwon Kang <dwkang@google.com> Remove libmediametrics dependency from libmediaextractor

- Removes media metrics related methods in the extractor plugin.
- Puts metrics related code in one place, RemoteMediaExtractor.

Test: build & pass android.media.cts.MediaExtractorTest
Bug: 67908556
Change-Id: I5e06e6b6b5c047a3c68d90479e55f1df9061da45
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
514674257ef6954a7479ac93a26a07061f54462d 08-Dec-2017 Dongwon Kang <dwkang@google.com> Remove IDataSource dependency from DataSource.

The logic for requesting file data source to media extractor service
moved to MediaExtractorFactory from GenericSource.

Bug: 65851881, Bug: 63934228
Test: MediaCts post commit tests
Change-Id: Iaba6d9bec560f0cd8a826f47d74bf491a34c83e9
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp
d91dc5a0602f54fc0d4d2187f37b5b8169bb62c3 10-Oct-2017 Dongwon Kang <dwkang@google.com> Remove libmedia and libstagefright dependency from extractors

- MediaSource, DataSource and MediaExtractor are moved to
libmediaextractor so that they can be used by extractor
implementations without depending on libmedia and libstagefright.
- XXXFactory classes has been added in order not to expose CreateXXX
methods in libmediaextractor.
- avc_utils is moved to libstagefright_foundation since most of
extractor implementations are relying on that.

Test: build + post submit media CTS tests
Bug: 65851881
Change-Id: I7d5cf18dd25abc10478ac3f6e7d1828ad023e3fb
/frameworks/av/media/libstagefright/MediaExtractorFactory.cpp