Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(91)

Unified Diff: third_party/WebKit/Source/core/input/TouchEventManager.cpp

Issue 2467913002: Touch event flag should control only DOM event firing. (Closed)
Patch Set: Change the string in "about://flags" Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/input/TouchEventManager.cpp
diff --git a/third_party/WebKit/Source/core/input/TouchEventManager.cpp b/third_party/WebKit/Source/core/input/TouchEventManager.cpp
index 8cbaab593786745703e2db82a91ca6fa5f5c0b07..d62bf2ec6078d31d78eaa2ee86bbea68137f1f02 100644
--- a/third_party/WebKit/Source/core/input/TouchEventManager.cpp
+++ b/third_party/WebKit/Source/core/input/TouchEventManager.cpp
@@ -19,6 +19,7 @@
#include "core/page/Page.h"
#include "platform/Histogram.h"
#include "platform/PlatformTouchEvent.h"
+#include "platform/RuntimeEnabledFeatures.h"
#include "wtf/CurrentTime.h"
#include "wtf/PtrUtil.h"
#include <memory>
@@ -487,6 +488,9 @@ class CurrentEventHolder {
WebInputEventResult TouchEventManager::handleTouchEvent(
const PlatformTouchEvent& event,
HeapVector<TouchInfo>& touchInfos) {
+ if (!RuntimeEnabledFeatures::touchAPIEnabled())
+ return WebInputEventResult::HandledSystem;
mustaq 2016/11/03 18:10:49 Should be NotHandled instead, to let the upstream
+
// Track the current event for the scope of this function.
CurrentEventHolder holder(m_currentEvent, event.type());

Powered by Google App Engine
This is Rietveld 408576698