 Chromium Code Reviews
 Chromium Code Reviews Issue 2467913002:
  Touch event flag should control only DOM event firing.  (Closed)
    
  
    Issue 2467913002:
  Touch event flag should control only DOM event firing.  (Closed) 
  | Index: third_party/WebKit/Source/web/DevToolsEmulator.cpp | 
| diff --git a/third_party/WebKit/Source/web/DevToolsEmulator.cpp b/third_party/WebKit/Source/web/DevToolsEmulator.cpp | 
| index 65527e56ad5d310e46ec8ec76ea516648561ba8c..42105c81ebf4d441c25e12e3f1f8de31af9b83ad 100644 | 
| --- a/third_party/WebKit/Source/web/DevToolsEmulator.cpp | 
| +++ b/third_party/WebKit/Source/web/DevToolsEmulator.cpp | 
| @@ -446,7 +446,7 @@ void DevToolsEmulator::setTouchEventEmulationEnabled(bool enabled) { | 
| if (m_touchEventEmulationEnabled == enabled) | 
| return; | 
| if (!m_touchEventEmulationEnabled) { | 
| - m_originalTouchEnabled = RuntimeEnabledFeatures::touchEnabled(); | 
| + m_originalTouchEnabled = RuntimeEnabledFeatures::touchEventAPIEnabled(); | 
| 
mustaq
2016/11/16 14:54:16
Please rename DevToolsEmulator::m_originalTouchEna
 
sunyunjia
2016/11/16 16:39:58
Done.
 | 
| m_originalDeviceSupportsMouse = | 
| m_webViewImpl->page()->settings().deviceSupportsMouse(); | 
| m_originalDeviceSupportsTouch = | 
| @@ -454,8 +454,8 @@ void DevToolsEmulator::setTouchEventEmulationEnabled(bool enabled) { | 
| m_originalMaxTouchPoints = | 
| m_webViewImpl->page()->settings().maxTouchPoints(); | 
| } | 
| - RuntimeEnabledFeatures::setTouchEnabled(enabled ? true | 
| - : m_originalTouchEnabled); | 
| + RuntimeEnabledFeatures::setTouchEventAPIEnabled( | 
| + enabled ? true : m_originalTouchEnabled); | 
| if (!m_originalDeviceSupportsTouch) { | 
| m_webViewImpl->page()->settings().setDeviceSupportsMouse( | 
| enabled ? false : m_originalDeviceSupportsMouse); |