Re: [Helix-client-dev] CR: EHYN-7W7GQM: WMDRM: Music player can not exit when play a WMDRM protected song which's license is expired

Looks good.
--greg.

On Oct 29, 2009, at 9:43 PM, [email protected] wrote:

> "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:  [email protected]
>
> Reviewed by:  [email protected]
>
> TSW Id: EHYN-7W7GQM
>
> Date: 10/29/2009
> 	
> Project: SymbianMmf_wm
>
> Synopsis: WMDRM: Music player can not exit when play a WMDRM  
> protected song which's license is expired
>
> Overview:
> After AddDatasourceL( ) we have a timer implemented which waits for  
> a certain interval before calling IntializeRenderers( ). But the  
> problem here is the GetTickCount() uses gettimeofday() to get the  
> tick count on Symbian platform, which is a future date (after  
> changing the system date/time) and hence we never reach time out.
> We are using a nano kernel tick counter (i.e. User::NTickCount() and  
> HAL::Get(HAL::ENanoTickPeriod, tickPeriod)) on branches other than  
> 221Cays (indirectly  was handled as a part of a separate CR:  
> ECZU-7AJDAB). Therefore we didn't face this problem in the platforms  
> other than S60 3.2 and S60 3.2.3
>
> Fix:
> Using the nano kernel tick count here too under a flag  
> HELIX_CONFIG_USE_NTICKCOUNT. In addition also checking for a  
> condition if hardware returns a time resolution other than 1000 (may  
> be in future) then converting tickPeriod to equivalent ms.
>
> Files modified & changes:
> /client/common/system/hxsched.cpp
> /common/system/platform/symbian/gettickcount.c
>
> Image Size and Heap Use impact: No major impact
>
> Module Release testing (STIF) : Passed
>
> Test case(s) Added  : No
>
> Memory leak check performed : Passed, No additional leaks introduced.
>
> Platforms and Profiles Build Verified: helix-client-s60-50-mmf-mdf-dsp
>
> Platforms and Profiles Functionality verified: armv5
>
> Branch: 221Cays
>
> CVS Diff on 221Cays:  
> Attached 
> < 
> hxsched.cpp.diff 
> ><gettickcount.c.diff>_______________________________________________
> Helix-client-dev mailing list
> [email protected]
> http://lists.helixcommunity.org/mailman/listinfo/helix-client-dev