REGRESSION(r282115): undefined reference to `IPC::messageArgumentDesc…

…riptions(IPC::MessageName)' in Debug build

https://bugs.webkit.org/show_bug.cgi?id=230064

Unreviewed build fix.

Debug build of non-Cocoa ports were broken since r282115 started
to use IPC::messageArgumentDescriptions for debug logging.
However, MessageArgumentDescriptions.cpp has a long-standing
compilation problem for non-Cocoa ports.

Temporarily disabled the debug logging for non-Cocoa ports by
using ENABLE(IPC_TESTING_API) macro. I'll revert this change after
I will fix the fundamental problem of messages.py.

* Platform/IPC/HandleMessage.h:
(IPC::logMessageImpl):
(IPC::logReply):

Canonical link: https://commits.webkit.org/241469@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@282177 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog

@@ -1,3 +1,23 @@
+2021-09-08  Fujii Hironori  <Hironori.Fujii@sony.com>
+
+        REGRESSION(r282115): undefined reference to `IPC::messageArgumentDescriptions(IPC::MessageName)' in Debug build
+        https://bugs.webkit.org/show_bug.cgi?id=230064
+
+        Unreviewed build fix.
+
+        Debug build of non-Cocoa ports were broken since r282115 started
+        to use IPC::messageArgumentDescriptions for debug logging.
+        However, MessageArgumentDescriptions.cpp has a long-standing
+        compilation problem for non-Cocoa ports.
+
+        Temporarily disabled the debug logging for non-Cocoa ports by
+        using ENABLE(IPC_TESTING_API) macro. I'll revert this change after
+        I will fix the fundamental problem of messages.py.
+
+        * Platform/IPC/HandleMessage.h:
+        (IPC::logMessageImpl):
+        (IPC::logReply):
+
 2021-09-08  Sihui Liu  <sihui_liu@apple.com>
 
         Remove unnecessary WTF:: in WebsiteDataStore files

Source/WebKit/Platform/IPC/HandleMessage.h

@@ -47,7 +47,7 @@ class Connection;
 
 constexpr unsigned loggingContainerSizeLimit = 200;
 
-#if !LOG_DISABLED
+#if !LOG_DISABLED && ENABLE(IPC_TESTING_API)
 enum class ForReply : bool { No, Yes };
 
 inline TextStream textStreamForLogging(const Connection& connection, MessageName messageName, ForReply forReply)
@@ -73,7 +73,7 @@ inline TextStream textStreamForLogging(const Connection& connection, MessageName
 template<typename ArgsTuple, size_t... ArgsIndex>
 void logMessageImpl(const Connection& connection, MessageName messageName, const ArgsTuple& args, std::index_sequence<ArgsIndex...>)
 {
-#if !LOG_DISABLED
+#if !LOG_DISABLED && ENABLE(IPC_TESTING_API)
     auto stream = textStreamForLogging(connection, messageName, ForReply::No);
 
     if (auto argumentDescriptions = messageArgumentDescriptions(messageName))
@@ -96,7 +96,7 @@ void logMessage(const Connection& connection, MessageName messageName, const Arg
 template<typename... T>
 void logReply(const Connection& connection, MessageName messageName, const T&... args)
 {
-#if !LOG_DISABLED
+#if !LOG_DISABLED && ENABLE(IPC_TESTING_API)
     if (!sizeof...(T))
         return;