Bug 193068 - [GTK] Add missing autocleanups
Summary: [GTK] Add missing autocleanups
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-01-01 17:29 PST by Michael Catanzaro
Modified: 2019-02-05 08:03 PST (History)
5 users (show)

See Also:


Attachments
Patch (2.46 KB, patch)
2019-01-01 17:38 PST, Michael Catanzaro
cgarcia: review+
cgarcia: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2019-01-01 17:29:02 PST
Add missing autocleanups

I wish we could switch to G_DECLARE_[FINAL,DERIVABLE]_TYPE so that we don't keep forgetting these, but I don't think it's possible without breaking ABI, and it would be insane to do with #ifdefs so we can't really do it for GTK+ 4 unless we drop GTK+ 3 at the same time... so we might be stuck with this legacy autocleanups file forever.
Comment 1 Michael Catanzaro 2019-01-01 17:38:50 PST
Created attachment 358177 [details]
Patch
Comment 2 Michael Catanzaro 2019-01-01 17:50:49 PST
(In reply to Michael Catanzaro from comment #0)
> so we might be stuck with this legacy autocleanups file forever.

FWIW I think it's reasonable to break ABI once every five years or so. This will cause zero problems for Linux distros as long as we don't make API changes. I want more padding in the WebKitWebView class struct anyway.
Comment 3 Michael Catanzaro 2019-01-02 13:42:51 PST
Er, I noticed WebKitOptionMenu is a GTK-specific type, and was surprised this didn't break the WPE EWS. But that's because it's a GTK-specific header. It looks like WPE does not have any autocleanups at all!
Comment 4 Carlos Garcia Campos 2019-01-09 00:59:45 PST
Comment on attachment 358177 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=358177&action=review

> Source/WebKit/ChangeLog:3
> +        [WPE][GTK] Add missing autocleanups

This is not affecting WPE
Comment 5 Michael Catanzaro 2019-01-09 07:06:56 PST
That's a problem in itself... how can we have failed to noticed that WPE does not offer autocleanups; that's awful. :( Adrian, why aren't you using these in cog?
Comment 6 Michael Catanzaro 2019-01-09 07:08:30 PST
Committed r239770: <https://trac.webkit.org/changeset/239770>