2007-12-06 Xan Lopez <xan@gnome.org>
authoralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Dec 2007 20:11:04 +0000 (20:11 +0000)
committeralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Dec 2007 20:11:04 +0000 (20:11 +0000)
        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=16329
        [GTK] Two small cleanups

        * WebCoreSupport/ChromeClientGtk.cpp:
        (WebKit::ChromeClient::createWindow):
        (WebKit::ChromeClient::addMessageToConsole):
        (WebKit::ChromeClient::runJavaScriptAlert):
        (WebKit::ChromeClient::runJavaScriptConfirm):
        (WebKit::ChromeClient::runJavaScriptPrompt):
        (WebKit::ChromeClient::setStatusbarText):
        (WebKit::ChromeClient::mouseDidMoveOverElement):
        (WebKit::ChromeClient::setToolTip):
        * WebCoreSupport/ChromeClientGtk.h:
        * WebView/webkitprivate.cpp:
        (WebKit::kit):
        s/m_webPage/m_webView/
        * WebView/webkitwebview.cpp:
        Chain up to the parent class to activate bindings instead
        of doing it explicitely.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@28490 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/gtk/ChangeLog
WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp
WebKit/gtk/WebCoreSupport/ChromeClientGtk.h
WebKit/gtk/WebView/webkitprivate.cpp
WebKit/gtk/WebView/webkitwebview.cpp

index 1d666474c03e0621a8ccf332d5059040ab015336..493355dbad6cd6769fd15a159274854950c32252 100644 (file)
@@ -1,3 +1,27 @@
+2007-12-06  Xan Lopez  <xan@gnome.org>
+
+        Reviewed by Alp Toker.
+
+        http://bugs.webkit.org/show_bug.cgi?id=16329
+        [GTK] Two small cleanups
+
+        * WebCoreSupport/ChromeClientGtk.cpp:
+        (WebKit::ChromeClient::createWindow):
+        (WebKit::ChromeClient::addMessageToConsole):
+        (WebKit::ChromeClient::runJavaScriptAlert):
+        (WebKit::ChromeClient::runJavaScriptConfirm):
+        (WebKit::ChromeClient::runJavaScriptPrompt):
+        (WebKit::ChromeClient::setStatusbarText):
+        (WebKit::ChromeClient::mouseDidMoveOverElement):
+        (WebKit::ChromeClient::setToolTip):
+        * WebCoreSupport/ChromeClientGtk.h:
+        * WebView/webkitprivate.cpp:
+        (WebKit::kit):
+        s/m_webPage/m_webView/
+        * WebView/webkitwebview.cpp:
+        Chain up to the parent class to activate bindings instead
+        of doing it explicitely.
+
 2007-12-06  Holger Hans Peter Freyther <holger.freyther@trolltech.com>
 
         Reviewed by Alp Toker.
index fdf03ed68de2987a0852a2ee5ce987fe60d9901b..8ec91b8662ba73f635a607cd0c5aca001477fb35 100644 (file)
@@ -43,8 +43,8 @@
 using namespace WebCore;
 
 namespace WebKit {
-ChromeClient::ChromeClient(WebKitWebView* page)
-    : m_webPage(page)
+ChromeClient::ChromeClient(WebKitWebView* webView)
+    : m_webView(webView)
 {
 }
 
@@ -93,11 +93,11 @@ Page* ChromeClient::createWindow(Frame*, const FrameLoadRequest&, const WindowFe
         return 0;
     } else {
         /* TODO: FrameLoadRequest is not used */
-        WebKitWebView* page = WEBKIT_WEB_VIEW_GET_CLASS(m_webPage)->create_web_view(m_webPage);
-        if (!page)
+        WebKitWebView* webView = WEBKIT_WEB_VIEW_GET_CLASS(m_webView)->create_web_view(m_webView);
+        if (!webView)
             return 0;
 
-        WebKitWebViewPrivate *privateData = WEBKIT_WEB_VIEW_GET_PRIVATE(WEBKIT_WEB_VIEW(page));
+        WebKitWebViewPrivate* privateData = WEBKIT_WEB_VIEW_GET_PRIVATE(webView);
         return privateData->corePage;
     }
 }
