mpxplugins/viewplugins/views/commonplaybackview/src/mpxcommonplaybackviewimp.cpp
branchRCL_3
changeset 66 1f1dad4af8f8
parent 53 3de6c4cf6b67
--- a/mpxplugins/viewplugins/views/commonplaybackview/src/mpxcommonplaybackviewimp.cpp	Wed Sep 15 12:09:46 2010 +0300
+++ b/mpxplugins/viewplugins/views/commonplaybackview/src/mpxcommonplaybackviewimp.cpp	Wed Oct 13 14:28:40 2010 +0300
@@ -113,7 +113,6 @@
 // CONSTANTS
 const TInt KMPXOneSecInMilliSecs( 1000 );
 const TUid KMPXEqualizerViewImplementationId = { 0x101FFC77 };
-const TUid KMPXMetadataEditorDialogImplementationId  = { 0x101FFC83 };
 const TInt KMPXPostponeForHandleDelayedError( 1000000 ); // 1S
 const TInt KMPXPlaybackViewWindowBackground = -1;
 const TInt KMPXDelayForTNRequest( 3000000 ); // 3S
@@ -1633,7 +1632,7 @@
     MPX_DEBUG1("CMPXPlaybackViewImp::HandleErrorL()");
     MPX_DEBUG1("    > Reset New Item Opened Flag");
     iNewItemOpened = EFalse;
-    iErrorOccured = EFalse;
+
     MPX_DEBUG1( "CMPXCommonPlaybackViewImp::HandleErrorL: Exiting");
     }
 
@@ -1982,7 +1981,6 @@
         }
     else if ( aError != KErrNone && iLastPBViewActivated )
         {
-        iErrorOccured = ETrue;
         TRAP_IGNORE( DoHandleErrorPlaybackMessageL( aError ) );
         }
     MPX_DEBUG2("<--CMPXCommonPlaybackViewImp::HandlePlaybackMessage(): aError = %d", aError);
@@ -2174,11 +2172,7 @@
             }
         case EMPXPbvCmdSongDetails:
             {
-            if( !iErrorOccured )
-                {
-                LaunchFileDetailsDialogL();
-                }
-            
+            LaunchFileDetailsDialogL();
             break;
             }
         case EMPXPbvCmdPlay:
@@ -2723,6 +2717,7 @@
                                 {
                                 MPX_DEBUG1( "CMPXCommonPlaybackViewImp::HandleForegroundEventL requesting media" );
                                 RequestMediaL();
+                                UpdateTitlePaneL();
                                 }
                             }
                         }
@@ -2766,12 +2761,6 @@
         {
         case R_MPX_PLAYBACK_VIEW_MENU:
             {
-            if( iErrorOccured ) 
-                {
-                MPX_DEBUG2( "iErrorOccured %d" , iErrorOccured);
-                aMenuPane->SetItemDimmed( EMPXPbvCmdSongDetails, ETrue );
-                }
-            
             if ( iIsffButtonPressed )
                 {
                 iIsffButtonPressed = EFalse;
@@ -3019,8 +3008,7 @@
             // since it'll mess up equilizer's screen
             if ( StatusPane()->CurrentLayoutResId() !=
                  R_AVKON_STATUS_PANE_LAYOUT_USUAL_FLAT &&
-                 activeView != KMPXEqualizerViewImplementationId && 
-				 activeView != KMPXMetadataEditorDialogImplementationId )
+                 activeView != KMPXEqualizerViewImplementationId )
                 {
                 TRAP_IGNORE(
                     StatusPane()->SwitchLayoutL( R_AVKON_STATUS_PANE_LAYOUT_USUAL_FLAT ));
@@ -3079,10 +3067,6 @@
     {
     MPX_FUNC_EX( "CMPXCommonPlaybackViewImp::HandleViewActivation VF" );
     iPBViewToBeActivated = ( KMPXPluginTypePlaybackUid == aCurrentViewType.iUid );
-	if(iPBViewToBeActivated)
-		{
-		HandleLayoutChange();
-		}
     MPX_DEBUG2( "CMPXCommonPlaybackViewImp::HandleViewActivation iPBViewToBeActivated %d", iPBViewToBeActivated );
     }
 // ---------------------------------------------------------------------------
@@ -3942,13 +3926,11 @@
                         if( pausePlayControl )
                         	{
 	                        TMPXPlaybackState state = iPlaybackUtility->StateL();
-	                        if ( state == EPbStatePlaying )
+	                        if ((state == EPbStateInitialising) || (state == EPbStatePlaying))
 	                           {
 	                           pausePlayControl->SetCurrentState(0, ETrue);
 	                           }
-	                        //get EPbStateInitialising when skipping to next, ignore it to remove play/pause key flicker 
-	                        //It is showing the state that was previously shown when EPbStateInitializing
-	                        else if ( state != EPbStateInitialising )
+	                        else
 	                           {
 	                           pausePlayControl->SetCurrentState(1, ETrue);
 	                           }