Changeset 225469 in webkit


Ignore:
Timestamp:
Dec 3, 2017 9:40:48 PM (6 years ago)
Author:
commit-queue@webkit.org
Message:

WebSocketChannel should ensure its client is live when calling it in error case
https://bugs.webkit.org/show_bug.cgi?id=180285

Patch by Youenn Fablet <youenn@apple.com> on 2017-12-03
Reviewed by Darin Adler.

No observable change of behavior.
This makes it consistent with other calls of didReceiveMessageError.

  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::fail):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r225468 r225469  
     12017-12-03  Youenn Fablet  <youenn@apple.com>
     2
     3        WebSocketChannel should ensure its client is live when calling it in error case
     4        https://bugs.webkit.org/show_bug.cgi?id=180285
     5
     6        Reviewed by Darin Adler.
     7
     8        No observable change of behavior.
     9        This makes it consistent with other calls of didReceiveMessageError.
     10
     11        * Modules/websockets/WebSocketChannel.cpp:
     12        (WebCore::WebSocketChannel::fail):
     13
    1142017-12-03  Alexey Proskuryakov  <ap@apple.com>
    215
  • trunk/Source/WebCore/Modules/websockets/WebSocketChannel.cpp

    r221562 r225469  
    233233    m_hasContinuousFrame = false;
    234234    m_continuousFrameData.clear();
    235     m_client->didReceiveMessageError();
     235    if (m_client)
     236        m_client->didReceiveMessageError();
    236237
    237238    if (m_handle && !m_closed)
Note: See TracChangeset for help on using the changeset viewer.