Bug 161438 - [GTK] Move GObject DOM bindings to WebKit2 layer and stop auto generating them
Summary: [GTK] Move GObject DOM bindings to WebKit2 layer and stop auto generating them
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks: 161440
  Show dependency treegraph
 
Reported: 2016-08-31 08:46 PDT by Carlos Garcia Campos
Modified: 2016-08-31 23:48 PDT (History)
3 users (show)

See Also:


Attachments
Patch (4.07 MB, patch)
2016-08-31 09:28 PDT, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff
Rebased again (4.07 MB, patch)
2016-08-31 09:46 PDT, Carlos Garcia Campos
mcatanzaro: review+
Details | Formatted Diff | Diff
Patch for landing (4.07 MB, patch)
2016-08-31 23:27 PDT, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2016-08-31 08:46:17 PDT
For now copy all the generated bindings there as a first step, and remove all scripts related to the auto-generated bindings.
Comment 1 Carlos Garcia Campos 2016-08-31 09:28:22 PDT
Created attachment 287514 [details]
Patch

I hope I haven't forgotten anything
Comment 2 Carlos Garcia Campos 2016-08-31 09:37:05 PDT
Patch is rebased to current trunk, I don't see anything wrong in the EWS output.
Comment 3 Carlos Garcia Campos 2016-08-31 09:46:00 PDT
Created attachment 287516 [details]
Rebased again
Comment 4 Carlos Garcia Campos 2016-08-31 09:56:03 PDT
Maybe the problem is that it's a big patch and others land after it's started to be processed and before it finishes, even if it applies anyway.
Comment 5 Michael Catanzaro 2016-08-31 10:08:15 PDT
WebKitDOMAnimatable.[cpp,h] looks like it can be deleted.

Unfortunately a lot of the files have "this file has been generated, DO NOT MODIFY!" and such at the top. You could try removing that and regenerating them all before removing the script again...?
Comment 6 Carlos Garcia Campos 2016-08-31 23:27:27 PDT
Created attachment 287602 [details]
Patch for landing
Comment 7 Carlos Garcia Campos 2016-08-31 23:38:51 PDT
It seems impossible to test this patch with EWS :-( I'll land and check the bots.
Comment 8 Carlos Garcia Campos 2016-08-31 23:48:16 PDT
Committed r205280: <http://trac.webkit.org/changeset/205280>