Title: [193620] trunk
Revision
193620
Author
carlo...@webkit.org
Date
2015-12-07 03:56:25 -0800 (Mon, 07 Dec 2015)

Log Message

[GTK] Add WebKitWebPage::console-message-sent signal to Web Extensions API
https://bugs.webkit.org/show_bug.cgi?id=79918

Reviewed by Gustavo Noronha Silva.

Source/WebKit2:

Add WebKitConsoleMessage boxed type that is passed to the
WebKitWebPage::console-message-sent signal with all the details
about the message.

* PlatformGTK.cmake: Add new files to compilation.
* UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt: Add new symbols.
* UIProcess/API/gtk/docs/webkit2gtk-4.0.types: Add webkit_console_message_get_type.
* UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Add new section to the docs.
* WebProcess/InjectedBundle/API/APIInjectedBundlePageUIClient.h:
(API::InjectedBundle::PageUIClient::willAddMessageToConsole): Add
paramaters for all console message details.
* WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.cpp: Added.
(webkit_console_message_copy):
(webkit_console_message_free):
(webkit_console_message_get_source):
(webkit_console_message_get_level):
(webkit_console_message_get_text):
(webkit_console_message_get_line):
(webkit_console_message_get_source_id):
* WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.h: Added.
* WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessagePrivate.h: Added.
(_WebKitConsoleMessage::_WebKitConsoleMessage):
* WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp:
(webkitWebPageDidSendConsoleMessage): Emit the
WebKitWebPage::console-message-sent signal.
(didReceiveResponseForResource): Generate a console message in
case of HTTP failure >= 400 for consistency with the inspector.
(didFailLoadForResource): Generate a console message in case of
resource load failure for consistency with the inspector.
(webkit_web_page_class_init): Add WebKitWebPage::console-message-sent signal.
(webkitWebPageCreate): Set custom UI client.
* WebProcess/InjectedBundle/API/gtk/webkit-web-extension.h:
* WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp:
(WebKit::InjectedBundlePageUIClient::willAddMessageToConsole):
* WebProcess/InjectedBundle/InjectedBundlePageUIClient.h:
* WebProcess/WebCoreSupport/WebChromeClient.cpp:
(WebKit::WebChromeClient::addMessageToConsole): Pass all the
console message details to the ui client.

Tools:

Add unit tests to check the different console messages.

* TestWebKitAPI/Tests/WebKit2Gtk/CMakeLists.txt:
* TestWebKitAPI/Tests/WebKit2Gtk/TestConsoleMessage.cpp: Added.
(ConsoleMessageTest::ConsoleMessage::operator==):
(ConsoleMessageTest::consoleMessageReceivedCallback):
(ConsoleMessageTest::ConsoleMessageTest):
(ConsoleMessageTest::~ConsoleMessageTest):
(ConsoleMessageTest::waitUntilConsoleMessageReceived):
(testWebKitConsoleMessageConsoleAPI):
(testWebKitConsoleMessageJavaScriptException):
(testWebKitConsoleMessageNetworkError):
(testWebKitConsoleMessageSecurityError):
(beforeAll):
(afterAll):
* TestWebKitAPI/Tests/WebKit2Gtk/WebExtensionTest.cpp:
(consoleMessageSentCallback):
(pageCreatedCallback):

Modified Paths

Added Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (193619 => 193620)


--- trunk/Source/WebKit2/ChangeLog	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/ChangeLog	2015-12-07 11:56:25 UTC (rev 193620)
@@ -1,3 +1,49 @@
+2015-12-07  Carlos Garcia Campos  <cgar...@igalia.com>
+
+        [GTK] Add WebKitWebPage::console-message-sent signal to Web Extensions API
+        https://bugs.webkit.org/show_bug.cgi?id=79918
+
+        Reviewed by Gustavo Noronha Silva.
+
+        Add WebKitConsoleMessage boxed type that is passed to the
+        WebKitWebPage::console-message-sent signal with all the details
+        about the message.
+
+        * PlatformGTK.cmake: Add new files to compilation.
+        * UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt: Add new symbols.
+        * UIProcess/API/gtk/docs/webkit2gtk-4.0.types: Add webkit_console_message_get_type.
+        * UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Add new section to the docs.
+        * WebProcess/InjectedBundle/API/APIInjectedBundlePageUIClient.h:
+        (API::InjectedBundle::PageUIClient::willAddMessageToConsole): Add
+        paramaters for all console message details.
+        * WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.cpp: Added.
+        (webkit_console_message_copy):
+        (webkit_console_message_free):
+        (webkit_console_message_get_source):
+        (webkit_console_message_get_level):
+        (webkit_console_message_get_text):
+        (webkit_console_message_get_line):
+        (webkit_console_message_get_source_id):
+        * WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.h: Added.
+        * WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessagePrivate.h: Added.
+        (_WebKitConsoleMessage::_WebKitConsoleMessage):
+        * WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp:
+        (webkitWebPageDidSendConsoleMessage): Emit the
+        WebKitWebPage::console-message-sent signal.
+        (didReceiveResponseForResource): Generate a console message in
+        case of HTTP failure >= 400 for consistency with the inspector.
+        (didFailLoadForResource): Generate a console message in case of
+        resource load failure for consistency with the inspector.
+        (webkit_web_page_class_init): Add WebKitWebPage::console-message-sent signal.
+        (webkitWebPageCreate): Set custom UI client.
+        * WebProcess/InjectedBundle/API/gtk/webkit-web-extension.h:
+        * WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp:
+        (WebKit::InjectedBundlePageUIClient::willAddMessageToConsole):
+        * WebProcess/InjectedBundle/InjectedBundlePageUIClient.h:
+        * WebProcess/WebCoreSupport/WebChromeClient.cpp:
+        (WebKit::WebChromeClient::addMessageToConsole): Pass all the
+        console message details to the ui client.
+
 2015-12-07  Ryuan Choi  <ryuan.c...@navercorp.com>
 
         [EFL] MiniBrowser doesn't exit when called ewk_view_try_close()

