Changeset 96680 in webkit

Timestamp:

Oct 5, 2011 1:03:01 AM (13 years ago)

Author:

Carlos Garcia Campos

Message:

[GTK] Use WKRetainPtr for WK types in WebKit2 GTK+
​https://bugs.webkit.org/show_bug.cgi?id=69404

Reviewed by Martin Robinson.

• UIProcess/API/gtk/WebKitWebContext.cpp:

(webkitWebContextFinalize):
(createDefaultWebContext):
(webkitWebContextGetWKContext):

• UIProcess/API/gtk/WebKitWebView.cpp:

(webkit_web_view_load_uri):
(webkit_web_view_load_alternate_html):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/API/gtk/WebKitWebContext.cpp (modified) (4 diffs)
• UIProcess/API/gtk/WebKitWebView.cpp (modified) (3 diffs)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2011-10-05  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] Use WKRetainPtr for WK types in WebKit2 GTK+
+        https://bugs.webkit.org/show_bug.cgi?id=69404
+
+        Reviewed by Martin Robinson.
+
+        * UIProcess/API/gtk/WebKitWebContext.cpp:
+        (webkitWebContextFinalize):
+        (createDefaultWebContext):
+        (webkitWebContextGetWKContext):
+        * UIProcess/API/gtk/WebKitWebView.cpp:
+        (webkit_web_view_load_uri):
+        (webkit_web_view_load_alternate_html):
+
 2011-10-04  Kent Tamura  <tkent@chromium.org>
 

trunk/Source/WebKit2/UIProcess/API/gtk/WebKitWebContext.cpp

@@ -23 +23 @@
 #include "WebKitWebContextPrivate.h"
 #include <WebKit2/WKContext.h>
+#include <WebKit2/WKRetainPtr.h>
 #include <WebKit2/WKType.h>
 
 struct _WebKitWebContextPrivate {
-    WKContextRef context;
+    WKRetainPtr<WKContextRef> context;
 };
 
@@ -33 +34 @@
 static void webkitWebContextFinalize(GObject* object)
 {
-    WebKitWebContext* context = WEBKIT_WEB_CONTEXT(object);
-
-    WKRelease(context->priv->context);
-    context->priv->context = 0;
-
-    context->priv->~WebKitWebContextPrivate();
-
+    WEBKIT_WEB_CONTEXT(object)->priv->~WebKitWebContextPrivate();
     G_OBJECT_CLASS(webkit_web_context_parent_class)->finalize(object);
 }
@@ -63 +58 @@
     WebKitWebContext* webContext = WEBKIT_WEB_CONTEXT(g_object_new(WEBKIT_TYPE_WEB_CONTEXT, NULL));
     webContext->priv->context = WKContextGetSharedProcessContext();
-    WKContextSetCacheModel(webContext->priv->context, kWKCacheModelPrimaryWebBrowser);
+    WKContextSetCacheModel(webContext->priv->context.get(), kWKCacheModelPrimaryWebBrowser);
     return webContext;
 }
@@ -84 +79 @@
     g_assert(WEBKIT_IS_WEB_CONTEXT(context));
 
-    return context->priv->context;
+    return context->priv->context.get();
 }
 

trunk/Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp

@@ -27 +27 @@
 #include "WebPageProxy.h"
 #include <WebKit2/WKBase.h>
+#include <WebKit2/WKRetainPtr.h>
 #include <WebKit2/WKURL.h>
 #include <wtf/gobject/GRefPtr.h>
@@ -217 +218 @@
     g_return_if_fail(uri);
 
-    WKURLRef url = WKURLCreateWithUTF8CString(uri);
-    WebPageProxy* page = webkitWebViewBaseGetPage(WEBKIT_WEB_VIEW_BASE(webView));
-    WKPageLoadURL(toAPI(page), url);
-    WKRelease(url);
+    WKRetainPtr<WKURLRef> url(AdoptWK, WKURLCreateWithUTF8CString(uri));
+    WebPageProxy* page = webkitWebViewBaseGetPage(WEBKIT_WEB_VIEW_BASE(webView));
+    WKPageLoadURL(toAPI(page), url.get());
 }
 
@@ -244 +244 @@
     g_return_if_fail(content);
 
-    WKStringRef htmlString = WKStringCreateWithUTF8CString(content);
-    WKURLRef baseURL = baseURI ? WKURLCreateWithUTF8CString(baseURI) : 0;
-    WKURLRef unreachableURL = unreachableURI ? WKURLCreateWithUTF8CString(unreachableURI) : 0;
-    WebPageProxy* page = webkitWebViewBaseGetPage(WEBKIT_WEB_VIEW_BASE(webView));
-    WKPageLoadAlternateHTMLString(toAPI(page), htmlString, baseURL, unreachableURL);
-    WKRelease(htmlString);
-    if (baseURL)
-        WKRelease(baseURL);
-    if (unreachableURL)
-        WKRelease(unreachableURL);
+    WKRetainPtr<WKStringRef> htmlString(AdoptWK, WKStringCreateWithUTF8CString(content));
+    WKRetainPtr<WKURLRef> baseURL = baseURI ? adoptWK(WKURLCreateWithUTF8CString(baseURI)) : 0;
+    WKRetainPtr<WKURLRef> unreachableURL = unreachableURI ? adoptWK(WKURLCreateWithUTF8CString(unreachableURI)) : 0;
+    WebPageProxy* page = webkitWebViewBaseGetPage(WEBKIT_WEB_VIEW_BASE(webView));
+    WKPageLoadAlternateHTMLString(toAPI(page), htmlString.get(), baseURL.get(), unreachableURL.get());
 }