summaryrefslogtreecommitdiffstats
path: root/media
diff options
context:
space:
mode:
authorGarret Pick <pickgr@pv.com>2010-08-23 11:49:52 -0700
committerSteve Kondik <shade@chemlab.org>2010-10-17 04:52:07 -0400
commita34cb442866609c9fa49a8d2ca09dff09dd43aa9 (patch)
tree7f3f0d5a756e1bcfd71c2b0b401d212f05881f69 /media
parente3e40046a02ea80ba70897d26886abfc0e62f77a (diff)
downloadframeworks_base-a34cb442866609c9fa49a8d2ca09dff09dd43aa9.zip
frameworks_base-a34cb442866609c9fa49a8d2ca09dff09dd43aa9.tar.gz
frameworks_base-a34cb442866609c9fa49a8d2ca09dff09dd43aa9.tar.bz2
For performance reasons, setMode() needs to be called before setDataSource()
Change-Id: Ibcc3cd3abf8d4572b56fc66431f6d14bf8be5d48
Diffstat (limited to 'media')
-rw-r--r--media/libstagefright/StagefrightMediaScanner.cpp10
1 files changed, 4 insertions, 6 deletions
diff --git a/media/libstagefright/StagefrightMediaScanner.cpp b/media/libstagefright/StagefrightMediaScanner.cpp
index 25236cf..ac0a160 100644
--- a/media/libstagefright/StagefrightMediaScanner.cpp
+++ b/media/libstagefright/StagefrightMediaScanner.cpp
@@ -255,9 +255,8 @@ status_t StagefrightMediaScanner::processFile(
return HandleFLAC(path, &client);
}
- if (mRetriever->setDataSource(path) == OK
- && mRetriever->setMode(
- METADATA_MODE_METADATA_RETRIEVAL_ONLY) == OK) {
+ if (mRetriever->setMode(METADATA_MODE_METADATA_RETRIEVAL_ONLY) == OK &&
+ mRetriever->setDataSource(path) == OK) {
const char *value;
if ((value = mRetriever->extractMetadata(
METADATA_KEY_MIMETYPE)) != NULL) {
@@ -305,9 +304,8 @@ char *StagefrightMediaScanner::extractAlbumArt(int fd) {
}
lseek(fd, 0, SEEK_SET);
- if (mRetriever->setDataSource(fd, 0, size) == OK
- && mRetriever->setMode(
- METADATA_MODE_FRAME_CAPTURE_ONLY) == OK) {
+ if (mRetriever->setMode(METADATA_MODE_FRAME_CAPTURE_ONLY) == OK
+ && mRetriever->setDataSource(fd, 0, size) == OK) {
sp<IMemory> mem = mRetriever->extractAlbumArt();
if (mem != NULL) {