Modified: trunk/Source/WebKit2/PlatformGTK.cmake (193619 => 193620)


--- trunk/Source/WebKit2/PlatformGTK.cmake	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/PlatformGTK.cmake	2015-12-07 11:56:25 UTC (rev 193620)
@@ -323,6 +323,7 @@
     WebProcess/Cookies/soup/WebCookieManagerSoup.cpp
     WebProcess/Cookies/soup/WebKitSoupCookieJarSqlite.cpp
 
+    WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.cpp
     WebProcess/InjectedBundle/API/gtk/WebKitFrame.cpp
     WebProcess/InjectedBundle/API/gtk/WebKitScriptWorld.cpp
     WebProcess/InjectedBundle/API/gtk/WebKitWebEditor.cpp
@@ -433,6 +434,7 @@
 )
 
 set(WebKit2WebExtension_INSTALLED_HEADERS
+    ${WEBKIT2_DIR}/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.h
     ${WEBKIT2_DIR}/WebProcess/InjectedBundle/API/gtk/WebKitFrame.h
     ${WEBKIT2_DIR}/WebProcess/InjectedBundle/API/gtk/WebKitScriptWorld.h
     ${WEBKIT2_DIR}/WebProcess/InjectedBundle/API/gtk/WebKitWebEditor.h

Modified: trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt (193619 => 193620)


--- trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt	2015-12-07 11:56:25 UTC (rev 193620)
@@ -1344,6 +1344,26 @@
 </SECTION>
 
 <SECTION>
+<FILE>WebKitConsoleMessage</FILE>
+WebKitConsoleMessage
+WebKitConsoleMessageSource
+WebKitConsoleMessageLevel
+webkit_console_message_copy
+webkit_console_message_free
+webkit_console_message_get_source
+webkit_console_message_get_level
+webkit_console_message_get_text
+webkit_console_message_get_line
+webkit_console_message_get_source_id
+
+<SUBSECTION Standard>
+WEBKIT_TYPE_CONSOLE_MESSAGE
+
+<SUBSECTION Private>
+webkit_console_message_get_type
+</SECTION>
+
+<SECTION>
 <FILE>WebKitColorChooserRequest</FILE>
 WebKitColorChooserRequest
 webkit_color_chooser_request_get_rgba

Modified: trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-4.0.types (193619 => 193620)


--- trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-4.0.types	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-4.0.types	2015-12-07 11:56:25 UTC (rev 193620)
@@ -32,3 +32,4 @@
 webkit_website_data_manager_get_type
 webkit_editor_state_get_type
 webkit_install_missing_media_plugins_permission_request_get_type
+webkit_console_message_get_type

Modified: trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-docs.sgml (193619 => 193620)


--- trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-docs.sgml	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-docs.sgml	2015-12-07 11:56:25 UTC (rev 193620)
@@ -62,6 +62,7 @@
     <xi:include href=""
     <xi:include href=""
     <xi:include href=""
+    <xi:include href=""
   </chapter>
 
   <index id="index-all">
@@ -98,5 +99,10 @@
     <xi:include href="" /></xi:include>
   </index>
 
+  <index id="api-index-2-12" role="2.12">
+    <title>Index of new symbols in 2.12</title>
+    <xi:include href="" /></xi:include>
+  </index>
+
   <xi:include href="" /></xi:include>
 </book>

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/APIInjectedBundlePageUIClient.h (193619 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/APIInjectedBundlePageUIClient.h	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/APIInjectedBundlePageUIClient.h	2015-12-07 11:56:25 UTC (rev 193620)
@@ -27,6 +27,7 @@
 #define APIInjectedBundlePageUIClient_h
 
 #include "WebEvent.h"
+#include <runtime/ConsoleTypes.h>
 
 namespace WebCore {
 class HitTestResult;
@@ -49,7 +50,7 @@
 public:
     virtual ~PageUIClient() { }
 
-    virtual void willAddMessageToConsole(WebKit::WebPage*, const WTF::String& message, int32_t lineNumber) { UNUSED_PARAM(message); UNUSED_PARAM(lineNumber); }
+    virtual void willAddMessageToConsole(WebKit::WebPage*, JSC::MessageSource, JSC::MessageLevel, const WTF::String& message, unsigned lineNumber, unsigned columnNumber, const WTF::String& sourceID) { UNUSED_PARAM(message); UNUSED_PARAM(lineNumber); UNUSED_PARAM(columnNumber); UNUSED_PARAM(sourceID); }
     virtual void willSetStatusbarText(WebKit::WebPage*, const WTF::String&) { }
     virtual void willRunJavaScriptAlert(WebKit::WebPage*, const WTF::String&, WebKit::WebFrame*) { }
     virtual void willRunJavaScriptConfirm(WebKit::WebPage*, const WTF::String&, WebKit::WebFrame*) { }

Added: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.cpp (0 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.cpp	                        (rev 0)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.cpp	2015-12-07 11:56:25 UTC (rev 193620)
@@ -0,0 +1,166 @@
+/*
+ * Copyright (C) 2015 Igalia S.L.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public License
+ * along with this library; see the file COPYING.LIB.  If not, write to
+ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
+ */
+
+#include "config.h"
+#include "WebKitConsoleMessage.h"
+
+#include "WebKitConsoleMessagePrivate.h"
+
+G_DEFINE_BOXED_TYPE(WebKitConsoleMessage, webkit_console_message, webkit_console_message_copy, webkit_console_message_free)
+
+/**
+ * webkit_console_message_copy:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Make a copy of @console_message.
+ *
+ * Returns: (transfer full): A copy of passed in #WebKitConsoleMessage
+ *
+ * Since: 2.12
+ */
+WebKitConsoleMessage* webkit_console_message_copy(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_val_if_fail(consoleMessage, nullptr);
+    WebKitConsoleMessage* copy = static_cast<WebKitConsoleMessage*>(fastZeroedMalloc(sizeof(WebKitConsoleMessage)));
+    new (copy) WebKitConsoleMessage(consoleMessage);
+    return copy;
+}
+
+/**
+ * webkit_console_message_free:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Free the #WebKitConsoleMessage
+ *
+ * Since: 2.12
+ */
+void webkit_console_message_free(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_if_fail(consoleMessage);
+    consoleMessage->~WebKitConsoleMessage();
+    fastFree(consoleMessage);
+}
+
+/**
+ * webkit_console_message_get_source:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Gets the source of a #WebKitConsoleMessage
+ *
+ * Returns: a #WebKitConsoleMessageSource indicating the source of @console_message
+ *
+ * Since: 2.12
+ */
+WebKitConsoleMessageSource webkit_console_message_get_source(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_val_if_fail(consoleMessage, WEBKIT_CONSOLE_MESSAGE_SOURCE_OTHER);
+    switch (consoleMessage->source) {
+    case JSC::MessageSource::JS:
+        return WEBKIT_CONSOLE_MESSAGE_SOURCE_JAVASCRIPT;
+    case JSC::MessageSource::Network:
+        return WEBKIT_CONSOLE_MESSAGE_SOURCE_NETWORK;
+    case JSC::MessageSource::ConsoleAPI:
+        return WEBKIT_CONSOLE_MESSAGE_SOURCE_CONSOLE_API;
+    case JSC::MessageSource::Security:
+        return WEBKIT_CONSOLE_MESSAGE_SOURCE_SECURITY;
+    case JSC::MessageSource::Other:
+    default:
+        break;
+    }
+
+    return WEBKIT_CONSOLE_MESSAGE_SOURCE_OTHER;
+}
+
+/**
+ * webkit_console_message_get_level:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Gets the log level of a #WebKitConsoleMessage
+ *
+ * Returns: a #WebKitConsoleMessageLevel indicating the log level of @console_message
+ *
+ * Since: 2.12
+ */
+WebKitConsoleMessageLevel webkit_console_message_get_level(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_val_if_fail(consoleMessage, WEBKIT_CONSOLE_MESSAGE_LEVEL_LOG);
+    switch (consoleMessage->level) {
+    case JSC::MessageLevel::Log:
+        return WEBKIT_CONSOLE_MESSAGE_LEVEL_LOG;
+    case JSC::MessageLevel::Warning:
+        return WEBKIT_CONSOLE_MESSAGE_LEVEL_WARNING;
+    case JSC::MessageLevel::Error:
+        return WEBKIT_CONSOLE_MESSAGE_LEVEL_ERROR;
+    case JSC::MessageLevel::Debug:
+        return WEBKIT_CONSOLE_MESSAGE_LEVEL_DEBUG;
+    case JSC::MessageLevel::Info:
+        return WEBKIT_CONSOLE_MESSAGE_LEVEL_INFO;
+    }
+
+    ASSERT_NOT_REACHED();
+    return WEBKIT_CONSOLE_MESSAGE_LEVEL_LOG;
+}
+
+/**
+ * webkit_console_message_get_text:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Gets the text message of a #WebKitConsoleMessage
+ *
+ * Returns: the text message of @console_message
+ *
+ * Since: 2.12
+ */
+const gchar* webkit_console_message_get_text(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_val_if_fail(consoleMessage, nullptr);
+    return consoleMessage->message.data();
+}
+
+/**
+ * webkit_console_message_get_line:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Gets the line number of a #WebKitConsoleMessage
+ *
+ * Returns: the line number of @console_message
+ *
+ * Since: 2.12
+ */
+guint webkit_console_message_get_line(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_val_if_fail(consoleMessage, 0);
+    return consoleMessage->lineNumber;
+}
+
+/**
+ * webkit_console_message_get_source_id:
+ * @console_message: a #WebKitConsoleMessage
+ *
+ * Gets the source identifier of a #WebKitConsoleMessage
+ *
+ * Returns: the source identifier of @console_message
+ *
+ * Since: 2.12
+ */
+const gchar* webkit_console_message_get_source_id(WebKitConsoleMessage* consoleMessage)
+{
+    g_return_val_if_fail(consoleMessage, nullptr);
+    return consoleMessage->sourceID.data();
+}

Added: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.h (0 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.h	                        (rev 0)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessage.h	2015-12-07 11:56:25 UTC (rev 193620)
@@ -0,0 +1,102 @@
+/*
+ * Copyright (C) 2015 Igalia S.L.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public License
+ * along with this library; see the file COPYING.LIB.  If not, write to
+ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
+ */
+
+#if !defined(__WEBKIT_WEB_EXTENSION_H_INSIDE__) && !defined(WEBKIT2_COMPILATION)
+#error "Only <webkit2/webkit-web-extension.h> can be included directly."
+#endif
+
+#ifndef WebKitConsoleMessage_h
+#define WebKitConsoleMessage_h
+
+#include <glib-object.h>
+#include <webkit2/WebKitDefines.h>
+
+G_BEGIN_DECLS
+
+#define WEBKIT_TYPE_CONSOLE_MESSAGE (webkit_console_message_get_type())
+
+/**
+ * WebKitConsoleMessageSource:
+ * @WEBKIT_CONSOLE_MESSAGE_SOURCE_JAVASCRIPT: Message produced by _javascript_.
+ * @WEBKIT_CONSOLE_MESSAGE_SOURCE_NETWORK: Network messages.
+ * @WEBKIT_CONSOLE_MESSAGE_SOURCE_CONSOLE_API: Messages produced by console API.
+ * @WEBKIT_CONSOLE_MESSAGE_SOURCE_SECURITY: Security messages.
+ * @WEBKIT_CONSOLE_MESSAGE_SOURCE_OTHER: Other messages.
+ *
+ * Enum values used to denote the various sources of console messages.
+ *
+ * Since: 2.12
+ */
+typedef enum {
+    WEBKIT_CONSOLE_MESSAGE_SOURCE_JAVASCRIPT,
+    WEBKIT_CONSOLE_MESSAGE_SOURCE_NETWORK,
+    WEBKIT_CONSOLE_MESSAGE_SOURCE_CONSOLE_API,
+    WEBKIT_CONSOLE_MESSAGE_SOURCE_SECURITY,
+    WEBKIT_CONSOLE_MESSAGE_SOURCE_OTHER
+} WebKitConsoleMessageSource;
+
+/**
+ * WebKitConsoleMessageLevel:
+ * @WEBKIT_CONSOLE_MESSAGE_LEVEL_INFO: Information message.
+ * @WEBKIT_CONSOLE_MESSAGE_LEVEL_LOG: Log message.
+ * @WEBKIT_CONSOLE_MESSAGE_LEVEL_WARNING: Warning message.
+ * @WEBKIT_CONSOLE_MESSAGE_LEVEL_ERROR: Error message.
+ * @WEBKIT_CONSOLE_MESSAGE_LEVEL_DEBUG: Debug message.
+ *
+ * Enum values used to denote the various levels of console messages.
+ *
+ * Since: 2.12
+ */
+typedef enum {
+    WEBKIT_CONSOLE_MESSAGE_LEVEL_INFO,
+    WEBKIT_CONSOLE_MESSAGE_LEVEL_LOG,
+    WEBKIT_CONSOLE_MESSAGE_LEVEL_WARNING,
+    WEBKIT_CONSOLE_MESSAGE_LEVEL_ERROR,
+    WEBKIT_CONSOLE_MESSAGE_LEVEL_DEBUG
+} WebKitConsoleMessageLevel;
+
+typedef struct _WebKitConsoleMessage WebKitConsoleMessage;
+
+WEBKIT_API GType
+webkit_console_message_get_type      (void);
+
+WEBKIT_API WebKitConsoleMessage *
+webkit_console_message_copy          (WebKitConsoleMessage *console_message);
+
+WEBKIT_API void
+webkit_console_message_free          (WebKitConsoleMessage *console_message);
+
+WEBKIT_API WebKitConsoleMessageSource
+webkit_console_message_get_source    (WebKitConsoleMessage *console_message);
+
+WEBKIT_API WebKitConsoleMessageLevel
+webkit_console_message_get_level     (WebKitConsoleMessage *console_message);
+
+WEBKIT_API const gchar *
+webkit_console_message_get_text      (WebKitConsoleMessage *console_message);
+
+WEBKIT_API guint
+webkit_console_message_get_line      (WebKitConsoleMessage *console_message);
+
+WEBKIT_API const gchar *
+webkit_console_message_get_source_id (WebKitConsoleMessage *console_message);
+
+G_END_DECLS
+
+#endif

Added: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessagePrivate.h (0 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessagePrivate.h	                        (rev 0)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitConsoleMessagePrivate.h	2015-12-07 11:56:25 UTC (rev 193620)
@@ -0,0 +1,54 @@
+/*
+ * Copyright (C) 2015 Igalia S.L.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public License
+ * along with this library; see the file COPYING.LIB.  If not, write to
+ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
+ */
+
+#ifndef WebKitConsoleMessagePrivate_h
+#define WebKitConsoleMessagePrivate_h
+
+#include "WebKitConsoleMessage.h"
+#include <runtime/ConsoleTypes.h>
+#include <wtf/text/CString.h>
+#include <wtf/text/WTFString.h>
+
+struct _WebKitConsoleMessage {
+    _WebKitConsoleMessage(JSC::MessageSource source, JSC::MessageLevel level, const String& message, unsigned lineNumber, const String& sourceID)
+        : source(source)
+        , level(level)
+        , message(message.utf8())
+        , lineNumber(lineNumber)
+        , sourceID(sourceID.utf8())
+    {
+    }
+
+    _WebKitConsoleMessage(WebKitConsoleMessage* consoleMessage)
+        : source(consoleMessage->source)
+        , level(consoleMessage->level)
+        , message(consoleMessage->message)
+        , lineNumber(consoleMessage->lineNumber)
+        , sourceID(consoleMessage->sourceID)
+    {
+    }
+
+    JSC::MessageSource source;
+    JSC::MessageLevel level;
+    CString message;
+    unsigned lineNumber;
+    CString sourceID;
+};
+
+#endif // WebKitConsoleMessagePrivate_h

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp (193619 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp	2015-12-07 11:56:25 UTC (rev 193620)
@@ -28,6 +28,7 @@
 #include "WKBundleFrame.h"
 #include "WebContextMenuItem.h"
 #include "WebImage.h"
+#include "WebKitConsoleMessagePrivate.h"
 #include "WebKitContextMenuPrivate.h"
 #include "WebKitDOMDocumentPrivate.h"
 #include "WebKitFramePrivate.h"
@@ -49,6 +50,7 @@
 #include <glib/gi18n-lib.h>
 #include <wtf/NeverDestroyed.h>
 #include <wtf/text/CString.h>
+#include <wtf/text/StringBuilder.h>
 
 using namespace WebKit;
 using namespace WebCore;
@@ -57,6 +59,7 @@
     DOCUMENT_LOADED,
     SEND_REQUEST,
     CONTEXT_MENU,
+    CONSOLE_MESSAGE_SENT,
 
     LAST_SIGNAL
 };
@@ -144,6 +147,12 @@
     g_object_notify(G_OBJECT(webPage), "uri");
 }
 
+static void webkitWebPageDidSendConsoleMessage(WebKitWebPage* webPage, MessageSource source, MessageLevel level, const String& message, unsigned lineNumber, const String& sourceID)
+{
+    WebKitConsoleMessage consoleMessage(source, level, message, lineNumber, sourceID);
+    g_signal_emit(webPage, signals[CONSOLE_MESSAGE_SENT], 0, &consoleMessage);
+}
+
 static void didStartProvisionalLoadForFrame(WKBundlePageRef, WKBundleFrameRef frame, WKTypeRef*, const void *clientInfo)
 {
     if (!WKBundleFrameIsMainFrame(frame))
@@ -219,13 +228,25 @@
     return toAPI(&newRequest.leakRef());
 }
 
-static void didReceiveResponseForResource(WKBundlePageRef page, WKBundleFrameRef, uint64_t identifier, WKURLResponseRef response, const void*)
+static void didReceiveResponseForResource(WKBundlePageRef page, WKBundleFrameRef, uint64_t identifier, WKURLResponseRef response, const void* clientInfo)
 {
     API::Dictionary::MapType message;
     message.set(String::fromUTF8("Page"), toImpl(page));
     message.set(String::fromUTF8("Identifier"), API::UInt64::create(identifier));
     message.set(String::fromUTF8("Response"), toImpl(response));
     WebProcess::singleton().injectedBundle()->postMessage(String::fromUTF8("WebPage.DidReceiveResponseForResource"), API::Dictionary::create(WTF::move(message)).ptr());
+
+    // Post on the console as well to be consistent with the inspector.
+    const ResourceResponse& resourceResponse = toImpl(response)->resourceResponse();
+    if (resourceResponse.httpStatusCode() >= 400) {
+        StringBuilder errorMessage;
+        errorMessage.appendLiteral("Failed to load resource: the server responded with a status of ");
+        errorMessage.appendNumber(resourceResponse.httpStatusCode());
+        errorMessage.appendLiteral(" (");
+        errorMessage.append(resourceResponse.httpStatusText());
+        errorMessage.append(')');
+        webkitWebPageDidSendConsoleMessage(WEBKIT_WEB_PAGE(clientInfo), MessageSource::Network, MessageLevel::Error, errorMessage.toString(), 0, resourceResponse.url().string());
+    }
 }
 
 static void didReceiveContentLengthForResource(WKBundlePageRef page, WKBundleFrameRef, uint64_t identifier, uint64_t length, const void*)
@@ -245,13 +266,25 @@
     WebProcess::singleton().injectedBundle()->postMessage(String::fromUTF8("WebPage.DidFinishLoadForResource"), API::Dictionary::create(WTF::move(message)).ptr());
 }
 
-static void didFailLoadForResource(WKBundlePageRef page, WKBundleFrameRef, uint64_t identifier, WKErrorRef error, const void*)
+static void didFailLoadForResource(WKBundlePageRef page, WKBundleFrameRef, uint64_t identifier, WKErrorRef error, const void* clientInfo)
 {
     API::Dictionary::MapType message;
     message.set(String::fromUTF8("Page"), toImpl(page));
     message.set(String::fromUTF8("Identifier"), API::UInt64::create(identifier));
     message.set(String::fromUTF8("Error"), toImpl(error));
     WebProcess::singleton().injectedBundle()->postMessage(String::fromUTF8("WebPage.DidFailLoadForResource"), API::Dictionary::create(WTF::move(message)).ptr());
+
+    // Post on the console as well to be consistent with the inspector.
+    const ResourceError& resourceError = toImpl(error)->platformError();
+    if (!resourceError.isCancellation()) {
+        StringBuilder errorMessage;
+        errorMessage.appendLiteral("Failed to load resource");
+        if (!resourceError.localizedDescription().isEmpty()) {
+            errorMessage.appendLiteral(": ");
+            errorMessage.append(resourceError.localizedDescription());
+        }
+        webkitWebPageDidSendConsoleMessage(WEBKIT_WEB_PAGE(clientInfo), MessageSource::Network, MessageLevel::Error, errorMessage.toString(), 0, resourceError.failingURL());
+    }
 }
 
 class PageContextMenuClient final : public API::InjectedBundle::PageContextMenuClient {
@@ -283,6 +316,22 @@
     WebKitWebPage* m_webPage;
 };
 
+class PageUIClient final : public API::InjectedBundle::PageUIClient {
+public:
+    explicit PageUIClient(WebKitWebPage* webPage)
+        : m_webPage(webPage)
+    {
+    }
+
+private:
+    void willAddMessageToConsole(WebPage*, MessageSource source, MessageLevel level, const String& message, unsigned lineNumber, unsigned /*columnNumber*/, const String& sourceID) override
+    {
+        webkitWebPageDidSendConsoleMessage(m_webPage, source, level, message, lineNumber, sourceID);
+    }
+
+    WebKitWebPage* m_webPage;
+};
+
 static void webkitWebPageGetProperty(GObject* object, guint propId, GValue* value, GParamSpec* paramSpec)
 {
     WebKitWebPage* webPage = WEBKIT_WEB_PAGE(object);
@@ -395,6 +444,27 @@
         G_TYPE_BOOLEAN, 2,
         WEBKIT_TYPE_CONTEXT_MENU,
         WEBKIT_TYPE_WEB_HIT_TEST_RESULT);
+
+    /**
+     * WebKitWebPage::console-message-sent:
+     * @web_page: the #WebKitWebPage on which the signal is emitted
+     * @console_message: the #WebKitConsoleMessage
+     *
+     * Emmited when a message is sent to the console. This can be a message
+     * produced by the use of _javascript_ console API, a _javascript_ exception,
+     * a security error or other errors, warnings, debug or log messages.
+     * The @console_message contains information of the message.
+     *
+     * Since: 2.12
+     */
+    signals[CONSOLE_MESSAGE_SENT] = g_signal_new(
+        "console-message-sent",
+        G_TYPE_FROM_CLASS(klass),
+        G_SIGNAL_RUN_LAST,
+        0, 0, nullptr,
+        g_cclosure_marshal_VOID__BOXED,
+        G_TYPE_NONE, 1,
+        WEBKIT_TYPE_CONSOLE_MESSAGE | G_SIGNAL_TYPE_STATIC_SCOPE);
 }
 
 WebPage* webkitWebPageGetPage(WebKitWebPage *webPage)
@@ -466,6 +536,7 @@
     WKBundlePageSetResourceLoadClient(toAPI(webPage), &resourceLoadClient.base);
 
     webPage->setInjectedBundleContextMenuClient(std::make_unique<PageContextMenuClient>(page));
+    webPage->setInjectedBundleUIClient(std::make_unique<PageUIClient>(page));
 
     return page;
 }

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/webkit-web-extension.h (193619 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/webkit-web-extension.h	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/API/gtk/webkit-web-extension.h	2015-12-07 11:56:25 UTC (rev 193620)
@@ -26,6 +26,7 @@
 
 #define __WEBKIT_WEB_EXTENSION_H_INSIDE__
 
+#include <webkit2/WebKitConsoleMessage.h>
 #include <webkit2/WebKitContextMenu.h>
 #include <webkit2/WebKitContextMenuActions.h>
 #include <webkit2/WebKitContextMenuItem.h>

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp (193619 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp	2015-12-07 11:56:25 UTC (rev 193620)
@@ -44,7 +44,7 @@
     initialize(client);
 }
 
-void InjectedBundlePageUIClient::willAddMessageToConsole(WebPage* page, const String& message, int32_t lineNumber)
+void InjectedBundlePageUIClient::willAddMessageToConsole(WebPage* page, MessageSource, MessageLevel, const String& message, unsigned lineNumber, unsigned /*columnNumber*/, const String& /*sourceID*/)
 {
     if (m_client.willAddMessageToConsole)
         m_client.willAddMessageToConsole(toAPI(page), toAPI(message.impl()), lineNumber, m_client.base.clientInfo);

Modified: trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageUIClient.h (193619 => 193620)


--- trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageUIClient.h	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageUIClient.h	2015-12-07 11:56:25 UTC (rev 193620)
@@ -47,7 +47,7 @@
 public:
     explicit InjectedBundlePageUIClient(const WKBundlePageUIClientBase*);
 
-    void willAddMessageToConsole(WebPage*, const String& message, int32_t lineNumber) override;
+    void willAddMessageToConsole(WebPage*, MessageSource, MessageLevel, const String& message, unsigned lineNumber, unsigned columnNumber, const String& sourceID) override;
     void willSetStatusbarText(WebPage*, const String&) override;
     void willRunJavaScriptAlert(WebPage*, const String&, WebFrame*) override;
     void willRunJavaScriptConfirm(WebPage*, const String&, WebFrame*) override;

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp (193619 => 193620)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp	2015-12-07 11:56:25 UTC (rev 193620)
@@ -319,12 +319,10 @@
     m_page->send(Messages::WebPageProxy::SetIsResizable(resizable));
 }
 
-void WebChromeClient::addMessageToConsole(MessageSource, MessageLevel, const String& message, unsigned lineNumber, unsigned /*columnNumber*/, const String& /*sourceID*/)
+void WebChromeClient::addMessageToConsole(MessageSource source, MessageLevel level, const String& message, unsigned lineNumber, unsigned columnNumber, const String& sourceID)
 {
     // Notify the bundle client.
-    m_page->injectedBundleUIClient().willAddMessageToConsole(m_page, message, lineNumber);
-
-    notImplemented();
+    m_page->injectedBundleUIClient().willAddMessageToConsole(m_page, source, level, message, lineNumber, columnNumber, sourceID);
 }
 
 bool WebChromeClient::canRunBeforeUnloadConfirmPanel()

Modified: trunk/Tools/ChangeLog (193619 => 193620)


--- trunk/Tools/ChangeLog	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Tools/ChangeLog	2015-12-07 11:56:25 UTC (rev 193620)
@@ -1,3 +1,29 @@
+2015-12-07  Carlos Garcia Campos  <cgar...@igalia.com>
+
+        [GTK] Add WebKitWebPage::console-message-sent signal to Web Extensions API
+        https://bugs.webkit.org/show_bug.cgi?id=79918
+
+        Reviewed by Gustavo Noronha Silva.
+
+        Add unit tests to check the different console messages.
+
+        * TestWebKitAPI/Tests/WebKit2Gtk/CMakeLists.txt:
+        * TestWebKitAPI/Tests/WebKit2Gtk/TestConsoleMessage.cpp: Added.
+        (ConsoleMessageTest::ConsoleMessage::operator==):
+        (ConsoleMessageTest::consoleMessageReceivedCallback):
+        (ConsoleMessageTest::ConsoleMessageTest):
+        (ConsoleMessageTest::~ConsoleMessageTest):
+        (ConsoleMessageTest::waitUntilConsoleMessageReceived):
+        (testWebKitConsoleMessageConsoleAPI):
+        (testWebKitConsoleMessageJavaScriptException):
+        (testWebKitConsoleMessageNetworkError):
+        (testWebKitConsoleMessageSecurityError):
+        (beforeAll):
+        (afterAll):
+        * TestWebKitAPI/Tests/WebKit2Gtk/WebExtensionTest.cpp:
+        (consoleMessageSentCallback):
+        (pageCreatedCallback):
+
 2015-12-07  Ryuan Choi  <ryuan.c...@navercorp.com>
 
         [EFL] MiniBrowser doesn't exit when called ewk_view_try_close()

Modified: trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/CMakeLists.txt (193619 => 193620)


--- trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/CMakeLists.txt	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/CMakeLists.txt	2015-12-07 11:56:25 UTC (rev 193620)
@@ -127,6 +127,7 @@
 ADD_WK2_TEST(TestWebKitWebView TestWebKitWebView.cpp)
 ADD_WK2_TEST(TestWebKitUserContentManager TestWebKitUserContentManager.cpp)
 ADD_WK2_TEST(TestEditor TestEditor.cpp)
+ADD_WK2_TEST(TestConsoleMessage TestConsoleMessage.cpp)
 
 if (ATSPI_FOUND)
     ADD_WK2_TEST(AccessibilityTestServer AccessibilityTestServer.cpp)

Added: trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestConsoleMessage.cpp (0 => 193620)


--- trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestConsoleMessage.cpp	                        (rev 0)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestConsoleMessage.cpp	2015-12-07 11:56:25 UTC (rev 193620)
@@ -0,0 +1,158 @@
+/*
+ * Copyright (C) 2015 Igalia S.L.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2,1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public License
+ * along with this library; see the file COPYING.LIB.  If not, write to
+ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
+ */
+
+#include "config.h"
+
+#include "WebViewTest.h"
+#include <gtk/gtk.h>
+#include <webkit2/webkit2.h>
+
+class ConsoleMessageTest : public WebViewTest {
+public:
+    MAKE_GLIB_TEST_FIXTURE(ConsoleMessageTest);
+
+    // This should be keep in sync with the public enums in WebKitConsoleMessage.h.
+    enum class MessageSource { _javascript_, Network, ConsoleAPI, Security, Other };
+    enum class MessageLevel { Info, Log, Warning, Error, Debug };
+    struct ConsoleMessage {
+        bool operator==(const ConsoleMessage& other)
+        {
+            return source == other.source
+                && level == other.level
+                && message == other.message
+                && lineNumber == other.lineNumber
+                && sourceID == other.sourceID;
+        }
+
+        MessageSource source;
+        MessageLevel level;
+        CString message;
+        unsigned lineNumber;
+        CString sourceID;
+    };
+
+    static void consoleMessageReceivedCallback(WebKitUserContentManager*, WebKitJavascriptResult* message, ConsoleMessageTest* test)
+    {
+        g_assert(message);
+        GUniquePtr<char> messageString(WebViewTest::_javascript_ResultToCString(message));
+        GRefPtr<GVariant> variant = g_variant_parse(G_VARIANT_TYPE("(uusus)"), messageString.get(), nullptr, nullptr, nullptr);
+        g_assert(variant.get());
+
+        unsigned source, level, lineNumber;
+        const char* messageText;
+        const char* sourceID;
+        g_variant_get(variant.get(), "(uu&su&s)", &source, &level, &messageText, &lineNumber, &sourceID);
+        test->m_consoleMessage = { static_cast<ConsoleMessageTest::MessageSource>(source), static_cast<ConsoleMessageTest::MessageLevel>(level), messageText, lineNumber, sourceID };
+
+        g_main_loop_quit(test->m_mainLoop);
+    }
+
+    ConsoleMessageTest()
+        : WebViewTest(webkit_user_content_manager_new())
+    {
+        WebKitUserContentManager* manager = webkit_web_view_get_user_content_manager(m_webView);
+        webkit_user_content_manager_register_script_message_handler(manager, "console");
+        g_signal_connect(manager, "script-message-received::console", G_CALLBACK(consoleMessageReceivedCallback), this);
+    }
+
+    ~ConsoleMessageTest()
+    {
+        WebKitUserContentManager* manager = webkit_web_view_get_user_content_manager(m_webView);
+        g_signal_handlers_disconnect_matched(manager, G_SIGNAL_MATCH_DATA, 0, 0, nullptr, nullptr, this);
+        webkit_user_content_manager_unregister_script_message_handler(manager, "console");
+    }
+
+    void waitUntilConsoleMessageReceived()
+    {
+        g_main_loop_run(m_mainLoop);
+    }
+
+    ConsoleMessage m_consoleMessage;
+};
+
+static void testWebKitConsoleMessageConsoleAPI(ConsoleMessageTest* test, gconstpointer)
+{
+    ConsoleMessageTest::ConsoleMessage referenceMessage = { ConsoleMessageTest::MessageSource::ConsoleAPI, ConsoleMessageTest::MessageLevel::Log, "Log Console Message", 1, "http://foo.com/bar" };
+    test->loadHtml("<html><body _onload_='console.log(\"Log Console Message\");'></body></html>", "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+
+    referenceMessage.level = ConsoleMessageTest::MessageLevel::Info;
+    referenceMessage.message = "Info Console Message";
+    test->loadHtml("<html><body _onload_='console.info(\"Info Console Message\");'></body></html>", "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+
+    referenceMessage.level = ConsoleMessageTest::MessageLevel::Warning;
+    referenceMessage.message = "Warning Console Message";
+    test->loadHtml("<html><body _onload_='console.warn(\"Warning Console Message\");'></body></html>", "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+
+    referenceMessage.level = ConsoleMessageTest::MessageLevel::Error;
+    referenceMessage.message = "Error Console Message";
+    test->loadHtml("<html><body _onload_='console.error(\"Error Console Message\");'></body></html>", "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+
+    referenceMessage.level = ConsoleMessageTest::MessageLevel::Debug;
+    referenceMessage.message = "Debug Console Message";
+    test->loadHtml("<html><body _onload_='console.debug(\"Debug Console Message\");'></body></html>", "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+}
+
+static void testWebKitConsoleMessageJavaScriptException(ConsoleMessageTest* test, gconstpointer)
+{
+    ConsoleMessageTest::ConsoleMessage referenceMessage = { ConsoleMessageTest::MessageSource::_javascript_, ConsoleMessageTest::MessageLevel::Error,
+        "ReferenceError: Can't find variable: foo", 1, "http://foo.com/bar" };
+    test->loadHtml("<html><body _onload_='foo()'></body></html>", "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+}
+
+static void testWebKitConsoleMessageNetworkError(ConsoleMessageTest* test, gconstpointer)
+{
+    ConsoleMessageTest::ConsoleMessage referenceMessage = { ConsoleMessageTest::MessageSource::Network, ConsoleMessageTest::MessageLevel::Error,
+        "Failed to load resource: Error opening file: No such file or directory", 0, "file:///foo/not-found.css" };
+    test->loadHtml("<html><head><link rel='stylesheet' href='' type='text/css'></head><body></body></html>", "file:///foo/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+}
+
+static void testWebKitConsoleMessageSecurityError(ConsoleMessageTest* test, gconstpointer)
+{
+    ConsoleMessageTest::ConsoleMessage referenceMessage = { ConsoleMessageTest::MessageSource::Security, ConsoleMessageTest::MessageLevel::Error,
+        "Not allowed to load local resource: file:///foo/bar/source.png", 1, "http://foo.com/bar" };
+    test->loadHtml("<html><body><img src="" "http://foo.com/bar");
+    test->waitUntilConsoleMessageReceived();
+    g_assert(test->m_consoleMessage == referenceMessage);
+}
+
+void beforeAll()
+{
+    ConsoleMessageTest::add("WebKitConsoleMessage", "console-api", testWebKitConsoleMessageConsoleAPI);
+    ConsoleMessageTest::add("WebKitConsoleMessage", "js-exception", testWebKitConsoleMessageJavaScriptException);
+    ConsoleMessageTest::add("WebKitConsoleMessage", "network-error", testWebKitConsoleMessageNetworkError);
+    ConsoleMessageTest::add("WebKitConsoleMessage", "security-error", testWebKitConsoleMessageSecurityError);
+}
+
+void afterAll()
+{
+}

Modified: trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/WebExtensionTest.cpp (193619 => 193620)


--- trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/WebExtensionTest.cpp	2015-12-07 11:46:28 UTC (rev 193619)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit2Gtk/WebExtensionTest.cpp	2015-12-07 11:56:25 UTC (rev 193620)
@@ -226,12 +226,25 @@
     return FALSE;
 }
 
+static void consoleMessageSentCallback(WebKitWebPage* webPage, WebKitConsoleMessage* consoleMessage)
+{
+    g_assert(consoleMessage);
+    GRefPtr<GVariant> variant = g_variant_new("(uusus)", webkit_console_message_get_source(consoleMessage),
+        webkit_console_message_get_level(consoleMessage), webkit_console_message_get_text(consoleMessage),
+        webkit_console_message_get_line(consoleMessage), webkit_console_message_get_source_id(consoleMessage));
+    GUniquePtr<char> messageString(g_variant_print(variant.get(), FALSE));
+    GRefPtr<WebKitDOMDOMWindow> window = adoptGRef(webkit_dom_document_get_default_view(webkit_web_page_get_dom_document(webPage)));
+    g_assert(WEBKIT_DOM_IS_DOM_WINDOW(window.get()));
+    webkit_dom_dom_window_webkit_message_handlers_post_message(window.get(), "console", messageString.get());
+}
+
 static void pageCreatedCallback(WebKitWebExtension* extension, WebKitWebPage* webPage, gpointer)
 {
     g_signal_connect(webPage, "document-loaded", G_CALLBACK(documentLoadedCallback), extension);
     g_signal_connect(webPage, "notify::uri", G_CALLBACK(uriChangedCallback), extension);
     g_signal_connect(webPage, "send-request", G_CALLBACK(sendRequestCallback), nullptr);
     g_signal_connect(webPage, "context-menu", G_CALLBACK(contextMenuCallback), nullptr);
+    g_signal_connect(webPage, "console-message-sent", G_CALLBACK(consoleMessageSentCallback), nullptr);
 }
 
 static JSValueRef echoCallback(JSContextRef jsContext, JSObjectRef, JSObjectRef, size_t argumentCount, const JSValueRef arguments[], JSValueRef*)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to