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

List:       helix-datatype-dev
Subject:    RE: [datatype-dev] CR needed: Symbian - Helix 3rd party drm
From:       <Yury.Ramanovich () nokia ! com>
Date:       2009-11-17 16:42:01
Message-ID: BE808F92D8B85A4FB9A6D26B46CB39EC286BBEC016 () NOK-EUMSG-03 ! mgdnok ! nokia ! com
[Download RAW message or body]

Thanks Eric, checked in to Cays210

BR
Yury
 

>-----Original Message-----
>From: ext Eric Hyche [mailto:ehyche@real.com] 
>Sent: Monday, November 16, 2009 5:37 PM
>To: Ramanovich Yury (Nokia-D/Dallas)
>Cc: datatype-dev@helixcommunity.org; 
>nokia-private-dev@helixcommunity.org
>Subject: Re: [datatype-dev] CR needed: Symbian - Helix 3rd 
>party drm playback support - SetClientThreadId crashes when 
>postprocessor is disabled
>
>Looks good.
>
>On Nov 16, 2009, at 4:04 PM, Yury.Ramanovich@nokia.com 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:  yury.ramanovich@nokia.com
>>  
>> Reviewed by:
>>  
>> Date: 16/11/2009
>>  
>> Project: SymbianMmf_wm
>>  
>> ErrorId: SUB 409-1052
>>  
>> Synopsis: Helix 3rd party drm playback support - SetClientThreadId 
>> crashes when postprocessor is disabled
>>  
>> Overview: there are use cases such as video thumbnail 
>generation, when 
>> postprocessor is not available;
>> CMDFDevVideoServerSession::SetClientThreadId() queries 
>MMmfVideoClientThreadInfo custom interface (CI) on 
>PostProcessor even if postprocessor is not available, which 
>was causing crash.
>>  
>> Solution: query MMmfVideoClientThreadInfo CI on video postprocessor 
>> only if the postprocessor is available
>>  
>>  
>> Files Added:
>> None.
>>  
>> Files Modified:
>>  
>> 
>/datatype/mdf/video/renderer/mdfdevice/server/CMDFDevVideoServerSessio
>> n.cpp 
>> 
>/datatype/mdf/video/renderer/mdfdevice/server/CMDFDevVideoServerSessio
>> n.h
>>  
>> Image Size and Heap Use impact: minor
>>  
>> Module Release testing (STIF) :  MRT subset ongoing
>>  
>> Test case(s) Added  :  No.
>>  
>> Memory leak check performed : Yes. No new leaks introduced
>>  
>> Platforms and Profiles Build Verified:
>> helix-client-s60-52-mmf-mdf-dsp
>>  
>> Platforms and Profiles Functionality verified: armv5, winscw
>>  
>> Branch: 210CayS, 420Brizo, HEAD
>>  
>> <diff.txt>_______________________________________________
>> Datatype-dev mailing list
>> Datatype-dev@helixcommunity.org
>> http://lists.helixcommunity.org/mailman/listinfo/datatype-dev
>
>Eric Hyche (ehyche@real.com)
>Principal Engineer
>RealNetworks, Inc.
>
>
>
>
_______________________________________________
Datatype-dev mailing list
Datatype-dev@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/datatype-dev

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