windowing/windowserver/nga/SERVER/openwfc/CLIENT.CPP
branchRCL_3
changeset 20 25ffed67c7ef
parent 19 bbf46f59e123
--- a/windowing/windowserver/nga/SERVER/openwfc/CLIENT.CPP	Tue Aug 31 16:31:06 2010 +0300
+++ b/windowing/windowserver/nga/SERVER/openwfc/CLIENT.CPP	Wed Sep 01 12:39:21 2010 +0100
@@ -1286,9 +1286,6 @@
 		case EWsClOpCreateDrawableSource:
 			CreateDrawableSourceL(*pData.CreateDrawableSource);
 			break;
-		case EWsClOpIndicateAppOrientation:
-			IndicateAppOrientation(*pData.Orientation);
-			break;
 		default:
 			PPanic(EWservPanicOpcode);
 			break;
@@ -1837,23 +1834,28 @@
 
 void CWsClient::CompleteMessage(const RMessage2& aMessage,TInt aReason)
 	{
-	WS_ASSERT_DEBUG(!aMessage.IsNull(),EWsPanicPanicFlagError);
-	if (iInternalFlags&EPanicClientAsSoonAsPossible)
+	WS_ASSERT_DEBUG(!aMessage.IsNull(),EWsPanicCompleteNullMessage);
+	// This defensive check should not be necessary as aMessage should never
+	// be null, but in rare situations it is and WServ would die without this check.
+	if (!aMessage.IsNull())
 		{
-		aMessage.Panic(KWSERVSessionPanicCategory,iPanicReason);
-		iInternalFlags&=~EPanicClientAsSoonAsPossible;
-		}
-	else
-		{
-		if(!iResponseHandle)
-			aMessage.Complete(aReason);
+		if (iInternalFlags&EPanicClientAsSoonAsPossible)
+			{
+			aMessage.Panic(KWSERVSessionPanicCategory,iPanicReason);
+			iInternalFlags&=~EPanicClientAsSoonAsPossible;
+			}
 		else
 			{
-			aMessage.Complete(*iResponseHandle);
-			iResponseHandle=NULL;
+			if(!iResponseHandle)
+				aMessage.Complete(aReason);
+			else
+				{
+				aMessage.Complete(*iResponseHandle);
+				iResponseHandle=NULL;
+				}
 			}
 		}
-	}
+    }
 
 void CWsClient::ServiceError(const RMessage2& /*aMessage*/,TInt aError)
 	{
@@ -2141,17 +2143,6 @@
 	drawableSource->ConstructL(aDrawableSourceData);
 	CleanupStack::Pop();
 	}
-	
-void CWsClient::IndicateAppOrientation(TRenderOrientation aOrientation)
-	{
-	iIndicatedAppOrientation = aOrientation;
-	CWsTop::CheckRenderOrientation();
-	}
-
-TInt CWsClient::GetIndicatedAppOrientation()
-	{
-	return iIndicatedAppOrientation;
-	}
 
 //
 // class CWsCliObj