@@ -197,20 +197,20 @@ bool ChromeClient::runBeforeUnloadConfirmPanel(const WebCore::String&, WebCore::
 void ChromeClient::addMessageToConsole(const WebCore::String& message, unsigned int lineNumber, const WebCore::String& sourceId)
 {
     gboolean retval;
-    g_signal_emit_by_name(m_webPage, "console-message", message.utf8().data(), lineNumber, sourceId.utf8().data(), &retval);
+    g_signal_emit_by_name(m_webView, "console-message", message.utf8().data(), lineNumber, sourceId.utf8().data(), &retval);
 }
 
 void ChromeClient::runJavaScriptAlert(Frame* frame, const String& message)
 {
     gboolean retval;
-    g_signal_emit_by_name(m_webPage, "script-alert", kit(frame), message.utf8().data(), &retval);
+    g_signal_emit_by_name(m_webView, "script-alert", kit(frame), message.utf8().data(), &retval);
 }
 
 bool ChromeClient::runJavaScriptConfirm(Frame* frame, const String& message)
 {
     gboolean retval;
     gboolean didConfirm;
-    g_signal_emit_by_name(m_webPage, "script-confirm", kit(frame), message.utf8().data(), &didConfirm, &retval);
+    g_signal_emit_by_name(m_webView, "script-confirm", kit(frame), message.utf8().data(), &didConfirm, &retval);
     return didConfirm == TRUE;
 }
 
@@ -218,7 +218,7 @@ bool ChromeClient::runJavaScriptPrompt(Frame* frame, const String& message, cons
 {
     gboolean retval;
     gchar* value = 0;
-    g_signal_emit_by_name(m_webPage, "script-prompt", kit(frame), message.utf8().data(), defaultValue.utf8().data(), &value, &retval);
+    g_signal_emit_by_name(m_webView, "script-prompt", kit(frame), message.utf8().data(), defaultValue.utf8().data(), &value, &retval);
     if (value) {
         result = String::fromUTF8(value);
         g_free(value);
@@ -230,7 +230,7 @@ bool ChromeClient::runJavaScriptPrompt(Frame* frame, const String& message, cons
 void ChromeClient::setStatusbarText(const String& string)
 {
     CString stringMessage = string.utf8();
-    g_signal_emit_by_name(m_webPage, "status-bar-text-changed", stringMessage.data());
+    g_signal_emit_by_name(m_webView, "status-bar-text-changed", stringMessage.data());
 }
 
 bool ChromeClient::shouldInterruptJavaScript()
@@ -274,11 +274,11 @@ void ChromeClient::mouseDidMoveOverElement(const HitTestResult& hit, unsigned mo
         if (!url.isEmpty() && url != m_hoveredLinkURL) {
             CString titleString = hit.title().utf8();
             DeprecatedCString urlString = url.prettyURL().utf8();
-            g_signal_emit_by_name(m_webPage, "hovering-over-link", titleString.data(), urlString.data());
+            g_signal_emit_by_name(m_webView, "hovering-over-link", titleString.data(), urlString.data());
             m_hoveredLinkURL = url;
         }
     } else if (!isLink && !m_hoveredLinkURL.isEmpty()) {
-        g_signal_emit_by_name(m_webPage, "hovering-over-link", 0, 0);
+        g_signal_emit_by_name(m_webView, "hovering-over-link", 0, 0);
         m_hoveredLinkURL = KURL();
     }
 }
@@ -287,9 +287,9 @@ void ChromeClient::setToolTip(const String& toolTip)
 {
 #if GTK_CHECK_VERSION(2,12,0)
     if (toolTip.isEmpty())
-        g_object_set(G_OBJECT(m_webPage), "has-tooltip", FALSE, NULL);
+        g_object_set(G_OBJECT(m_webView), "has-tooltip", FALSE, NULL);
     else
-        gtk_widget_set_tooltip_text(GTK_WIDGET(m_webPage), toolTip.utf8().data());
+        gtk_widget_set_tooltip_text(GTK_WIDGET(m_webView), toolTip.utf8().data());
 #else
     // TODO: Support older GTK+ versions
     // See http://bugs.webkit.org/show_bug.cgi?id=15793
index 7af0e606c96b242d2ee74252b51e697b8b7089bf..1c96b420ddab68b93d3aca1cd0327b6c702bfe87 100644 (file)
@@ -30,7 +30,7 @@ namespace WebKit {
     class ChromeClient : public WebCore::ChromeClient {
     public:
         ChromeClient(WebKitWebView*);
-        WebKitWebView* webPage() const { return m_webPage; }
+        WebKitWebView* webView() const { return m_webView; }
 
         virtual void chromeDestroyed();
 
@@ -97,7 +97,7 @@ namespace WebKit {
         virtual unsigned long long requestQuotaIncreaseForDatabaseOperation(WebCore::Frame*, const WebCore::SecurityOriginData&, const WebCore::String&, unsigned long long);
 
     private:
-        WebKitWebView* m_webPage;
+        WebKitWebView* m_webView;
         WebCore::KURL m_hoveredLinkURL;
     };
 }
index 55a39a8e37f22db38bad76b4e536236b07312acf..0bc2b18dbf094fe8aaaa61a013a36d8c9f0c913c 100644 (file)
@@ -84,6 +84,6 @@ WebKitWebView* kit(WebCore::Page* corePage)
 
     ASSERT(corePage->chrome());
     WebKit::ChromeClient* client = static_cast<WebKit::ChromeClient*>(corePage->chrome()->client());
-    return client ? client->webPage() : 0;
+    return client ? client->webView() : 0;
 }
 }
index f92a5b1818a18ee27fc35eaa34856e99e67f00dd..a649695256259f62698ce010e2a6664c19cb468c 100644 (file)
@@ -141,7 +141,8 @@ static gboolean webkit_web_view_key_press_event(GtkWidget* widget, GdkEventKey*
         return TRUE;
     }
 
-    return gtk_bindings_activate_event(GTK_OBJECT(widget), event);
+    /* Chain up to our parent class for binding activation */
+    return GTK_WIDGET_CLASS(webkit_web_view_parent_class)->key_press_event(widget, event);
 }
 
 static gboolean webkit_web_view_key_release_event(GtkWidget* widget, GdkEventKey* event)
@@ -152,7 +153,8 @@ static gboolean webkit_web_view_key_release_event(GtkWidget* widget, GdkEventKey
     if (frame->eventHandler()->keyEvent(keyboardEvent))
         return TRUE;
 
-    return FALSE;
+    /* Chain up to our parent class for binding activation */
+    return GTK_WIDGET_CLASS(webkit_web_view_parent_class)->key_release_event(widget, event);
 }
 
 static gboolean webkit_web_view_button_press_event(GtkWidget* widget, GdkEventButton* event)