⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 276221 in webkit


Ignore:
Timestamp:
Apr 17, 2021, 9:47:47 PM (5 years ago)
Author:
commit-queue@webkit.org
Message:

Unhandled IPC messages should use correct format with the decoder.destinationID() ASSERT message
​https://bugs.webkit.org/show_bug.cgi?id=224335

Patch by Kimmo Kinnunen <​kkinnunen@apple.com> on 2021-04-17
Reviewed by Antti Koivisto.

Use PRIu64 instead of llu in the ASSERT messages for printing unhandled message destination.
uint64_t is printf'ed with PRIu64.

  • Scripts/webkit/messages.py:

(generate_message_handler):

  • Scripts/webkit/tests/TestWithIfMessageMessageReceiver.cpp:

(WebKit::TestWithIfMessage::didReceiveMessage):

  • Scripts/webkit/tests/TestWithImageDataMessageReceiver.cpp:

(WebKit::TestWithImageData::didReceiveMessage):
(WebKit::TestWithImageData::didReceiveSyncMessage):

  • Scripts/webkit/tests/TestWithLegacyReceiverMessageReceiver.cpp:

(WebKit::TestWithLegacyReceiver::didReceiveTestWithLegacyReceiverMessage):
(WebKit::TestWithLegacyReceiver::didReceiveSyncTestWithLegacyReceiverMessage):

  • Scripts/webkit/tests/TestWithSemaphoreMessageReceiver.cpp:

(WebKit::TestWithSemaphore::didReceiveMessage):
(WebKit::TestWithSemaphore::didReceiveSyncMessage):

  • Scripts/webkit/tests/TestWithStreamBufferMessageReceiver.cpp:

(WebKit::TestWithStreamBuffer::didReceiveMessage):

  • Scripts/webkit/tests/TestWithStreamMessageReceiver.cpp:

(WebKit::TestWithStream::didReceiveStreamMessage):

  • Scripts/webkit/tests/TestWithSuperclassMessageReceiver.cpp:

(WebKit::TestWithSuperclass::didReceiveSyncMessage):

  • Scripts/webkit/tests/TestWithoutAttributesMessageReceiver.cpp:

(WebKit::TestWithoutAttributes::didReceiveMessage):
(WebKit::TestWithoutAttributes::didReceiveSyncMessage):

