diff -r c54d95799c80 -r a1247965635c mpxplugins/viewplugins/views/commonplaybackview/src/mpxcommonplaybackviewlayout.cpp --- a/mpxplugins/viewplugins/views/commonplaybackview/src/mpxcommonplaybackviewlayout.cpp Wed Apr 14 15:54:18 2010 +0300 +++ b/mpxplugins/viewplugins/views/commonplaybackview/src/mpxcommonplaybackviewlayout.cpp Tue Apr 27 16:34:37 2010 +0300 @@ -81,8 +81,8 @@ TAknLayoutRect res; TMPXPbvLayoutVariants lafVariant(EPbvUndefinedVariant); - lafVariant = GetIndicatorVariant(aIndicator); - if (lafVariant == EPbvUndefinedVariant) + MPX_TRAPD( err, lafVariant = GetIndicatorVariantL(aIndicator) ); + if ( err != KErrNone || lafVariant == EPbvUndefinedVariant) { // unsupported ASSERT(0); @@ -178,8 +178,8 @@ TInt /*aOffsety*/ /* =0 */ ) { TMPXPbvLayoutVariants lafVariant(EPbvUndefinedVariant); - lafVariant = GetTextVariant(aText); - if (lafVariant == EPbvUndefinedVariant) + MPX_TRAPD( err, lafVariant = GetTextVariantL(aText) ); + if ( lafVariant == EPbvUndefinedVariant || err != KErrNone ) { ASSERT(0); } @@ -520,6 +520,19 @@ return res.Rect(); } +// --------------------------------------------------------------------------- +// Init common layout data +// --------------------------------------------------------------------------- +// +EXPORT_C void CMPXCommonPlaybackViewLayout::InitCommonLayoutL() + { + TInt flags( 0 ); + CRepository* repository = CRepository::NewL( KCRUidMPXMPFeatures ); + User::LeaveIfError( repository->Get( KMPXMPLocalVariation, flags )); + delete repository; + repository = NULL; + iShowRockerGraphics = static_cast( flags & KMPXRockerMappingSupport ); + } // --------------------------------------------------------------------------- // Determine the layout (touch, non-touch, landscape, portrait, rocker, etc...) @@ -529,12 +542,6 @@ { TMPXPbvLayoutVariants variant(EPbvUndefinedVariant); - TInt flags( 0 ); - CRepository* repository = CRepository::NewL( KCRUidMPXMPFeatures ); - User::LeaveIfError( repository->Get( KMPXMPLocalVariation, flags )); - delete repository; - repository = NULL; - TBool showRockerGraphics = static_cast( flags & KMPXRockerMappingSupport ); TBool isLandscape = Layout_Meta_Data::IsLandscapeOrientation(); @@ -548,19 +555,19 @@ { variant = EPbvLandscapeNhdTouchWithFM; } - else if (!showRockerGraphics && !isLandscape ) + else if (!iShowRockerGraphics && !isLandscape ) { variant = EPbvPortraitWithoutRocker; } - else if (!showRockerGraphics && isLandscape) + else if (!iShowRockerGraphics && isLandscape) { variant = EPbvLandscapeWithoutRocker; } - else if (showRockerGraphics && !isLandscape) + else if (iShowRockerGraphics && !isLandscape) { variant = EPbvPortraitWithRocker; } - else if (showRockerGraphics && isLandscape) + else if (iShowRockerGraphics && isLandscape) { variant = EPbvLandscapeWithRocker; } @@ -569,11 +576,12 @@ return(variant); } + // --------------------------------------------------------------------------- // Determine the layout variant for a text item // --------------------------------------------------------------------------- // -EXPORT_C TMPXPbvLayoutVariants CMPXCommonPlaybackViewLayout::GetTextVariant(TMPXPbvTextIndicator aText) +EXPORT_C TMPXPbvLayoutVariants CMPXCommonPlaybackViewLayout::GetTextVariantL(TMPXPbvTextIndicator aText) { TMPXPbvLayoutVariants layoutVariant = DetermineLayoutVariant(); TMPXPbvLayoutVariants textVariant; @@ -608,7 +616,7 @@ // Determine the layout variant type for an indicator item // --------------------------------------------------------------------------- // -EXPORT_C TMPXPbvLayoutVariants CMPXCommonPlaybackViewLayout::GetIndicatorVariant(TMPXPbvIndicator aIndicator) +EXPORT_C TMPXPbvLayoutVariants CMPXCommonPlaybackViewLayout::GetIndicatorVariantL(TMPXPbvIndicator aIndicator) { TMPXPbvLayoutVariants layoutVariant = DetermineLayoutVariant(); TMPXPbvLayoutVariants indicatorVariant;