[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-clientapps-cvs
Subject:    [Clientapps-cvs]
From:       junhliu () helixcommunity ! org
Date:       2008-02-11 18:26:00
Message-ID: 200802111826.m1BIQJ3s014816 () mailer ! progressive-comp ! com
[Download RAW message or body]

Update of /cvsroot/clientapps/symbianMmf/videocontroller
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv30497

Modified Files:
	hxmmfctrlimpl.cpp 
Log Message:
"Nokia submits this code under the terms of a commercial contribution agreement 
with RealNetworks, and I am authorized to contribute this code under said agreement." \


Modified by: junhong.liu@nokia.com 
  
Reviewed by:  

Date: 07-Feb-2008 
  
Project: SymbianMmf_Rel 

TSW: RBEL-7ART7A 

Synopsis: Helix won't set scaling option to DevVideo when the frame size info is not \
avalible.  

Overview: When the frame size info is not avaliable for helix engine, either because \
it's missing from SDP or from the file header, helix won't set scaling option to \
DevVideo. The post-processor will do the auto scaling based on the frame size info \
from the decoder in this case.


  
Files Modified: 
  
/clientapps/symbianMmf/videocontroller/hxmmfctrlimpl.cpp
/datatype/mdf/video/renderer/mdfvideoadapter.cpp,v


Image Size and Heap Use impact: None 
  
Module Release testing (STIF) : Pass 
  
Test case(s) Added  :  No 

Memory leak check performed : No new leaks introduced.  

Platforms and Profiles Build Verified: 
Profile -> helix-client-s60-32-mmf-mdf-arm 
BIF branch  -> helix_restricted 
SYSTEM_ID -> symbian-91-armv5 
Target -> symbianMmf_rel 
  
Platforms and Profiles Functionality verified: armv5, winscw 
  
Branch: 210CayS,  221CayS& head 


Index: hxmmfctrlimpl.cpp
===================================================================
RCS file: /cvsroot/clientapps/symbianMmf/videocontroller/hxmmfctrlimpl.cpp,v
retrieving revision 1.73
retrieving revision 1.74
diff -u -d -r1.73 -r1.74
--- hxmmfctrlimpl.cpp	8 Feb 2008 18:38:09 -0000	1.73
+++ hxmmfctrlimpl.cpp	11 Feb 2008 18:25:57 -0000	1.74
@@ -1329,7 +1329,11 @@
 
 #if defined(HELIX_FEATURE_MMF_SITE_CONTROL)
         /* Initialization. */
-        m_pSurfaceControl->SetFrameRect(CHXxRect(0, 0, m_lFrameWidth, \
m_lFrameHeight)); +
+        if ((m_lFrameWidth != 0) || (m_lFrameHeight != 0))
+        {
+            m_pSurfaceControl->SetFrameRect(CHXxRect(0, 0, m_lFrameWidth, \
m_lFrameHeight)); +        }
 
         //The Site nor the MDF will have knowledge of any roatation value
         //so we send them again


_______________________________________________
Clientapps-cvs mailing list
Clientapps-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/clientapps-cvs


[prev in list] [next in list] [prev in thread] [next in thread]