Location:
trunk/Source/WebKit
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebKit/ChangeLog

    r276220 r276221  
     12021-04-17  Kimmo Kinnunen  <kkinnunen@apple.com>
     2
     3        Unhandled IPC messages should use correct format with the decoder.destinationID() ASSERT message
     4        https://bugs.webkit.org/show_bug.cgi?id=224335
     5
     6        Reviewed by Antti Koivisto.
     7
     8        Use PRIu64 instead of llu in the ASSERT messages for printing unhandled message destination.
     9        uint64_t is printf'ed with PRIu64.
     10
     11        * Scripts/webkit/messages.py:
     12        (generate_message_handler):
     13        * Scripts/webkit/tests/TestWithIfMessageMessageReceiver.cpp:
     14        (WebKit::TestWithIfMessage::didReceiveMessage):
     15        * Scripts/webkit/tests/TestWithImageDataMessageReceiver.cpp:
     16        (WebKit::TestWithImageData::didReceiveMessage):
     17        (WebKit::TestWithImageData::didReceiveSyncMessage):
     18        * Scripts/webkit/tests/TestWithLegacyReceiverMessageReceiver.cpp:
     19        (WebKit::TestWithLegacyReceiver::didReceiveTestWithLegacyReceiverMessage):
     20        (WebKit::TestWithLegacyReceiver::didReceiveSyncTestWithLegacyReceiverMessage):
     21        * Scripts/webkit/tests/TestWithSemaphoreMessageReceiver.cpp:
     22        (WebKit::TestWithSemaphore::didReceiveMessage):
     23        (WebKit::TestWithSemaphore::didReceiveSyncMessage):
     24        * Scripts/webkit/tests/TestWithStreamBufferMessageReceiver.cpp:
     25        (WebKit::TestWithStreamBuffer::didReceiveMessage):
     26        * Scripts/webkit/tests/TestWithStreamMessageReceiver.cpp:
     27        (WebKit::TestWithStream::didReceiveStreamMessage):
     28        * Scripts/webkit/tests/TestWithSuperclassMessageReceiver.cpp:
     29        (WebKit::TestWithSuperclass::didReceiveSyncMessage):
     30        * Scripts/webkit/tests/TestWithoutAttributesMessageReceiver.cpp:
     31        (WebKit::TestWithoutAttributes::didReceiveMessage):
     32        (WebKit::TestWithoutAttributes::didReceiveSyncMessage):
     33
    1342021-04-17  Wenson Hsieh  <wenson_hsieh@apple.com>
    235

  • trunk/Source/WebKit/Scripts/webkit/messages.py

    r276177 r276221  
    953953            result.append('    UNUSED_PARAM(decoder);\n')
    954954            result.append('    UNUSED_PARAM(connection);\n')
    955             result.append('    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled stream message %s to %llu", description(decoder.messageName()), decoder.destinationID());\n')
     955            result.append('    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled stream message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());\n')
    956956        result.append('}\n')
    957957    elif async_messages or receiver.has_attribute(WANTS_DISPATCH_MESSAGE_ATTRIBUTE) or receiver.has_attribute(WANTS_ASYNC_DISPATCH_MESSAGE_ATTRIBUTE):
    … …  
    972972            result.append('    UNUSED_PARAM(connection);\n')
    973973            result.append('    UNUSED_PARAM(decoder);\n')
    974             result.append('    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());\n')
     974            result.append('    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());\n')
    975975        result.append('}\n')
    976976
    … …  
    988988        result.append('    UNUSED_PARAM(decoder);\n')
    989989        result.append('    UNUSED_PARAM(replyEncoder);\n')
    990         result.append('    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %llu", description(decoder.messageName()), decoder.destinationID());\n')
     990        result.append('    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());\n')
    991991        result.append('    return false;\n')
    992992        result.append('}\n')

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithIfMessageMessageReceiver.cpp

    r275655 r276221  
    5151    UNUSED_PARAM(connection);
    5252    UNUSED_PARAM(decoder);
    53     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     53    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    5454}
    5555

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithImageDataMessageReceiver.cpp

    r275655 r276221  
    4343    UNUSED_PARAM(connection);
    4444    UNUSED_PARAM(decoder);
    45     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     45    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    4646}
    4747
    … …  
    5454    UNUSED_PARAM(decoder);
    5555    UNUSED_PARAM(replyEncoder);
    56     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     56    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    5757    return false;
    5858}

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithLegacyReceiverMessageReceiver.cpp

    r275655 r276221  
    132132    UNUSED_PARAM(connection);
    133133    UNUSED_PARAM(decoder);
    134     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     134    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    135135}
    136136
    … …  
    155155    UNUSED_PARAM(decoder);
    156156    UNUSED_PARAM(replyEncoder);
    157     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     157    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    158158    return false;
    159159}

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithSemaphoreMessageReceiver.cpp

    r275655 r276221  
    4040    UNUSED_PARAM(connection);
    4141    UNUSED_PARAM(decoder);
    42     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     42    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    4343}
    4444
    … …  
    5151    UNUSED_PARAM(decoder);
    5252    UNUSED_PARAM(replyEncoder);
    53     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     53    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    5454    return false;
    5555}

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithStreamBufferMessageReceiver.cpp

    r275655 r276221  
    4040    UNUSED_PARAM(connection);
    4141    UNUSED_PARAM(decoder);
    42     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     42    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    4343}
    4444

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithStreamMessageReceiver.cpp

    r275655 r276221  
    5757    UNUSED_PARAM(decoder);
    5858    UNUSED_PARAM(connection);
    59     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled stream message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     59    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled stream message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    6060}
    6161

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithSuperclassMessageReceiver.cpp

    r275655 r276221  
    201201    UNUSED_PARAM(decoder);
    202202    UNUSED_PARAM(replyEncoder);
    203     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     203    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    204204    return false;
    205205}

  • trunk/Source/WebKit/Scripts/webkit/tests/TestWithoutAttributesMessageReceiver.cpp

    r275655 r276221  
    132132    UNUSED_PARAM(connection);
    133133    UNUSED_PARAM(decoder);
    134     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     134    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    135135}
    136136
    … …  
    155155    UNUSED_PARAM(decoder);
    156156    UNUSED_PARAM(replyEncoder);
    157     ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %llu", description(decoder.messageName()), decoder.destinationID());
     157    ASSERT_NOT_REACHED_WITH_MESSAGE("Unhandled synchronous message %s to %" PRIu64, description(decoder.messageName()), decoder.destinationID());
    158158    return false;
    159159}
Note: See TracChangeset for help on using the changeset viewer.