RESOLVED FIXED Bug 37485
Make touch event support a RuntimeEnabledFeature 
https://bugs.webkit.org/show_bug.cgi?id=37485
Summary Make touch event support a RuntimeEnabledFeature
Garret Kelly
Reported 2010-04-12 21:20:07 PDT
In anticipation of being able to disable and enable this feature under certain platforms at runtime.
Attachments
Initial patch (6.90 KB, patch)
2010-04-12 21:22 PDT, Garret Kelly 		eric: review+
eric: commit-queue-
DetailsFormatted DiffDiff
Fix build issues in initial patch (7.73 KB, patch)
2010-04-22 07:41 PDT, Garret Kelly 		no flags
DetailsFormatted DiffDiff
Qt Fix (8.06 KB, patch)
2010-04-22 10:59 PDT, Garret Kelly 		ossy: review-
abarth: commit-queue-
DetailsFormatted DiffDiff
Remove platform entaglement (6.85 KB, patch)
2010-04-27 07:37 PDT, Garret Kelly 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Garret Kelly
Comment 1 2010-04-12 21:22:27 PDT
Created attachment 53220 [details] Initial patch
WebKit Review Bot
Comment 2 2010-04-12 21:57:11 PDT
Attachment 53220 [details] did not build on gtk: Build output: http://webkit-commit-queue.appspot.com/results/1669314
Darin Fisher (:fishd, Google)
Comment 3 2010-04-13 21:28:36 PDT
Comment on attachment 53220 [details] Initial patch Why is the default state enabled and not disabled? It looks like the qt/gtk bots are not happy.
Garret Kelly
Comment 4 2010-04-14 07:08:02 PDT
The default state is enabled to keep the Chromium build bots happy (but will be changed to disabled in a future patch that adds the enable to DRT for Chromium). The feature's not even enabled on Gtk, but both bots are failing because their include paths don't have WebCore/bindings/generic in them.
Eric Seidel (no email)
Comment 5 2010-04-21 18:27:34 PDT
Comment on attachment 53220 [details] Initial patch OK. Looks like this might break Qt/Gtk though so cq-
Garret Kelly
Comment 6 2010-04-22 07:41:10 PDT
Created attachment 54059 [details] Fix build issues in initial patch
Garret Kelly
Comment 7 2010-04-22 10:59:35 PDT
Created attachment 54075 [details] Qt Fix
Adam Barth
Comment 8 2010-04-22 12:25:02 PDT
Comment on attachment 54075 [details] Qt Fix Please make sure this works on Qt before landing.
Csaba Osztrogonác
Comment 9 2010-04-24 02:57:16 PDT
I tested the patch manually, but unfortunately I got a link-time error: obj/debug/Document.o: In function `WebCore::RuntimeEnabledFeatures::touchEnabled()': ../WebCore/bindings/generic/RuntimeEnabledFeatures.h:98: undefined reference to `WebCore::RuntimeEnabledFeatures::isTouchEnabled' collect2: ld returned 1 exit status make[1]: *** [../lib/libQtWebKit.so.4.6.0] Error 1 The root of the problem is that the initialization of RuntimeEnabledFeatures::isTouchEnabled (static member) can be found in WebCore/bindings/generic/RuntimeEnabledFeatures.cpp, but this file isn't builded for Qt port. Adding RuntimeEnabledFeatures.cpp wouldn't solve the problem, because of the following error: ../../../WebCore/bindings/generic/RuntimeEnabledFeatures.cpp: In static member function 'static bool WebCore::RuntimeEnabledFeatures::webSocketEnabled()': ../../../WebCore/bindings/generic/RuntimeEnabledFeatures.cpp:90: error: 'isAvailable' is not a member of 'WebCore::WebSocket' WebSocket::isAvailable() is implemented in WebCore/websockets/WebSocket.cpp guarded by #if USE(V8).
Csaba Osztrogonác
Comment 10 2010-04-24 02:59:02 PDT
Comment on attachment 54075 [details] Qt Fix r- -ed because this patch would brake Qt build. See my comment #9.
Garret Kelly
Comment 11 2010-04-27 07:37:07 PDT
Created attachment 54416 [details] Remove platform entaglement
WebKit Commit Bot
Comment 12 2010-04-27 18:09:49 PDT
Comment on attachment 54416 [details] Remove platform entaglement Clearing flags on attachment: 54416 Committed r58363: <http://trac.webkit.org/changeset/58363>
WebKit Commit Bot
Comment 13 2010-04-27 18:09:55 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.