Bug 178125

Summary: [WPE] Remove the possibility of installing the old WebKit2 C API
Product: WebKit Reporter: Adrian Perez <aperez>
Component: WPE WebKitAssignee: Adrian Perez <aperez>
Severity: Normal CC: bugs-noreply, cgarcia, clopez, commit-queue, mcatanzaro, webkit-bug-importer, zan
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 176475, 178100, 178104    
Bug Blocks:    
Description Flags
Patch none

Description Adrian Perez 2017-10-10 04:26:45 PDT
We have the new GLib-based API, and we really don't want to maintain
two different sets of public API.
Comment 1 Adrian Perez 2017-10-10 04:30:46 PDT
Updated dependency bugs. This shouldn't be removed until we ensure
that the new GLib WPE API is usable.
Comment 2 Michael Catanzaro 2017-10-10 04:45:51 PDT
You can remove it now if you want; I'll be doing so at first opportunity in any case. We agreed that removing the C API and ensuring the GLib API works are both blockers for the first release and it doesn't matter which order that happens in.
Comment 3 Carlos Alberto Lopez Perez 2017-10-10 12:20:17 PDT
IIRC WPEWebKitLauncher doesn't build anymore with current trunk (even when enabling the deprecated headers).

dyz is works fine for me as a replacement for WPEWebKitLauncher.
Comment 4 Adrian Perez 2017-10-10 13:22:37 PDT
First I am going to take care of the rest of bugs which are
preventing usage of the new GLib-based API, and then I will
post the patch for this one ;-)

Thanks for the comments supporting the removal — I was aware
of the agreement, but I wanted to make sure it would not go
completely unannounced.
Comment 5 Adrian Perez 2017-10-10 16:03:34 PDT
Created attachment 323357 [details]
Comment 6 WebKit Commit Bot 2017-10-10 18:01:57 PDT
Comment on attachment 323357 [details]

Clearing flags on attachment: 323357

Committed r223162: <http://trac.webkit.org/changeset/223162>
Comment 7 WebKit Commit Bot 2017-10-10 18:01:58 PDT
All reviewed patches have been landed.  Closing bug.