Bug 36750

Summary:

[Qt] Package build without touch support is broken

Product:

WebKit

Reporter:

Simon Hausmann <hausmann>

Component:

DOM

Assignee:

Simon Hausmann <hausmann>

Status:

CLOSED FIXED

Severity:

Blocker

CC:

gustavo, jturcotte, laszlo.gombos, webkit.review.bot, xan.lopez

Priority:

Keywords:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

37149

Bug Blocks:

35784

Attachments:

Description	Flags
Patch	laszlo.gombos: review+, eric: commit-queue-

Simon Hausmann

Reported 2010-03-29 02:32:35 PDT

The IDL files Document.idl, Element.idl and DOMWindow.idl define touch event related properties depending on the ENABLE_TOUCH_EVENTS feature define. The conditional build is done using an #ifdef inside the idl file. That makes the generated sources depend on the setting at idl generation time, which breaks the package build where all files are pre-generated but it's possible to disable certain features. For example touch support is disabled when compiling against Qt 4.5. The idl files should use the Conditional=TOUCH_EVENTS syntax, to place the #ifdefs into the generated code.

Attachments
Patch (4.07 KB, patch) 2010-03-29 02:42 PDT, Simon Hausmann	laszlo.gombos: review+ eric: commit-queue-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Simon Hausmann

Comment 1 2010-03-29 02:42:16 PDT

Created attachment 51894 [details] Patch

WebKit Review Bot

Comment 2 2010-03-29 03:15:01 PDT

Attachment 51894 [details] did not build on gtk: Build output: http://webkit-commit-queue.appspot.com/results/1592046

Simon Hausmann

Comment 3 2010-03-29 03:40:57 PDT

(In reply to comment #2) > Attachment 51894 [details] did not build on gtk: > Build output: http://webkit-commit-queue.appspot.com/results/1592046 It looks like the Gtk EWS needs a clean rebuild.

Laszlo Gombos

Comment 4 2010-03-30 18:27:01 PDT

Comment on attachment 51894 [details] Patch LGTM, r+. Have not set to the cq flag yet, as it seems that this needs some extra (manual) care when landed.

Eric Seidel (no email)

Comment 5 2010-04-02 11:50:56 PDT

Comment on attachment 51894 [details] Patch Extra manual care means cq-. Also looks like this might cause Gtk and Mac to fail to build, which would mean r-. I'm not sure why the mac failed since it didn't produce a log. Sadly we don't yet have a way to re-queue a patch on the EWS.

Simon Hausmann

Comment 6 2010-04-06 06:45:31 PDT

The error is real, the EWS worked correctly :-) I've filed 37149 to fix the bindings generator bug first before landing this.

Simon Hausmann

Comment 7 2010-04-07 05:50:00 PDT

<cherry-pick-for-backport: 57204>

Simon Hausmann

Comment 8 2010-04-07 06:23:46 PDT

Committed r57205: <http://trac.webkit.org/changeset/57205>

Simon Hausmann

Comment 9 2010-04-07 06:45:15 PDT

Revision r57204 cherry-picked into qtwebkit-2.0 with commit f544a8b519878c81f454876d80e1cd17a2f876ebRevision r57205 cherry-picked into qtwebkit-2.0 with commit 181033d9a0e7d1a6b1fed2dfff0cbc563e9a958c

Note You need to log in before you can comment on or make changes to this bug.