Title: [260836] trunk/Source/WebKit
- Revision
- 260836
- Author
- [email protected]
- Date
- 2020-04-28 11:59:13 -0700 (Tue, 28 Apr 2020)
Log Message
Fix iOS API tests after r260790
https://bugs.webkit.org/show_bug.cgi?id=211093
* UIProcess/AuxiliaryProcessProxy.cpp:
(WebKit::AuxiliaryProcessProxy::sendMessage):
Only call the completion handler once.
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (260835 => 260836)
--- trunk/Source/WebKit/ChangeLog 2020-04-28 18:57:06 UTC (rev 260835)
+++ trunk/Source/WebKit/ChangeLog 2020-04-28 18:59:13 UTC (rev 260836)
@@ -1,3 +1,12 @@
+2020-04-28 Alex Christensen <[email protected]>
+
+ Fix iOS API tests after r260790
+ https://bugs.webkit.org/show_bug.cgi?id=211093
+
+ * UIProcess/AuxiliaryProcessProxy.cpp:
+ (WebKit::AuxiliaryProcessProxy::sendMessage):
+ Only call the completion handler once.
+
2020-04-28 Jer Noble <[email protected]>
[Mac] Adopt kMTSupportNotification_ShouldPlayHDRVideoChanged notification
Modified: trunk/Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp (260835 => 260836)
--- trunk/Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp 2020-04-28 18:57:06 UTC (rev 260835)
+++ trunk/Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp 2020-04-28 18:59:13 UTC (rev 260836)
@@ -160,7 +160,7 @@
case State::Running:
if (asyncReplyInfo)
- IPC::addAsyncReplyHandler(*connection(), asyncReplyInfo->second, WTFMove(asyncReplyInfo->first));
+ IPC::addAsyncReplyHandler(*connection(), asyncReplyInfo->second, std::exchange(asyncReplyInfo->first, nullptr));
if (connection()->sendMessage(WTFMove(encoder), sendOptions))
return true;
break;
@@ -169,7 +169,7 @@
break;
}
- if (asyncReplyInfo) {
+ if (asyncReplyInfo && asyncReplyInfo->first) {
RunLoop::current().dispatch([completionHandler = WTFMove(asyncReplyInfo->first)]() mutable {
completionHandler(nullptr);
});
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
