Changeset 26566 in vbox for trunk/src/libs/xpcom18a4/ipc/ipcd/client

Timestamp:

Feb 16, 2010 11:03:52 AM (15 years ago)

Author:

vboxsync

Message:

XPCOM: back out r57733 which breaks automatic SVC startup

File:

• trunk/src/libs/xpcom18a4/ipc/ipcd/client/src/ipcdclient.cpp (modified) (5 diffs)

trunk/src/libs/xpcom18a4/ipc/ipcd/client/src/ipcdclient.cpp

@@ -405 +405 @@
       break;
     }
-#ifdef VBOX
-    else
-    {
-      /* Special client liveness check if there is no message to process.
-       * This is necessary as there might be several threads waiting for
-       * a message from a single client, and only one gets the DOWN msg. */
-      PRBool alive = (aSelector)(aArg, td, NULL);
-      if (!alive)
-      {
-        *aMsg = NULL;
-        break;
-      }
-    }
-#endif /* VBOX */
 
     PRIntervalTime t = PR_IntervalNow();
@@ -601 +587 @@
 WaitIPCMResponseSelector(void *arg, ipcTargetData *td, const ipcMessage *msg)
 {
-#ifdef VBOX
-  if (!msg)
-    return PR_TRUE;
-#endif /* VBOX */
   PRUint32 requestIndex = *(PRUint32 *) arg;
   return IPCM_GetRequestIndex(msg) == requestIndex;
@@ -932 +914 @@
 {
   WaitMessageSelectorData *data = (WaitMessageSelectorData *) arg;
-#ifdef VBOX
-  if (!msg)
-  {
-    /* Special NULL message which asks to check whether the client is
-     * still alive. Called when there is nothing suitable in the queue. */
-    ipcIMessageObserver *obs = data->observer;
-    if (!obs)
-      obs = td->observer;
-    NS_ASSERTION(obs, "must at least have a default observer");
-
-    nsresult rv = obs->OnMessageAvailable(IPC_SENDER_ANY, nsID(), 0, 0);
-    if (rv != IPC_WAIT_NEXT_MESSAGE)
-    {
-      data->senderDead = PR_TRUE;
-      return PR_FALSE;
-    }
-    return PR_TRUE;
-  }
-#endif /* VBOX */
 
   // process the specially forwarded client state message to see if the
@@ -980 +943 @@
           {
             // otherwise inform the observer about the client death using a special
-            // null message with an empty target id, and fail IPC_WaitMessage call
+            // null message with an emply target id, and fail IPC_WaitMessage call
             // with NS_ERROR_xxx only if the observer accepts this message.
 
@@ -996 +959 @@
           }
         }
-#ifdef VBOX
-        else if ((data->senderID == IPC_SENDER_ANY ||
-                  status->ClientID() == data->senderID) &&
-                 status->ClientState() == IPCM_CLIENT_STATE_UP)
-        {
-          LOG(("sender (%d) we're waiting a message from (%d) has come up\n",
-               status->ClientID(), data->senderID));
-          if (data->senderID == IPC_SENDER_ANY)
-          {
-            // inform the observer about the client appearance using a special
-            // null message with an empty target id, but a length of 1.
-
-            ipcIMessageObserver *obs = data->observer;
-            if (!obs)
-              obs = td->observer;
-            NS_ASSERTION(obs, "must at least have a default observer");
-
-            nsresult rv = obs->OnMessageAvailable(status->ClientID(), nsID(), 0, 1);
-            if (rv != IPC_WAIT_NEXT_MESSAGE)
-            {
-              /* It might sound a bit paradoxical to declare the sender as
-               * dead, but the fact that a client up message is received
-               * while waiting for a message from this client clearly
-               * indicates that this is no longer the client we were waiting
-               * for, but a new one got the same client ID (due to wraparound).
-               * Shouldn't happen in real life, but better be safe. */
-              data->senderDead = PR_TRUE;
-              return PR_TRUE; // consume the message
-            }
-          }
-        }
-#endif /* VBOX */
         break;
       }