From: Paul Cornett Date: Thu, 1 Mar 2012 16:36:58 +0000 (+0000) Subject: use m_ prefix for web_view member, and make it a WebKitWebView* to eliminate many... X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/4c9bde5ec04f4c9d1ba1d0181c1ae8a56882fd6a?ds=inline use m_ prefix for web_view member, and make it a WebKitWebView* to eliminate many casts git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@70767 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- diff --git a/include/wx/gtk/webview_webkit.h b/include/wx/gtk/webview_webkit.h index 2cef91f1c6..d72a398b61 100644 --- a/include/wx/gtk/webview_webkit.h +++ b/include/wx/gtk/webview_webkit.h @@ -17,6 +17,8 @@ #include "wx/sharedptr.h" #include "wx/webview.h" +typedef struct _WebKitWebView WebKitWebView; + //----------------------------------------------------------------------------- // wxWebViewWebKit //----------------------------------------------------------------------------- @@ -145,7 +147,7 @@ private: // focus event handler: calls GTKUpdateBitmap() void GTKOnFocus(wxFocusEvent& event); - GtkWidget *web_view; + WebKitWebView *m_web_view; int m_historyLimit; wxVector > m_handlerList; diff --git a/src/gtk/webview_webkit.cpp b/src/gtk/webview_webkit.cpp index 02117610b2..60d04e3c3e 100644 --- a/src/gtk/webview_webkit.cpp +++ b/src/gtk/webview_webkit.cpp @@ -408,29 +408,29 @@ bool wxWebViewWebKit::Create(wxWindow *parent, m_widget = gtk_scrolled_window_new(NULL, NULL); g_object_ref(m_widget); - web_view = webkit_web_view_new (); + m_web_view = WEBKIT_WEB_VIEW(webkit_web_view_new()); /* Place the WebKitWebView in the GtkScrolledWindow */ - gtk_container_add(GTK_CONTAINER(m_widget), web_view); - gtk_widget_show(web_view); + gtk_container_add(GTK_CONTAINER(m_widget), GTK_WIDGET(m_web_view)); + gtk_widget_show(GTK_WIDGET(m_web_view)); - g_signal_connect_after(web_view, "notify::load-status", + g_signal_connect_after(m_web_view, "notify::load-status", G_CALLBACK(wxgtk_webview_webkit_load_status), this); - g_signal_connect_after(web_view, "navigation-policy-decision-requested", + g_signal_connect_after(m_web_view, "navigation-policy-decision-requested", G_CALLBACK(wxgtk_webview_webkit_navigation), this); - g_signal_connect_after(web_view, "load-error", + g_signal_connect_after(m_web_view, "load-error", G_CALLBACK(wxgtk_webview_webkit_error), this); - g_signal_connect_after(web_view, "new-window-policy-decision-requested", + g_signal_connect_after(m_web_view, "new-window-policy-decision-requested", G_CALLBACK(wxgtk_webview_webkit_new_window), this); - g_signal_connect_after(web_view, "title-changed", + g_signal_connect_after(m_web_view, "title-changed", G_CALLBACK(wxgtk_webview_webkit_title_changed), this); - g_signal_connect_after(web_view, "resource-request-starting", + g_signal_connect_after(m_web_view, "resource-request-starting", G_CALLBACK(wxgtk_webview_webkit_resource_req), this); m_parent->DoAddChild( this ); @@ -438,11 +438,11 @@ bool wxWebViewWebKit::Create(wxWindow *parent, PostCreation(size); /* Open a webpage */ - webkit_web_view_load_uri (WEBKIT_WEB_VIEW (web_view), url.utf8_str()); + webkit_web_view_load_uri(m_web_view, url.utf8_str()); //Get the initial history limit so we can enable and disable it later WebKitWebBackForwardList* history; - history = webkit_web_view_get_back_forward_list(WEBKIT_WEB_VIEW(web_view)); + history = webkit_web_view_get_back_forward_list(m_web_view); m_historyLimit = webkit_web_back_forward_list_get_limit(history); m_ready = true; @@ -472,80 +472,80 @@ wxWebViewWebKit::GTKGetWindow(wxArrayGdkWindows& WXUNUSED(windows)) const void wxWebViewWebKit::ZoomIn() { - webkit_web_view_zoom_in (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_zoom_in(m_web_view); } void wxWebViewWebKit::ZoomOut() { - webkit_web_view_zoom_out (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_zoom_out(m_web_view); } void wxWebViewWebKit::SetWebkitZoom(float level) { - webkit_web_view_set_zoom_level (WEBKIT_WEB_VIEW(web_view), level); + webkit_web_view_set_zoom_level(m_web_view, level); } float wxWebViewWebKit::GetWebkitZoom() const { - return webkit_web_view_get_zoom_level (WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_get_zoom_level(m_web_view); } void wxWebViewWebKit::Stop() { - webkit_web_view_stop_loading (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_stop_loading(m_web_view); } void wxWebViewWebKit::Reload(wxWebViewReloadFlags flags) { if (flags & wxWEB_VIEW_RELOAD_NO_CACHE) { - webkit_web_view_reload_bypass_cache (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_reload_bypass_cache(m_web_view); } else { - webkit_web_view_reload (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_reload(m_web_view); } } void wxWebViewWebKit::LoadURL(const wxString& url) { - webkit_web_view_load_uri(WEBKIT_WEB_VIEW(web_view), wxGTK_CONV(url)); + webkit_web_view_load_uri(m_web_view, wxGTK_CONV(url)); } void wxWebViewWebKit::GoBack() { - webkit_web_view_go_back (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_go_back(m_web_view); } void wxWebViewWebKit::GoForward() { - webkit_web_view_go_forward (WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_go_forward(m_web_view); } bool wxWebViewWebKit::CanGoBack() const { - return webkit_web_view_can_go_back (WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_go_back(m_web_view); } bool wxWebViewWebKit::CanGoForward() const { - return webkit_web_view_can_go_forward (WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_go_forward(m_web_view); } void wxWebViewWebKit::ClearHistory() { WebKitWebBackForwardList* history; - history = webkit_web_view_get_back_forward_list(WEBKIT_WEB_VIEW(web_view)); + history = webkit_web_view_get_back_forward_list(m_web_view); webkit_web_back_forward_list_clear(history); } void wxWebViewWebKit::EnableHistory(bool enable) { WebKitWebBackForwardList* history; - history = webkit_web_view_get_back_forward_list(WEBKIT_WEB_VIEW(web_view)); + history = webkit_web_view_get_back_forward_list(m_web_view); if(enable) { webkit_web_back_forward_list_set_limit(history, m_historyLimit); @@ -560,7 +560,7 @@ wxVector > wxWebViewWebKit::GetBackwardHistory { wxVector > backhist; WebKitWebBackForwardList* history; - history = webkit_web_view_get_back_forward_list(WEBKIT_WEB_VIEW(web_view)); + history = webkit_web_view_get_back_forward_list(m_web_view); GList* list = webkit_web_back_forward_list_get_back_list_with_limit(history, m_historyLimit); //We need to iterate in reverse to get the order we desire @@ -581,7 +581,7 @@ wxVector > wxWebViewWebKit::GetForwardHistory( { wxVector > forwardhist; WebKitWebBackForwardList* history; - history = webkit_web_view_get_back_forward_list(WEBKIT_WEB_VIEW(web_view)); + history = webkit_web_view_get_back_forward_list(m_web_view); GList* list = webkit_web_back_forward_list_get_forward_list_with_limit(history, m_historyLimit); for(guint i = 0; i < g_list_length(list); i++) @@ -602,81 +602,78 @@ void wxWebViewWebKit::LoadHistoryItem(wxSharedPtr item) WebKitWebHistoryItem* gtkitem = (WebKitWebHistoryItem*)item->m_histItem; if(gtkitem) { - webkit_web_view_go_to_back_forward_item(WEBKIT_WEB_VIEW(web_view), + webkit_web_view_go_to_back_forward_item(m_web_view, WEBKIT_WEB_HISTORY_ITEM(gtkitem)); } } bool wxWebViewWebKit::CanCut() const { - return webkit_web_view_can_cut_clipboard(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_cut_clipboard(m_web_view); } bool wxWebViewWebKit::CanCopy() const { - return webkit_web_view_can_copy_clipboard(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_copy_clipboard(m_web_view); } bool wxWebViewWebKit::CanPaste() const { - return webkit_web_view_can_paste_clipboard(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_paste_clipboard(m_web_view); } void wxWebViewWebKit::Cut() { - webkit_web_view_cut_clipboard(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_cut_clipboard(m_web_view); } void wxWebViewWebKit::Copy() { - webkit_web_view_copy_clipboard(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_copy_clipboard(m_web_view); } void wxWebViewWebKit::Paste() { - webkit_web_view_paste_clipboard(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_paste_clipboard(m_web_view); } bool wxWebViewWebKit::CanUndo() const { - return webkit_web_view_can_undo(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_undo(m_web_view); } bool wxWebViewWebKit::CanRedo() const { - return webkit_web_view_can_redo(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_can_redo(m_web_view); } void wxWebViewWebKit::Undo() { - webkit_web_view_undo(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_undo(m_web_view); } void wxWebViewWebKit::Redo() { - webkit_web_view_redo(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_redo(m_web_view); } wxString wxWebViewWebKit::GetCurrentURL() const { // FIXME: check which encoding the web kit control uses instead of // assuming UTF8 (here and elsewhere too) - return wxString::FromUTF8(webkit_web_view_get_uri( - WEBKIT_WEB_VIEW(web_view))); + return wxString::FromUTF8(webkit_web_view_get_uri(m_web_view)); } wxString wxWebViewWebKit::GetCurrentTitle() const { - return wxString::FromUTF8(webkit_web_view_get_title( - WEBKIT_WEB_VIEW(web_view))); + return wxString::FromUTF8(webkit_web_view_get_title(m_web_view)); } wxString wxWebViewWebKit::GetPageSource() const { - WebKitWebFrame* frame = webkit_web_view_get_main_frame( - WEBKIT_WEB_VIEW(web_view)); + WebKitWebFrame* frame = webkit_web_view_get_main_frame(m_web_view); WebKitWebDataSource* src = webkit_web_frame_get_data_source (frame); // TODO: check encoding with @@ -750,15 +747,14 @@ void wxWebViewWebKit::SetZoom(wxWebViewZoom zoom) void wxWebViewWebKit::SetZoomType(wxWebViewZoomType type) { - webkit_web_view_set_full_content_zoom(WEBKIT_WEB_VIEW(web_view), + webkit_web_view_set_full_content_zoom(m_web_view, (type == wxWEB_VIEW_ZOOM_TYPE_LAYOUT ? TRUE : FALSE)); } wxWebViewZoomType wxWebViewWebKit::GetZoomType() const { - gboolean fczoom = webkit_web_view_get_full_content_zoom( - WEBKIT_WEB_VIEW(web_view)); + gboolean fczoom = webkit_web_view_get_full_content_zoom(m_web_view); if (fczoom) return wxWEB_VIEW_ZOOM_TYPE_LAYOUT; else return wxWEB_VIEW_ZOOM_TYPE_TEXT; @@ -772,7 +768,7 @@ bool wxWebViewWebKit::CanSetZoomType(wxWebViewZoomType) const void wxWebViewWebKit::SetPage(const wxString& html, const wxString& baseUri) { - webkit_web_view_load_string (WEBKIT_WEB_VIEW(web_view), + webkit_web_view_load_string (m_web_view, html.mb_str(wxConvUTF8), "text/html", "UTF-8", @@ -781,8 +777,7 @@ void wxWebViewWebKit::SetPage(const wxString& html, const wxString& baseUri) void wxWebViewWebKit::Print() { - WebKitWebFrame* frame = webkit_web_view_get_main_frame( - WEBKIT_WEB_VIEW(web_view)); + WebKitWebFrame* frame = webkit_web_view_get_main_frame(m_web_view); webkit_web_frame_print (frame); // GtkPrintOperationResult webkit_web_frame_print_full @@ -822,27 +817,27 @@ bool wxWebViewWebKit::IsBusy() const void wxWebViewWebKit::SetEditable(bool enable) { - webkit_web_view_set_editable(WEBKIT_WEB_VIEW(web_view), enable); + webkit_web_view_set_editable(m_web_view, enable); } bool wxWebViewWebKit::IsEditable() const { - return webkit_web_view_get_editable(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_get_editable(m_web_view); } void wxWebViewWebKit::DeleteSelection() { - webkit_web_view_delete_selection(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_delete_selection(m_web_view); } bool wxWebViewWebKit::HasSelection() const { - return webkit_web_view_has_selection(WEBKIT_WEB_VIEW(web_view)); + return webkit_web_view_has_selection(m_web_view); } void wxWebViewWebKit::SelectAll() { - webkit_web_view_select_all(WEBKIT_WEB_VIEW(web_view)); + webkit_web_view_select_all(m_web_view); } wxString wxWebViewWebKit::GetSelectedText() const @@ -852,7 +847,7 @@ wxString wxWebViewWebKit::GetSelectedText() const WebKitDOMDOMSelection* sel; WebKitDOMRange* range; - doc = webkit_web_view_get_dom_document(WEBKIT_WEB_VIEW(web_view)); + doc = webkit_web_view_get_dom_document(m_web_view); win = webkit_dom_document_get_default_view(WEBKIT_DOM_DOCUMENT(doc)); sel = webkit_dom_dom_window_get_selection(WEBKIT_DOM_DOM_WINDOW(win)); range = webkit_dom_dom_selection_get_range_at(WEBKIT_DOM_DOM_SELECTION(sel), @@ -871,7 +866,7 @@ wxString wxWebViewWebKit::GetSelectedSource() const WebKitDOMDocumentFragment* clone; WebKitDOMHTMLElement* html; - doc = webkit_web_view_get_dom_document(WEBKIT_WEB_VIEW(web_view)); + doc = webkit_web_view_get_dom_document(m_web_view); win = webkit_dom_document_get_default_view(WEBKIT_DOM_DOCUMENT(doc)); sel = webkit_dom_dom_window_get_selection(WEBKIT_DOM_DOM_WINDOW(win)); range = webkit_dom_dom_selection_get_range_at(WEBKIT_DOM_DOM_SELECTION(sel), @@ -892,7 +887,7 @@ void wxWebViewWebKit::ClearSelection() WebKitDOMDOMWindow* win; WebKitDOMDOMSelection* sel; - doc = webkit_web_view_get_dom_document(WEBKIT_WEB_VIEW(web_view)); + doc = webkit_web_view_get_dom_document(m_web_view); win = webkit_dom_document_get_default_view(WEBKIT_DOM_DOCUMENT(doc)); sel = webkit_dom_dom_window_get_selection(WEBKIT_DOM_DOM_WINDOW(win)); webkit_dom_dom_selection_remove_all_ranges(WEBKIT_DOM_DOM_SELECTION(sel)); @@ -904,7 +899,7 @@ wxString wxWebViewWebKit::GetPageText() const WebKitDOMDocument* doc; WebKitDOMHTMLElement* body; - doc = webkit_web_view_get_dom_document(WEBKIT_WEB_VIEW(web_view)); + doc = webkit_web_view_get_dom_document(m_web_view); body = webkit_dom_document_get_body(WEBKIT_DOM_DOCUMENT(doc)); return wxString(webkit_dom_html_element_get_inner_text(WEBKIT_DOM_HTML_ELEMENT(body)), wxConvUTF8); @@ -912,7 +907,7 @@ wxString wxWebViewWebKit::GetPageText() const void wxWebViewWebKit::RunScript(const wxString& javascript) { - webkit_web_view_execute_script(WEBKIT_WEB_VIEW(web_view), + webkit_web_view_execute_script(m_web_view, javascript.mb_str(wxConvUTF8)); }