Attachment 373644 Details for Bug 199579 – Patch

[patch] Patch

bug-199579-20190708130952.patch (text/plain), 7.37 KB, created by Michael Catanzaro on 2019-07-08 11:09:53 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Michael Catanzaro

Created: 2019-07-08 11:09:53 PDT

Size: 7.37 KB

patch

obsolete

>Subversion Revision: 247212
>diff --git a/Source/WTF/ChangeLog b/Source/WTF/ChangeLog
>index 71ce2ab600389a4e698f695a3df897766f3fae83..17c10ba6b3f5670141765a781b261a55678f9a73 100644
>--- a/Source/WTF/ChangeLog
>+++ b/Source/WTF/ChangeLog
>@@ -1,3 +1,18 @@
>+2019-07-08  Michael Catanzaro  <mcatanzaro@igalia.com>
>+
>+        [WPE][GTK] GUniqueOutPtr::release should return a raw pointer
>+        https://bugs.webkit.org/show_bug.cgi?id=199579
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        GUniqueOutPtr::release should release to a raw pointer, rather than a GUniquePtr. If a
>+        GUniquePtr is desired, it's trivial to construct one from the raw pointer, but all existing
>+        callsites under Source/ would rather have a raw pointer. Currently they have to call
>+        release().release() to get the raw pointer, which is annoying.
>+
>+        * wtf/glib/GUniquePtr.h:
>+        (WTF::GUniqueOutPtr::release):
>+
> 2019-07-08  Antoine Quint  <graouts@apple.com>
> 
>         [Pointer Events] Enable only on the most recent version of the supported iOS family
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 86ae6f5d27adcca1980f3d2de0e6293e89d14b35..34f5ebd75c9ffc20f996f32474f9b56a86d677cc 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,19 @@
>+2019-07-08  Michael Catanzaro  <mcatanzaro@igalia.com>
>+
>+        [WPE][GTK] GUniqueOutPtr::release should return a raw pointer
>+        https://bugs.webkit.org/show_bug.cgi?id=199579
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * NetworkProcess/glib/WebKitCachedResolver.cpp:
>+        (webkitCachedResolverLookupByNameAsync):
>+        (webkitCachedResolverLookupByNameWithFlagsAsync):
>+        * UIProcess/API/glib/WebKitFaviconDatabase.cpp:
>+        (processPendingIconsForPageURL):
>+        (webkit_favicon_database_get_favicon):
>+        * UIProcess/API/glib/WebKitUserContentFilterStore.cpp:
>+        (webkit_user_content_filter_store_save_from_file):
>+
> 2019-07-08  Antoine Quint  <graouts@apple.com>
> 
>         [Pointer Events] "touch-action: none" does not prevent double-tap-to-zoom
>diff --git a/Source/WTF/wtf/glib/GUniquePtr.h b/Source/WTF/wtf/glib/GUniquePtr.h
>index e1cff8ddea44f6da72fdf0f00f11ca188396a6ed..706ca2fa58ed4849610d8a70503c4a5603d34f56 100644
>--- a/Source/WTF/wtf/glib/GUniquePtr.h
>+++ b/Source/WTF/wtf/glib/GUniquePtr.h
>@@ -78,11 +78,11 @@ public:
>         return m_ptr;
>     }
> 
>-    GUniquePtr<T> release()
>+    T* release()
>     {
>         GUniquePtr<T> ptr(m_ptr);
>         m_ptr = nullptr;
>-        return ptr;
>+        return ptr.release();
>     }
> 
>     T& operator*() const
>diff --git a/Source/WebKit/NetworkProcess/glib/WebKitCachedResolver.cpp b/Source/WebKit/NetworkProcess/glib/WebKitCachedResolver.cpp
>index 3deab9e46ed1ffd85d1af2022b51adb1fbb21dfe..2d17c22c9e15ad96e5bab37a82672f836acd970a 100644
>--- a/Source/WebKit/NetworkProcess/glib/WebKitCachedResolver.cpp
>+++ b/Source/WebKit/NetworkProcess/glib/WebKitCachedResolver.cpp
>@@ -108,7 +108,7 @@ static void webkitCachedResolverLookupByNameAsync(GResolver* resolver, const cha
>             priv->cache.update(asyncData->hostname, addressListGListToVector(addressList));
>             g_task_return_pointer(task.get(), addressList, reinterpret_cast<GDestroyNotify>(g_resolver_free_addresses));
>         } else
>-            g_task_return_error(task.get(), error.release().release());
>+            g_task_return_error(task.get(), error.release());
>     }, task.leakRef());
> }
> 
>@@ -170,7 +170,7 @@ static void webkitCachedResolverLookupByNameWithFlagsAsync(GResolver* resolver,
>             priv->cache.update(asyncData->hostname, addressListGListToVector(addressList), asyncData->dnsCacheType);
>             g_task_return_pointer(task.get(), addressList, reinterpret_cast<GDestroyNotify>(g_resolver_free_addresses));
>         } else
>-            g_task_return_error(task.get(), error.release().release());
>+            g_task_return_error(task.get(), error.release());
>     }, task.leakRef());
> }
> 
>diff --git a/Source/WebKit/UIProcess/API/glib/WebKitFaviconDatabase.cpp b/Source/WebKit/UIProcess/API/glib/WebKitFaviconDatabase.cpp
>index e0164868171a5d8bee8a3e186ac6825c491d9700..218083e7697a51398f2ee094cdb068c20837b1fa 100644
>--- a/Source/WebKit/UIProcess/API/glib/WebKitFaviconDatabase.cpp
>+++ b/Source/WebKit/UIProcess/API/glib/WebKitFaviconDatabase.cpp
>@@ -175,7 +175,7 @@ static void processPendingIconsForPageURL(WebKitFaviconDatabase* database, const
>     for (size_t i = 0; i < pendingIconRequests->size(); ++i) {
>         GTask* task = pendingIconRequests->at(i).get();
>         if (error)
>-            g_task_return_error(task, error.release().release());
>+            g_task_return_error(task, error.release());
>         else {
>             GetFaviconSurfaceAsyncData* data = static_cast<GetFaviconSurfaceAsyncData*>(g_task_get_task_data(task));
>             data->icon = icon;
>@@ -413,7 +413,7 @@ void webkit_favicon_database_get_favicon(WebKitFaviconDatabase* database, const
>     data->shouldReleaseIconForPageURL = true;
> 
>     if (g_error_matches(error.get(), WEBKIT_FAVICON_DATABASE_ERROR, WEBKIT_FAVICON_DATABASE_ERROR_FAVICON_NOT_FOUND)) {
>-        g_task_return_error(task.get(), error.release().release());
>+        g_task_return_error(task.get(), error.release());
>         return;
>     }
> 
>diff --git a/Source/WebKit/UIProcess/API/glib/WebKitUserContentFilterStore.cpp b/Source/WebKit/UIProcess/API/glib/WebKitUserContentFilterStore.cpp
>index 6241bb5a0d0c512ea2f6b1711ebbc1a9afd04cb8..42bd997c7de93e34de5ebcbfc0199fc9a00765b8 100644
>--- a/Source/WebKit/UIProcess/API/glib/WebKitUserContentFilterStore.cpp
>+++ b/Source/WebKit/UIProcess/API/glib/WebKitUserContentFilterStore.cpp
>@@ -307,7 +307,7 @@ void webkit_user_content_filter_store_save_from_file(WebKitUserContentFilterStor
>             SaveTaskData* data = static_cast<SaveTaskData*>(g_task_get_task_data(task.get()));
>             webkitUserContentFilterStoreSaveBytes(WTFMove(task), WTFMove(data->identifier), GRefPtr<GBytes>(g_bytes_new_take(sourceData, sourceSize)));
>         } else
>-            g_task_return_error(task.get(), error.release().release());
>+            g_task_return_error(task.get(), error.release());
>     }, task.leakRef());
> }
> 
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 64f079d784818e71add312992451f8e53c7e7c64..f1a90d14269cce342a76dca1a7c8e7d3d4500b83 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,13 @@
>+2019-07-08  Michael Catanzaro  <mcatanzaro@igalia.com>
>+
>+        [WPE][GTK] GUniqueOutPtr::release should return a raw pointer
>+        https://bugs.webkit.org/show_bug.cgi?id=199579
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * TestWebKitAPI/Tests/WTF/glib/GUniquePtr.cpp:
>+        (TestWebKitAPI::TEST):
>+
> 2019-07-08  Antoine Quint  <graouts@apple.com>
> 
>         [Pointer Events] Enable only on the most recent version of the supported iOS family
>diff --git a/Tools/TestWebKitAPI/Tests/WTF/glib/GUniquePtr.cpp b/Tools/TestWebKitAPI/Tests/WTF/glib/GUniquePtr.cpp
>index 11f02c7b0f7fd70d83f6071d71ae2468362cb7a8..288591c8e3f32737c550931ecdc2e9fede25ceb1 100644
>--- a/Tools/TestWebKitAPI/Tests/WTF/glib/GUniquePtr.cpp
>+++ b/Tools/TestWebKitAPI/Tests/WTF/glib/GUniquePtr.cpp
>@@ -184,7 +184,7 @@ TEST(WTF_GUniquePtr, OutPtr)
>     {
>         GUniqueOutPtr<char> a;
>         returnOutChar(&a.outPtr());
>-        GUniquePtr<char> b = a.release();
>+        GUniquePtr<char> b(a.release());
>         ASSERT_STREQ(actual.str().c_str(), takeLogStr().c_str());
>         actual << "g_free(" << b.get() << ");";
>     }

Actions: View | Formatted Diff | Diff

Attachments on bug 199579: 373644 | 373654 | 373764