Changeset 260836 in webkit


Ignore:
Timestamp:
Apr 28, 2020 11:59:13 AM (4 years ago)
Author:
achristensen@apple.com
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.

Location:
trunk/Source/WebKit
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit/ChangeLog

    r260832 r260836  
     12020-04-28  Alex Christensen  <achristensen@webkit.org>
     2
     3        Fix iOS API tests after r260790
     4        https://bugs.webkit.org/show_bug.cgi?id=211093
     5
     6        * UIProcess/AuxiliaryProcessProxy.cpp:
     7        (WebKit::AuxiliaryProcessProxy::sendMessage):
     8        Only call the completion handler once.
     9
    1102020-04-28  Jer Noble  <jer.noble@apple.com>
    211
  • trunk/Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp

    r260790 r260836  
    161161    case State::Running:
    162162        if (asyncReplyInfo)
    163             IPC::addAsyncReplyHandler(*connection(), asyncReplyInfo->second, WTFMove(asyncReplyInfo->first));
     163            IPC::addAsyncReplyHandler(*connection(), asyncReplyInfo->second, std::exchange(asyncReplyInfo->first, nullptr));
    164164        if (connection()->sendMessage(WTFMove(encoder), sendOptions))
    165165            return true;
     
    170170    }
    171171
    172     if (asyncReplyInfo) {
     172    if (asyncReplyInfo && asyncReplyInfo->first) {
    173173        RunLoop::current().dispatch([completionHandler = WTFMove(asyncReplyInfo->first)]() mutable {
    174174            completionHandler(nullptr);
Note: See TracChangeset for help on using the changeset viewer.