Frequent ASSERT_NOT_REACHED in Connection::processIncomingMessage when running regres...
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Aug 2010 17:39:50 +0000 (17:39 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Aug 2010 17:39:50 +0000 (17:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=42926
<rdar://problem/8237329>

Reviewed by Adam Roben.

Remove this assert, it's bogus. It would fire when a reply came in on the connection queue before
waitForReply was called on the client thread, but that case is already covered.

* Platform/CoreIPC/Connection.cpp:
(CoreIPC::Connection::processIncomingMessage):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@64564 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit2/ChangeLog
WebKit2/Platform/CoreIPC/Connection.cpp

index fca10eb..2f3da91 100644 (file)
@@ -1,3 +1,17 @@
+2010-08-03  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Adam Roben.
+
+        Frequent ASSERT_NOT_REACHED in Connection::processIncomingMessage when running regression tests
+        https://bugs.webkit.org/show_bug.cgi?id=42926
+        <rdar://problem/8237329>
+
+        Remove this assert, it's bogus. It would fire when a reply came in on the connection queue before 
+        waitForReply was called on the client thread, but that case is already covered.
+
+        * Platform/CoreIPC/Connection.cpp:
+        (CoreIPC::Connection::processIncomingMessage):
+
 2010-08-03  Adam Roben  <aroben@apple.com>
 
         Compile out two Mac-only localized strings on non-Mac platforms
 2010-08-03  Adam Roben  <aroben@apple.com>
 
         Compile out two Mac-only localized strings on non-Mac platforms
index e1d009e..adcfa16 100644 (file)
@@ -175,11 +175,6 @@ void Connection::processIncomingMessage(MessageID messageID, PassOwnPtr<Argument
         }
     }
 
         }
     }
 
-    if (messageID == MessageID(CoreIPCMessage::SyncMessageReply)) {
-        // FIXME: We got a reply for another sync message someone sent, handle this.
-        ASSERT_NOT_REACHED();
-    }
-
     MutexLocker locker(m_incomingMessagesLock);
     m_incomingMessages.append(IncomingMessage(messageID, arguments));
 
     MutexLocker locker(m_incomingMessagesLock);
     m_incomingMessages.append(IncomingMessage(messageID, arguments));