commit 8806b706693c0992724f6603353af18aeb4a0f80
author Hong Teng <hongteng@google.com> Wed Jul 06 18:29:28 2011 -0700
committer Hong Teng <hongteng@google.com> Wed Jul 20 16:52:31 2011 -0700
tree 236b8c2fe12c6c15f528d626f64b05e5d512d276
parent 460ba2a3a1fa9318463d3c00595b6732e3d776de

fix for issue 4142219
Don't hard code platform-specific limitations
engine part.

Change-Id: Ibc8a539e5bbac738f60ef935670a333b7017e20e

libvideoeditor/lvpp/VideoEditorPreviewController.cpp

diff --git a/libvideoeditor/lvpp/VideoEditorPreviewController.cpp b/libvideoeditor/lvpp/VideoEditorPreviewController.cpp
index c174585..f99ca79 100755
--- a/libvideoeditor/lvpp/VideoEditorPreviewController.cpp
+++ b/libvideoeditor/lvpp/VideoEditorPreviewController.cpp
@@ -1376,85 +1376,18 @@
     return err;
 }
 
-M4OSA_ERR VideoEditorPreviewController::setPreviewFrameRenderingMode(
+status_t VideoEditorPreviewController::setPreviewFrameRenderingMode(
     M4xVSS_MediaRendering mode, M4VIDEOEDITING_VideoFrameSize outputVideoSize) {
 
     LOGV("setMediaRenderingMode: outputVideoSize = %d", outputVideoSize);
     mRenderingMode = mode;
 
-    switch(outputVideoSize) {
-        case M4VIDEOEDITING_kSQCIF:
-            mOutputVideoWidth = 128;
-            mOutputVideoHeight = 96;
-            break;
+    status_t err = OK;
+    /* get the video width and height by resolution */
+    err = getVideoSizeByResolution(outputVideoSize,
+              &mOutputVideoWidth, &mOutputVideoHeight);
 
-        case M4VIDEOEDITING_kQQVGA:
-            mOutputVideoWidth = 160;
-            mOutputVideoHeight = 120;
-            break;
-
-        case M4VIDEOEDITING_kQCIF:
-            mOutputVideoWidth = 176;
-            mOutputVideoHeight = 144;
-            break;
-
-        case M4VIDEOEDITING_kQVGA:
-            mOutputVideoWidth = 320;
-            mOutputVideoHeight = 240;
-            break;
-
-        case M4VIDEOEDITING_kCIF:
-            mOutputVideoWidth = 352;
-            mOutputVideoHeight = 288;
-            break;
-
-        case M4VIDEOEDITING_kVGA:
-            mOutputVideoWidth = 640;
-            mOutputVideoHeight = 480;
-            break;
-
-        case M4VIDEOEDITING_kWVGA:
-            mOutputVideoWidth = 800;
-            mOutputVideoHeight = 480;
-            break;
-
-        case M4VIDEOEDITING_kNTSC:
-            mOutputVideoWidth = 720;
-            mOutputVideoHeight = 480;
-            break;
-
-        case M4VIDEOEDITING_k640_360:
-            mOutputVideoWidth = 640;
-            mOutputVideoHeight = 360;
-            break;
-
-        case M4VIDEOEDITING_k854_480:
-            mOutputVideoWidth = 854;
-            mOutputVideoHeight = 480;
-            break;
-
-        case M4VIDEOEDITING_kHD1280:
-            mOutputVideoWidth = 1280;
-            mOutputVideoHeight = 720;
-            break;
-
-        case M4VIDEOEDITING_kHD1080:
-            mOutputVideoWidth = 1080;
-            mOutputVideoHeight = 720;
-            break;
-
-        case M4VIDEOEDITING_kHD960:
-            mOutputVideoWidth = 960;
-            mOutputVideoHeight = 720;
-            break;
-
-        default:
-            mOutputVideoWidth = 0;
-            mOutputVideoHeight = 0;
-            break;
-    }
-
-    return OK;
+    return err;
 }
 
 M4OSA_ERR VideoEditorPreviewController::doImageRenderingMode(