diff options
| author | James Dong <jdong@google.com> | 2010-09-16 11:47:17 -0700 |
|---|---|---|
| committer | Android Git Automerger <android-git-automerger@android.com> | 2010-09-16 11:47:17 -0700 |
| commit | e30d039f0ef797c35dd9f75e590f0b819c1c6194 (patch) | |
| tree | 7acb3e9ee9574d8e5c1dce3b07b012bf1a402248 | |
| parent | 62d4ebc0423831dfd8165d359e5b809f00f60656 (diff) | |
| parent | a7f5e476d54e6f7da588909d370580c3e3457e9b (diff) | |
| download | frameworks_base-e30d039f0ef797c35dd9f75e590f0b819c1c6194.zip frameworks_base-e30d039f0ef797c35dd9f75e590f0b819c1c6194.tar.gz frameworks_base-e30d039f0ef797c35dd9f75e590f0b819c1c6194.tar.bz2 | |
am a7f5e476: Merge "Fixed a bug in the query to the supported profiles and levels" into gingerbread
Merge commit 'a7f5e476d54e6f7da588909d370580c3e3457e9b' into gingerbread-plus-aosp
* commit 'a7f5e476d54e6f7da588909d370580c3e3457e9b':
Fixed a bug in the query to the supported profiles and levels
| -rw-r--r-- | media/libstagefright/OMXCodec.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/media/libstagefright/OMXCodec.cpp b/media/libstagefright/OMXCodec.cpp index 3d25a4b..cd70a3d 100644 --- a/media/libstagefright/OMXCodec.cpp +++ b/media/libstagefright/OMXCodec.cpp @@ -1035,7 +1035,7 @@ status_t OMXCodec::getVideoProfileLevel( mNode, OMX_IndexParamVideoProfileLevelQuerySupported, ¶m, sizeof(param)); - if (err != OK) return err; + if (err != OK) break; int32_t supportedProfile = static_cast<int32_t>(param.eProfile); int32_t supportedLevel = static_cast<int32_t>(param.eLevel); @@ -1043,7 +1043,10 @@ status_t OMXCodec::getVideoProfileLevel( supportedProfile, supportedLevel); if (profile == supportedProfile && - level == supportedLevel) { + level <= supportedLevel) { + // We can further check whether the level is a valid + // value; but we will leave that to the omx encoder component + // via OMX_SetParameter call. profileLevel.mProfile = profile; profileLevel.mLevel = level; return OK; |
