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

Modified Files:
      Tag: hxclient_2_1_0_cayennes
	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 at 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.12.2.59
retrieving revision 1.12.2.60
diff -u -d -r1.12.2.59 -r1.12.2.60
--- hxmmfctrlimpl.cpp	7 Feb 2008 18:29:05 -0000	1.12.2.59
+++ hxmmfctrlimpl.cpp	11 Feb 2008 18:27:54 -0000	1.12.2.60
@@ -1322,7 +1322,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