Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

Allow to compile phantomjs without touch events in webkit #408

Closed
wants to merge 1 commit into from

Conversation

Havelock-Vetinari
Copy link

Added missing #if ENABLE(TOUCH_EVENTS)
for generated/JSDocument.* files

issue: http://code.google.com/p/phantomjs/issues/detail?id=375

Just compile with ./build.sh --qt-config "-D ENABLE_TOUCH_EVENTS=0"

Please cherry-pick this change to 1.8 branch for upcoming bug fix release, thx

@vitallium
Copy link
Collaborator

I think this pull request should set ENABLE_TOUCH_EVENTS to 0 by default when building QtWebkit.
You can do this in this file src/qt/src/3rdparty/webkit/Source/WebCore/features.pri

@Havelock-Vetinari
Copy link
Author

Added commit with disabled TOUCH_EVENTS by default. Still may opted in by --qt-config

@JamesMGreene
Copy link
Collaborator

For the record, I agree with @vitallium.

@Havelock-Vetinari
Copy link
Author

Should this pull request be fine tuned further in any way?

@JamesMGreene
Copy link
Collaborator

Squash it into a single commit. Other than that, might just have to wait on @ariya a bit as his family just welcomed a new baby girl into the world. :)

Fix compilation of webkit without the TOUCH_EVENTS

Added missing #if ENABLE(TOUCH_EVENTS)
for generated/JSDocument.* files

Disabled webkit touch events compilation by default

issue: http://code.google.com/p/phantomjs/issues/detail?id=375
@Havelock-Vetinari
Copy link
Author

Congratulation to @ariya :-)
Commits squashed and waiting

@ariya
Copy link
Owner

ariya commented Mar 7, 2013

Note: I won't land this one, see https://code.google.com/p/phantomjs/issues/detail?id=375#c11.

@ariya
Copy link
Owner

ariya commented Mar 19, 2013

Stashed as a patch in commit 63dc1e2.

@ariya ariya closed this Mar 19, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants